)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"39dd6188cfb53635ef5b296b8b83869951f7da16","unresolved":false,"context_lines":[{"line_number":18,"context_line":"calculation of the provisioned_capacity_gb in the scheduler."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Change-Id: Ic106dbcae8ceaac265b710756ab1874e445ca826"},{"line_number":21,"context_line":"Partial-Bug: #1714209"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"bf57530f_b32626b7","line":21,"updated":"2017-09-20 19:26:39.000000000","message":"Why is this partial?","commit_id":"f2f717c2a6a10779042990d41071832e6701c1c1"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"74ede970bfe67ddb93f040ed46b412d95fb9faac","unresolved":false,"context_lines":[{"line_number":18,"context_line":"calculation of the provisioned_capacity_gb in the scheduler."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Change-Id: Ic106dbcae8ceaac265b710756ab1874e445ca826"},{"line_number":21,"context_line":"Partial-Bug: #1714209"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9f5c4f37_271e7b0c","line":21,"in_reply_to":"9f5c4f37_fbca574d","updated":"2017-09-21 17:56:24.000000000","message":"Opened a new bug for E-Series: https://bugs.launchpad.net/cinder/+bug/1718739\nNow this is a complete fix for https://bugs.launchpad.net/cinder/+bug/1714209","commit_id":"f2f717c2a6a10779042990d41071832e6701c1c1"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"f843b116a50dbf95755a62b335d3d4fdea45d155","unresolved":false,"context_lines":[{"line_number":18,"context_line":"calculation of the provisioned_capacity_gb in the scheduler."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Change-Id: Ic106dbcae8ceaac265b710756ab1874e445ca826"},{"line_number":21,"context_line":"Partial-Bug: #1714209"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9f5c4f37_fbca574d","line":21,"in_reply_to":"bf57530f_b32626b7","updated":"2017-09-21 02:20:38.000000000","message":"because the bug refers to the E-Series driver as well, which is not fixed in this patch, I\u0027ll have another fix for the E-Series driver - this fix only affects the ONTAP family (cDOT/7mode - all 3 protocols: iSCSI, NFS, FC)","commit_id":"f2f717c2a6a10779042990d41071832e6701c1c1"}],"cinder/volume/drivers/netapp/dataontap/block_7mode.py":[{"author":{"_account_id":18529,"name":"Dongdong Pei","email":"peidongdong121@163.com","username":"dongdongpei"},"change_message_id":"c10467fcc649826a7d002221fd590f1b27fe1709","unresolved":false,"context_lines":[{"line_number":341,"context_line":"            free /\u003d units.Gi"},{"line_number":342,"context_line":"            pool[\u0027free_capacity_gb\u0027] \u003d na_utils.round_down(free, \u00270.01\u0027)"},{"line_number":343,"context_line":""},{"line_number":344,"context_line":"            pool[\u0027provisioned_capacity_gb\u0027] \u003d (round("},{"line_number":345,"context_line":"                pool[\u0027total_capacity_gb\u0027] - pool[\u0027free_capacity_gb\u0027], 2))"},{"line_number":346,"context_line":""},{"line_number":347,"context_line":"            thick \u003d ("}],"source_content_type":"text/x-python","patch_set":3,"id":"7f515b1d_49ec618e","side":"PARENT","line":344,"range":{"start_line":344,"start_character":52,"end_line":344,"end_character":53},"updated":"2017-09-28 01:33:54.000000000","message":"If you delete this line, scheduler will take allocated_capacity_gb as provisioned_capacity_gb, which is inaccurate.\nWhy not provisioned_capacity_gb \u003d (total_capacity_gb - free_capacity_gb) * max_over_subscription_ratio","commit_id":"6c236bca3ddebb8b70e96bf41a275fe026bfb995"},{"author":{"_account_id":18529,"name":"Dongdong Pei","email":"peidongdong121@163.com","username":"dongdongpei"},"change_message_id":"359b24ef63b1afeda7ae1401f85da70f39d2a16e","unresolved":false,"context_lines":[{"line_number":341,"context_line":"            free /\u003d units.Gi"},{"line_number":342,"context_line":"            pool[\u0027free_capacity_gb\u0027] \u003d na_utils.round_down(free, \u00270.01\u0027)"},{"line_number":343,"context_line":""},{"line_number":344,"context_line":"            pool[\u0027provisioned_capacity_gb\u0027] \u003d (round("},{"line_number":345,"context_line":"                pool[\u0027total_capacity_gb\u0027] - pool[\u0027free_capacity_gb\u0027], 2))"},{"line_number":346,"context_line":""},{"line_number":347,"context_line":"            thick \u003d ("}],"source_content_type":"text/x-python","patch_set":3,"id":"7f515b1d_b7e320d9","side":"PARENT","line":344,"range":{"start_line":344,"start_character":52,"end_line":344,"end_character":53},"in_reply_to":"7f515b1d_0fce6981","updated":"2017-09-28 14:30:12.000000000","message":"You are right. I have mistaken used capacity with provisioned capacity.","commit_id":"6c236bca3ddebb8b70e96bf41a275fe026bfb995"},{"author":{"_account_id":2417,"name":"Ben Swartzlander","email":"ben@swartzlander.org","username":"bswartz"},"change_message_id":"f278ca17f89bfa34c1a998689e1769893cda989b","unresolved":false,"context_lines":[{"line_number":341,"context_line":"            free /\u003d units.Gi"},{"line_number":342,"context_line":"            pool[\u0027free_capacity_gb\u0027] \u003d na_utils.round_down(free, \u00270.01\u0027)"},{"line_number":343,"context_line":""},{"line_number":344,"context_line":"            pool[\u0027provisioned_capacity_gb\u0027] \u003d (round("},{"line_number":345,"context_line":"                pool[\u0027total_capacity_gb\u0027] - pool[\u0027free_capacity_gb\u0027], 2))"},{"line_number":346,"context_line":""},{"line_number":347,"context_line":"            thick \u003d ("}],"source_content_type":"text/x-python","patch_set":3,"id":"7f515b1d_0fce6981","side":"PARENT","line":344,"range":{"start_line":344,"start_character":52,"end_line":344,"end_character":53},"in_reply_to":"7f515b1d_49ec618e","updated":"2017-09-28 03:24:05.000000000","message":"It\u0027s not inaccurate, it\u0027s exactly what we want to do. Please read about provisioned and allocated capacity and what they mean. Provisioned capacity is explicitly NOT total minus free.","commit_id":"6c236bca3ddebb8b70e96bf41a275fe026bfb995"}],"releasenotes/notes/bug-1714209-netapp-ontap-drivers-oversubscription-issue-c4655b9c4858d7c6.yaml":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"39dd6188cfb53635ef5b296b8b83869951f7da16","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - The ONTAP drivers (\"7mode\" and \"cmode\") have been fixed to not report"},{"line_number":4,"context_line":"    consumed space as \"provisioned_capacity_gb\". They instead rely on"},{"line_number":5,"context_line":"    the cinder scheduler\u0027s calculation of \"provisioned_capacity_gb\". This"},{"line_number":6,"context_line":"    fixes the oversubscription miscalculations with the ONTAP drivers."}],"source_content_type":"text/x-yaml","patch_set":1,"id":"bf57530f_d36f5ac3","line":3,"updated":"2017-09-20 19:26:39.000000000","message":"?: What about nfs mode?  Hasn\u0027t it also been fixed?","commit_id":"f2f717c2a6a10779042990d41071832e6701c1c1"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"f843b116a50dbf95755a62b335d3d4fdea45d155","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - The ONTAP drivers (\"7mode\" and \"cmode\") have been fixed to not report"},{"line_number":4,"context_line":"    consumed space as \"provisioned_capacity_gb\". They instead rely on"},{"line_number":5,"context_line":"    the cinder scheduler\u0027s calculation of \"provisioned_capacity_gb\". This"},{"line_number":6,"context_line":"    fixes the oversubscription miscalculations with the ONTAP drivers."}],"source_content_type":"text/x-yaml","patch_set":1,"id":"9f5c4f37_bb75fffc","line":3,"in_reply_to":"bf57530f_d36f5ac3","updated":"2017-09-21 02:20:38.000000000","message":"It has, the driver modes refer to flavors of the ONTAP operating system. Each of these flavors (7mode and cmode (aka cDOT)) support three protocols: iSCSI, FC and NFS.","commit_id":"f2f717c2a6a10779042990d41071832e6701c1c1"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"39dd6188cfb53635ef5b296b8b83869951f7da16","unresolved":false,"context_lines":[{"line_number":3,"context_line":"  - The ONTAP drivers (\"7mode\" and \"cmode\") have been fixed to not report"},{"line_number":4,"context_line":"    consumed space as \"provisioned_capacity_gb\". They instead rely on"},{"line_number":5,"context_line":"    the cinder scheduler\u0027s calculation of \"provisioned_capacity_gb\". This"},{"line_number":6,"context_line":"    fixes the oversubscription miscalculations with the ONTAP drivers."}],"source_content_type":"text/x-yaml","patch_set":1,"id":"bf57530f_9340c237","line":6,"updated":"2017-09-20 19:26:39.000000000","message":"-1: Missing upgrade notes, since max_over_subscription_ratio will need to be adjusted or deployments may fail to create new volumes.","commit_id":"f2f717c2a6a10779042990d41071832e6701c1c1"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"74ede970bfe67ddb93f040ed46b412d95fb9faac","unresolved":false,"context_lines":[{"line_number":3,"context_line":"  - The ONTAP drivers (\"7mode\" and \"cmode\") have been fixed to not report"},{"line_number":4,"context_line":"    consumed space as \"provisioned_capacity_gb\". They instead rely on"},{"line_number":5,"context_line":"    the cinder scheduler\u0027s calculation of \"provisioned_capacity_gb\". This"},{"line_number":6,"context_line":"    fixes the oversubscription miscalculations with the ONTAP drivers."}],"source_content_type":"text/x-yaml","patch_set":1,"id":"9f5c4f37_e72aa38e","line":6,"in_reply_to":"9f5c4f37_dbd673c4","updated":"2017-09-21 17:56:24.000000000","message":"Took a stab at it. Please let me know what you think of the update.","commit_id":"f2f717c2a6a10779042990d41071832e6701c1c1"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"f843b116a50dbf95755a62b335d3d4fdea45d155","unresolved":false,"context_lines":[{"line_number":3,"context_line":"  - The ONTAP drivers (\"7mode\" and \"cmode\") have been fixed to not report"},{"line_number":4,"context_line":"    consumed space as \"provisioned_capacity_gb\". They instead rely on"},{"line_number":5,"context_line":"    the cinder scheduler\u0027s calculation of \"provisioned_capacity_gb\". This"},{"line_number":6,"context_line":"    fixes the oversubscription miscalculations with the ONTAP drivers."}],"source_content_type":"text/x-yaml","patch_set":1,"id":"9f5c4f37_dbd673c4","line":6,"in_reply_to":"bf57530f_9340c237","updated":"2017-09-21 02:20:38.000000000","message":"Absolutely, How would you suggest I phrase that? On upgrade, deployers will see the correct behavior as determined by the design of Cinder Oversubscription.","commit_id":"f2f717c2a6a10779042990d41071832e6701c1c1"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"c9e14562cb67fd86ad2816485aef8db9ea715c24","unresolved":false,"context_lines":[{"line_number":8,"context_line":"    (iSCSI/FC/NFS)."},{"line_number":9,"context_line":"upgrade:"},{"line_number":10,"context_line":"  - If using the NetApp ONTAP drivers (7mode/cmode), the configuration value"},{"line_number":11,"context_line":"    for \"max_over_subscription_ratio\" may need to be adjusted to"},{"line_number":12,"context_line":"    take advantage of the bugfix that aligns the oversubscription logic"},{"line_number":13,"context_line":"    within these drivers with the design in OpenStack Cinder. See"},{"line_number":14,"context_line":"    documentation `here \u003chttps://docs.openstack"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"7f515b1d_9b92e39d","line":11,"range":{"start_line":11,"start_character":50,"end_line":11,"end_character":61},"updated":"2017-09-26 09:13:09.000000000","message":"be increased","commit_id":"8d67e5a49322ede4ef5f07aaad1808d00719d0e0"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"c9e14562cb67fd86ad2816485aef8db9ea715c24","unresolved":false,"context_lines":[{"line_number":9,"context_line":"upgrade:"},{"line_number":10,"context_line":"  - If using the NetApp ONTAP drivers (7mode/cmode), the configuration value"},{"line_number":11,"context_line":"    for \"max_over_subscription_ratio\" may need to be adjusted to"},{"line_number":12,"context_line":"    take advantage of the bugfix that aligns the oversubscription logic"},{"line_number":13,"context_line":"    within these drivers with the design in OpenStack Cinder. See"},{"line_number":14,"context_line":"    documentation `here \u003chttps://docs.openstack"},{"line_number":15,"context_line":"    .org/cinder/latest/admin/blockstorage-over-subscription.html\u003e`_."}],"source_content_type":"text/x-yaml","patch_set":2,"id":"7f515b1d_fb7317b0","line":12,"range":{"start_line":12,"start_character":4,"end_line":12,"end_character":32},"updated":"2017-09-26 09:13:09.000000000","message":"This makes it look like it\u0027s a nice thing to do that will  add some benefit, where as I see it what could happen is that if you don\u0027t change the configuration before restarting the upgraded service, you may not be able to create new volumes.","commit_id":"8d67e5a49322ede4ef5f07aaad1808d00719d0e0"}]}