)]}'
{"specs/pike/approved/fix-to-return-server-groups-quota-in-quota-class.rst":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"a245612dabccf4d102d2e4a35964434b166d13ef","unresolved":false,"context_lines":[{"line_number":35,"context_line":"This spec propose to fix this bug as microversion by adding `server_groups`"},{"line_number":36,"context_line":"and `server_group_memeber` back to the v2.1 API."},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"For legacy v2 API, we added those two quotas back to the API directly."},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"Alternatives"},{"line_number":41,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"df140735_09f5b557","line":38,"range":{"start_line":38,"start_character":0,"end_line":38,"end_character":70},"updated":"2017-05-30 04:45:54.000000000","message":"not sure we should do this directly? if yes then we should backport it for v2 only.\n\nv2 dropped those keys since newton (when legacy v2 code was removed) and now we will add them directly which seems API changes from Newton to Pike without discovery. \n\nDo we really need to fix for v2?","commit_id":"e07b0de27df97854a1f220355f38434d7b1678ce"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"38ee5119c956ea6a64e95c51bdc07a6feb80dc78","unresolved":false,"context_lines":[{"line_number":35,"context_line":"This spec propose to fix this bug as microversion by adding `server_groups`"},{"line_number":36,"context_line":"and `server_group_memeber` back to the v2.1 API."},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"For legacy v2 API, we added those two quotas back to the API directly."},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"Alternatives"},{"line_number":41,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"df140735_8a81924d","line":38,"range":{"start_line":38,"start_character":0,"end_line":38,"end_character":70},"in_reply_to":"df140735_09f5b557","updated":"2017-05-31 07:32:54.000000000","message":"It is also strange for legacy v2, or we can remove the os-server-group-quota extension from the extension API.\n\nWe do similar thing for the vif-net bug https://review.openstack.org/#/c/198944/2","commit_id":"e07b0de27df97854a1f220355f38434d7b1678ce"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"a245612dabccf4d102d2e4a35964434b166d13ef","unresolved":false,"context_lines":[{"line_number":41,"context_line":"------------"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"Fix this bug without microversion for v2.1 API, it will lead to the API"},{"line_number":44,"context_line":"interoperability."},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"Data model impact"},{"line_number":47,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"df140735_294cf99c","line":44,"range":{"start_line":44,"start_character":0,"end_line":44,"end_character":16},"updated":"2017-05-30 04:45:54.000000000","message":"interoperability issue","commit_id":"e07b0de27df97854a1f220355f38434d7b1678ce"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"f63307f219a5598ea1e4b4c7ec46549bf942293a","unresolved":false,"context_lines":[{"line_number":21,"context_line":"`server_groups` and `server_group_members` in the quota-classes API. But"},{"line_number":22,"context_line":"during the porting the v2 extensions to the v2.1 API, this extension was"},{"line_number":23,"context_line":"missed. It leads to the quota `server_groups` and `server_group_members`"},{"line_number":24,"context_line":"missing in the repsonse of quota-classes API."},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"Use Cases"},{"line_number":27,"context_line":"---------"}],"source_content_type":"text/x-rst","patch_set":4,"id":"df140735_be49618c","line":24,"range":{"start_line":24,"start_character":15,"end_line":24,"end_character":23},"updated":"2017-06-07 13:43:09.000000000","message":"response","commit_id":"4b92f9b61d8c8aada6b91df83e4caa458277b9f8"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"f63307f219a5598ea1e4b4c7ec46549bf942293a","unresolved":false,"context_lines":[{"line_number":35,"context_line":"This spec propose to fix this bug as microversion by adding `server_groups`"},{"line_number":36,"context_line":"and `server_group_memeber` back to the v2.1 API."},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"For legacy v2 API, the quotas for server groups won\u0027t be added back to the API."},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"But the extension entry `os-server-group-quotas` in the extension API will be"},{"line_number":41,"context_line":"removed both in the v2.1 API and legacy v2 API."}],"source_content_type":"text/x-rst","patch_set":4,"id":"df140735_9e0c5d43","line":38,"range":{"start_line":38,"start_character":0,"end_line":38,"end_character":79},"updated":"2017-06-07 13:43:09.000000000","message":"Because the code is already gone, so there is no way to put it back. Or do you mean, we won\u0027t do this in the v2.1-\u003ev2.46 APIs?","commit_id":"4b92f9b61d8c8aada6b91df83e4caa458277b9f8"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"f2fe5ae409f3795f8f62a2adfb5b5a2779486eba","unresolved":false,"context_lines":[{"line_number":35,"context_line":"This spec propose to fix this bug as microversion by adding `server_groups`"},{"line_number":36,"context_line":"and `server_group_memeber` back to the v2.1 API."},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"For legacy v2 API, the quotas for server groups won\u0027t be added back to the API."},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"But the extension entry `os-server-group-quotas` in the extension API will be"},{"line_number":41,"context_line":"removed both in the v2.1 API and legacy v2 API."}],"source_content_type":"text/x-rst","patch_set":4,"id":"df140735_49c5ed5a","line":38,"range":{"start_line":38,"start_character":0,"end_line":38,"end_character":79},"in_reply_to":"df140735_9e0c5d43","updated":"2017-06-07 13:59:16.000000000","message":"We run v2.1 API code as legacy v2 compatible mode under the endpoint \u0027/v2\u0027. I mean we won\u0027t fix that legacy v2 API.","commit_id":"4b92f9b61d8c8aada6b91df83e4caa458277b9f8"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"f63307f219a5598ea1e4b4c7ec46549bf942293a","unresolved":false,"context_lines":[{"line_number":62,"context_line":"    GET /os-quota-class-sets/{quota_class}"},{"line_number":63,"context_line":"    PUT /os-quota-class-sets/{quota-class}"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"The `os-server-group-quotas` extension entry in the extension API will be"},{"line_number":66,"context_line":"removed both in the v2.1 API and legacy v2 API."},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"Security impact"},{"line_number":69,"context_line":"---------------"}],"source_content_type":"text/x-rst","patch_set":4,"id":"df140735_49184d74","line":66,"range":{"start_line":65,"start_character":0,"end_line":66,"end_character":47},"updated":"2017-06-07 13:43:09.000000000","message":"Isn\u0027t this really kind of blueprint api-no-more-extensions-pike? Maybe it makes more sense to lump the extension remove in with this microversion for context.","commit_id":"4b92f9b61d8c8aada6b91df83e4caa458277b9f8"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"f2fe5ae409f3795f8f62a2adfb5b5a2779486eba","unresolved":false,"context_lines":[{"line_number":62,"context_line":"    GET /os-quota-class-sets/{quota_class}"},{"line_number":63,"context_line":"    PUT /os-quota-class-sets/{quota-class}"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"The `os-server-group-quotas` extension entry in the extension API will be"},{"line_number":66,"context_line":"removed both in the v2.1 API and legacy v2 API."},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"Security impact"},{"line_number":69,"context_line":"---------------"}],"source_content_type":"text/x-rst","patch_set":4,"id":"df140735_09bc95b9","line":66,"range":{"start_line":65,"start_character":0,"end_line":66,"end_character":47},"in_reply_to":"df140735_49184d74","updated":"2017-06-07 13:59:16.000000000","message":"I mean remove this entry https://github.com/openstack/nova/blob/master/doc/api_samples/extension-info/extensions-list-resp.json#L652 from the extension info API.\n\nThe legacy v2 client code based on this entry to ensure whether the os-server-group-quotas supported or not in the quota-class and quota-set API.\n\nBut I missed the point, the quota-sets API still returned the server-group-quotas...that is really inconsistent whatever we have os-server-group-quotas entry in the exntesion-info API.....\n\nMaybe we should just leave it alone.","commit_id":"4b92f9b61d8c8aada6b91df83e4caa458277b9f8"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"f63307f219a5598ea1e4b4c7ec46549bf942293a","unresolved":false,"context_lines":[{"line_number":78,"context_line":"Other end user impact"},{"line_number":79,"context_line":"---------------------"},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"For query the default for `server_groups` and `server_group_memeber` by the"},{"line_number":82,"context_line":"new microversion or using the `quota-set` API. The `quota-set` API only"},{"line_number":83,"context_line":"return specific project\u0027s default quota."},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"Performance Impact"},{"line_number":86,"context_line":"------------------"}],"source_content_type":"text/x-rst","patch_set":4,"id":"df140735_097a5587","line":83,"range":{"start_line":81,"start_character":0,"end_line":83,"end_character":40},"updated":"2017-06-07 13:43:09.000000000","message":"This is a bit confusing, I\u0027m not sure if you\u0027re trying to explain the workaround, which I think we said we\u0027d document.\n\nFor this section, I think we\u0027d just say this python-novaclient will also need to be updated to handle the new microversion for the python API binding and the \"nova quota-class *\" CLIs.","commit_id":"4b92f9b61d8c8aada6b91df83e4caa458277b9f8"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"f2fe5ae409f3795f8f62a2adfb5b5a2779486eba","unresolved":false,"context_lines":[{"line_number":78,"context_line":"Other end user impact"},{"line_number":79,"context_line":"---------------------"},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"For query the default for `server_groups` and `server_group_memeber` by the"},{"line_number":82,"context_line":"new microversion or using the `quota-set` API. The `quota-set` API only"},{"line_number":83,"context_line":"return specific project\u0027s default quota."},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"Performance Impact"},{"line_number":86,"context_line":"------------------"}],"source_content_type":"text/x-rst","patch_set":4,"id":"df140735_e98161c5","line":83,"range":{"start_line":81,"start_character":0,"end_line":83,"end_character":40},"in_reply_to":"df140735_097a5587","updated":"2017-06-07 13:59:16.000000000","message":"Done","commit_id":"4b92f9b61d8c8aada6b91df83e4caa458277b9f8"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"f63307f219a5598ea1e4b4c7ec46549bf942293a","unresolved":false,"context_lines":[{"line_number":113,"context_line":"----------"},{"line_number":114,"context_line":""},{"line_number":115,"context_line":"* Fix the v2.1 API by new microversion"},{"line_number":116,"context_line":"* Fix the legacy v2 API directly."},{"line_number":117,"context_line":""},{"line_number":118,"context_line":"Dependencies"},{"line_number":119,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":4,"id":"df140735_e9f16103","line":116,"range":{"start_line":116,"start_character":2,"end_line":116,"end_character":33},"updated":"2017-06-07 13:43:09.000000000","message":"Above it says we aren\u0027t fixing the v2 API.","commit_id":"4b92f9b61d8c8aada6b91df83e4caa458277b9f8"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"f2fe5ae409f3795f8f62a2adfb5b5a2779486eba","unresolved":false,"context_lines":[{"line_number":113,"context_line":"----------"},{"line_number":114,"context_line":""},{"line_number":115,"context_line":"* Fix the v2.1 API by new microversion"},{"line_number":116,"context_line":"* Fix the legacy v2 API directly."},{"line_number":117,"context_line":""},{"line_number":118,"context_line":"Dependencies"},{"line_number":119,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":4,"id":"df140735_c9869ddc","line":116,"range":{"start_line":116,"start_character":2,"end_line":116,"end_character":33},"in_reply_to":"df140735_e9f16103","updated":"2017-06-07 13:59:16.000000000","message":"Done","commit_id":"4b92f9b61d8c8aada6b91df83e4caa458277b9f8"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"f63307f219a5598ea1e4b4c7ec46549bf942293a","unresolved":false,"context_lines":[{"line_number":128,"context_line":"Documentation Impact"},{"line_number":129,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":130,"context_line":""},{"line_number":131,"context_line":"Update the `api-ref`_ about this bug fix. The reno is required to note this"},{"line_number":132,"context_line":"bug."},{"line_number":133,"context_line":""},{"line_number":134,"context_line":"References"},{"line_number":135,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":4,"id":"df140735_495f2d0e","line":132,"range":{"start_line":131,"start_character":0,"end_line":132,"end_character":4},"updated":"2017-06-07 13:43:09.000000000","message":"I think this is where we mention that we\u0027ll be adding a note to the API reference about the bug and the workaround to use the os-quota-sets API if you\u0027re using a microversion lower than the new one that fixes this bug. Maybe that\u0027s what you meant here anyway.","commit_id":"4b92f9b61d8c8aada6b91df83e4caa458277b9f8"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"f2fe5ae409f3795f8f62a2adfb5b5a2779486eba","unresolved":false,"context_lines":[{"line_number":128,"context_line":"Documentation Impact"},{"line_number":129,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":130,"context_line":""},{"line_number":131,"context_line":"Update the `api-ref`_ about this bug fix. The reno is required to note this"},{"line_number":132,"context_line":"bug."},{"line_number":133,"context_line":""},{"line_number":134,"context_line":"References"},{"line_number":135,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":4,"id":"df140735_0966d552","line":132,"range":{"start_line":131,"start_character":0,"end_line":132,"end_character":4},"in_reply_to":"df140735_495f2d0e","updated":"2017-06-07 13:59:16.000000000","message":"Done","commit_id":"4b92f9b61d8c8aada6b91df83e4caa458277b9f8"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"f63307f219a5598ea1e4b4c7ec46549bf942293a","unresolved":false,"context_lines":[{"line_number":135,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":136,"context_line":""},{"line_number":137,"context_line":".. _api-ref: http://developer.openstack.org/api-ref/compute/"},{"line_number":138,"context_line":".. launchpad bug link: https://bugs.launchpad.net/nova/+bug/1693168"},{"line_number":139,"context_line":""},{"line_number":140,"context_line":"History"},{"line_number":141,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":4,"id":"df140735_c9a8dddc","line":138,"range":{"start_line":138,"start_character":3,"end_line":138,"end_character":21},"updated":"2017-06-07 13:43:09.000000000","message":"This isn\u0027t linked from anything above.","commit_id":"4b92f9b61d8c8aada6b91df83e4caa458277b9f8"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"f2fe5ae409f3795f8f62a2adfb5b5a2779486eba","unresolved":false,"context_lines":[{"line_number":135,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":136,"context_line":""},{"line_number":137,"context_line":".. _api-ref: http://developer.openstack.org/api-ref/compute/"},{"line_number":138,"context_line":".. launchpad bug link: https://bugs.launchpad.net/nova/+bug/1693168"},{"line_number":139,"context_line":""},{"line_number":140,"context_line":"History"},{"line_number":141,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":4,"id":"df140735_694e71c2","line":138,"range":{"start_line":138,"start_character":3,"end_line":138,"end_character":21},"in_reply_to":"df140735_c9a8dddc","updated":"2017-06-07 13:59:16.000000000","message":"I will fix it.","commit_id":"4b92f9b61d8c8aada6b91df83e4caa458277b9f8"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"04e1e1ec156f93e117141bdba3bde7448b380886","unresolved":false,"context_lines":[{"line_number":135,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":136,"context_line":""},{"line_number":137,"context_line":".. _api-ref: http://developer.openstack.org/api-ref/compute/"},{"line_number":138,"context_line":".. launchpad bug link: https://bugs.launchpad.net/nova/+bug/1693168"},{"line_number":139,"context_line":""},{"line_number":140,"context_line":"History"},{"line_number":141,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":4,"id":"df140735_69be7100","line":138,"range":{"start_line":138,"start_character":3,"end_line":138,"end_character":21},"in_reply_to":"df140735_c9a8dddc","updated":"2017-06-07 13:44:08.000000000","message":"http://docs-draft.openstack.org/19/468619/4/check/gate-nova-specs-docs-ubuntu-xenial/4de9dda//doc/build/html/specs/pike/approved/fix-to-return-server-groups-quota-in-quota-class.html#references","commit_id":"4b92f9b61d8c8aada6b91df83e4caa458277b9f8"}]}
