)]}'
{"specs/liberty/approved/nova-api-remove-extensions.rst":[{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"e5a048c846ca5076afcc341ef2b407bac54ee7a4","unresolved":false,"context_lines":[{"line_number":40,"context_line":""},{"line_number":41,"context_line":"There are couple things we need to do:"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"* Remove all the extension white/black list options. This also need tuning the"},{"line_number":44,"context_line":"  tests, because our tests based on those option to test specific plugin."},{"line_number":45,"context_line":"* Remove extension discover entry from policy.json. This is another kind of"},{"line_number":46,"context_line":"  extension enable/disable configuration based on permission."}],"source_content_type":"text/x-rst","patch_set":1,"id":"9a41bdd9_76c903ac","line":43,"updated":"2015-07-20 07:37:04.000000000","message":"s/tuning/tune","commit_id":"87df68673be941005b2497ee729ddd0d4cb7e893"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"a167ef79cd161aa8a6678ba85f38be6bc8b45910","unresolved":false,"context_lines":[{"line_number":40,"context_line":""},{"line_number":41,"context_line":"There are couple things we need to do:"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"* Remove all the extension white/black list options. This also need tuning the"},{"line_number":44,"context_line":"  tests, because our tests based on those option to test specific plugin."},{"line_number":45,"context_line":"* Remove extension discover entry from policy.json. This is another kind of"},{"line_number":46,"context_line":"  extension enable/disable configuration based on permission."}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a50d1a3_3aba5f0b","line":43,"in_reply_to":"9a41bdd9_76c903ac","updated":"2015-07-21 08:40:24.000000000","message":"Thanks, Jeffery","commit_id":"87df68673be941005b2497ee729ddd0d4cb7e893"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"f3fb11d65619919f20b9f0b74718d96ca641774c","unresolved":false,"context_lines":[{"line_number":41,"context_line":"There are couple things we need to do:"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"* Remove all the extension white/black list options. This also need tuning the"},{"line_number":44,"context_line":"  tests, because our tests based on those option to test specific plugin."},{"line_number":45,"context_line":"* Remove extension discover entry from policy.json. This is another kind of"},{"line_number":46,"context_line":"  extension enable/disable configuration based on permission."},{"line_number":47,"context_line":"  For example: \"os_compute_api:os-access-ips:discoverable\": \"\""}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a50d1a3_7b9229a1","line":44,"updated":"2015-07-21 09:37:44.000000000","message":"So we will keep all plugin support like access_ips.py adding \u0027access_ip_v4\u0027 in request arg. and makes all plugins loaded all the time which are currently loaded based on white/black list. right?\n\nIf so it will good if we add that kind of impl info also.\n\nWith this we will not need any changes in json request schema and plugins etc.","commit_id":"87df68673be941005b2497ee729ddd0d4cb7e893"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"74ab9c63690c22e17e5be97da2eb4b75fe9a3766","unresolved":false,"context_lines":[{"line_number":41,"context_line":"There are couple things we need to do:"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"* Remove all the extension white/black list options. This also need tuning the"},{"line_number":44,"context_line":"  tests, because our tests based on those option to test specific plugin."},{"line_number":45,"context_line":"* Remove extension discover entry from policy.json. This is another kind of"},{"line_number":46,"context_line":"  extension enable/disable configuration based on permission."},{"line_number":47,"context_line":"  For example: \"os_compute_api:os-access-ips:discoverable\": \"\""}],"source_content_type":"text/x-rst","patch_set":1,"id":"1a4dcd0f_34b6183e","line":44,"in_reply_to":"1a4dcd0f_ca154d40","updated":"2015-08-18 07:29:11.000000000","message":"Gman, sorry I\u0027m little didn\u0027t quite understand you.\n\nWe should make all the tests load all the plugins, that need us merge some api sample test I think.","commit_id":"87df68673be941005b2497ee729ddd0d4cb7e893"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"20ca848fd88207474aba55683909e5fd67281254","unresolved":false,"context_lines":[{"line_number":41,"context_line":"There are couple things we need to do:"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"* Remove all the extension white/black list options. This also need tuning the"},{"line_number":44,"context_line":"  tests, because our tests based on those option to test specific plugin."},{"line_number":45,"context_line":"* Remove extension discover entry from policy.json. This is another kind of"},{"line_number":46,"context_line":"  extension enable/disable configuration based on permission."},{"line_number":47,"context_line":"  For example: \"os_compute_api:os-access-ips:discoverable\": \"\""}],"source_content_type":"text/x-rst","patch_set":1,"id":"1a4dcd0f_ca154d40","line":44,"in_reply_to":"3a50d1a3_7b9229a1","updated":"2015-08-06 11:00:58.000000000","message":"We should deprecate this really, but I think we should skip that in this case.","commit_id":"87df68673be941005b2497ee729ddd0d4cb7e893"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"f3fb11d65619919f20b9f0b74718d96ca641774c","unresolved":false,"context_lines":[{"line_number":49,"context_line":"  There is v2.1 API plugins to v2 API extensions mapping in extension API. As"},{"line_number":50,"context_line":"  we return all the V2 API extensions in the extension API to keep backward-"},{"line_number":51,"context_line":"  compatible with v2 API, we use hard-code v2 API extensions list instead of"},{"line_number":52,"context_line":"  the mapping."},{"line_number":53,"context_line":"* Merge API sample tests which separate API into several parts."},{"line_number":54,"context_line":"  For example, we have separated api sample tests for"},{"line_number":55,"context_line":"  ExtendedStatusSampleJsonTests and ExtendedServerAttributesJsonTest plugins."}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a50d1a3_60a4b8a5","line":52,"updated":"2015-07-21 09:37:44.000000000","message":"As mentioned in \u0027http://docs.openstack.org/developer/nova/stable_api.html#stable-api\u0027,\nShould we deprecate /extensions for v2.1 now or may be in M or later?","commit_id":"87df68673be941005b2497ee729ddd0d4cb7e893"},{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"d0bb9242cd7cea6706605191ee838fcc1284e7d8","unresolved":false,"context_lines":[{"line_number":49,"context_line":"  There is v2.1 API plugins to v2 API extensions mapping in extension API. As"},{"line_number":50,"context_line":"  we return all the V2 API extensions in the extension API to keep backward-"},{"line_number":51,"context_line":"  compatible with v2 API, we use hard-code v2 API extensions list instead of"},{"line_number":52,"context_line":"  the mapping."},{"line_number":53,"context_line":"* Merge API sample tests which separate API into several parts."},{"line_number":54,"context_line":"  For example, we have separated api sample tests for"},{"line_number":55,"context_line":"  ExtendedStatusSampleJsonTests and ExtendedServerAttributesJsonTest plugins."}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a50d1a3_18ae52e6","line":52,"in_reply_to":"3a50d1a3_60a4b8a5","updated":"2015-07-30 01:58:43.000000000","message":"I think, deprecate /extensions in future may be better? This can be done with a bump api version?","commit_id":"87df68673be941005b2497ee729ddd0d4cb7e893"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"f3fb11d65619919f20b9f0b74718d96ca641774c","unresolved":false,"context_lines":[{"line_number":55,"context_line":"  ExtendedStatusSampleJsonTests and ExtendedServerAttributesJsonTest plugins."},{"line_number":56,"context_line":"  Those api sample tests only enable specific plugin, the generated API sample"},{"line_number":57,"context_line":"  files just show part of API. We should merge them into single api sample"},{"line_number":58,"context_line":"  tests."},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"Alternatives"},{"line_number":61,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a50d1a3_7b91c951","line":58,"updated":"2015-07-21 09:37:44.000000000","message":"We have unit tests also doing testing based on extensions loading like - https://github.com/openstack/nova/blob/master/nova/tests/unit/api/openstack/compute/contrib/test_availability_zone.py#L245\n\nWe need to remove/cleanup those kind of unit tests also which tests unavailability of v2.1 extensions.","commit_id":"87df68673be941005b2497ee729ddd0d4cb7e893"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"74ab9c63690c22e17e5be97da2eb4b75fe9a3766","unresolved":false,"context_lines":[{"line_number":55,"context_line":"  ExtendedStatusSampleJsonTests and ExtendedServerAttributesJsonTest plugins."},{"line_number":56,"context_line":"  Those api sample tests only enable specific plugin, the generated API sample"},{"line_number":57,"context_line":"  files just show part of API. We should merge them into single api sample"},{"line_number":58,"context_line":"  tests."},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"Alternatives"},{"line_number":61,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"1a4dcd0f_4dd55cb7","line":58,"in_reply_to":"3a50d1a3_7b91c951","updated":"2015-08-18 07:29:11.000000000","message":"thanks! good catch, I will add those items","commit_id":"87df68673be941005b2497ee729ddd0d4cb7e893"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"20ca848fd88207474aba55683909e5fd67281254","unresolved":false,"context_lines":[{"line_number":70,"context_line":"REST API impact"},{"line_number":71,"context_line":"---------------"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"None"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"Security impact"},{"line_number":76,"context_line":"---------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"1a4dcd0f_6aa2f9fe","line":73,"updated":"2015-08-06 11:00:58.000000000","message":"we should do a bump of the API version, and remove /extensions from v2.1, I assume thats possible?","commit_id":"87df68673be941005b2497ee729ddd0d4cb7e893"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"74ab9c63690c22e17e5be97da2eb4b75fe9a3766","unresolved":false,"context_lines":[{"line_number":70,"context_line":"REST API impact"},{"line_number":71,"context_line":"---------------"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"None"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"Security impact"},{"line_number":76,"context_line":"---------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"1a4dcd0f_ed01c83a","line":73,"in_reply_to":"1a4dcd0f_6aa2f9fe","updated":"2015-08-18 07:29:11.000000000","message":"I remember we said keep /extensions always return True in the summit. We deprecate it in the future.","commit_id":"87df68673be941005b2497ee729ddd0d4cb7e893"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"f3fb11d65619919f20b9f0b74718d96ca641774c","unresolved":false,"context_lines":[{"line_number":120,"context_line":"* Remove the extension black/white list options."},{"line_number":121,"context_line":"* Remove extension discoverable entry."},{"line_number":122,"context_line":"* Tuning the extension API"},{"line_number":123,"context_line":"* Merge api sample tests."},{"line_number":124,"context_line":""},{"line_number":125,"context_line":"Dependencies"},{"line_number":126,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a50d1a3_bb851108","line":123,"updated":"2015-07-21 09:37:44.000000000","message":"Unit tests cleanup too.","commit_id":"87df68673be941005b2497ee729ddd0d4cb7e893"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"74ab9c63690c22e17e5be97da2eb4b75fe9a3766","unresolved":false,"context_lines":[{"line_number":120,"context_line":"* Remove the extension black/white list options."},{"line_number":121,"context_line":"* Remove extension discoverable entry."},{"line_number":122,"context_line":"* Tuning the extension API"},{"line_number":123,"context_line":"* Merge api sample tests."},{"line_number":124,"context_line":""},{"line_number":125,"context_line":"Dependencies"},{"line_number":126,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":1,"id":"1a4dcd0f_edda28e9","line":123,"in_reply_to":"3a50d1a3_bb851108","updated":"2015-08-18 07:29:11.000000000","message":"Done","commit_id":"87df68673be941005b2497ee729ddd0d4cb7e893"}]}