)]}'
{"specs/liberty/approved/api_cleanup_remove_v3.rst":[{"author":{"_account_id":9420,"name":"Matthew Gilliard","email":"matthew.gilliard@hp.com","username":"gilliard"},"change_message_id":"ef17fe23427cd7ba4a089238634a6bdc85fbe656","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"The Nova V2.1 REST API released at Kilo. But the V2.1 API code still under"},{"line_number":14,"context_line":"old directory \u0027nova/api/openstack/compute/plugins/v3\u0027 which with old name"},{"line_number":15,"context_line":"\u0027v3\u0027. V3 API isn\u0027t existed anymore, it is replaced by V2.1 API."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"So we should cleanup any V3 related stuff and restructure the nova code tree"},{"line_number":18,"context_line":"to avoid confuse for the developer."}],"source_content_type":"text/x-rst","patch_set":1,"id":"3afb71cf_185b2a89","line":15,"updated":"2015-06-08 08:47:38.000000000","message":"english nit: s/isn\u0027t existed/doesn\u0027t exist/","commit_id":"5225e76b205435e97fc713cc5b5e81e605e3d919"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"a90ed33597d13258a33bc8693eccf78223da1418","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"The Nova V2.1 REST API released at Kilo. But the V2.1 API code still under"},{"line_number":14,"context_line":"old directory \u0027nova/api/openstack/compute/plugins/v3\u0027 which with old name"},{"line_number":15,"context_line":"\u0027v3\u0027. V3 API isn\u0027t existed anymore, it is replaced by V2.1 API."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"So we should cleanup any V3 related stuff and restructure the nova code tree"},{"line_number":18,"context_line":"to avoid confuse for the developer."}],"source_content_type":"text/x-rst","patch_set":1,"id":"3afb71cf_aa65a01e","line":15,"in_reply_to":"3afb71cf_185b2a89","updated":"2015-06-09 12:22:58.000000000","message":"Done","commit_id":"5225e76b205435e97fc713cc5b5e81e605e3d919"},{"author":{"_account_id":9420,"name":"Matthew Gilliard","email":"matthew.gilliard@hp.com","username":"gilliard"},"change_message_id":"ef17fe23427cd7ba4a089238634a6bdc85fbe656","unresolved":false,"context_lines":[{"line_number":15,"context_line":"\u0027v3\u0027. V3 API isn\u0027t existed anymore, it is replaced by V2.1 API."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"So we should cleanup any V3 related stuff and restructure the nova code tree"},{"line_number":18,"context_line":"to avoid confuse for the developer."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Problem description"},{"line_number":21,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3afb71cf_784a1eae","line":18,"updated":"2015-06-08 08:47:38.000000000","message":"\"confusion\"","commit_id":"5225e76b205435e97fc713cc5b5e81e605e3d919"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"a90ed33597d13258a33bc8693eccf78223da1418","unresolved":false,"context_lines":[{"line_number":15,"context_line":"\u0027v3\u0027. V3 API isn\u0027t existed anymore, it is replaced by V2.1 API."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"So we should cleanup any V3 related stuff and restructure the nova code tree"},{"line_number":18,"context_line":"to avoid confuse for the developer."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Problem description"},{"line_number":21,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3afb71cf_eaa83822","line":18,"in_reply_to":"3afb71cf_784a1eae","updated":"2015-06-09 12:22:58.000000000","message":"Done","commit_id":"5225e76b205435e97fc713cc5b5e81e605e3d919"},{"author":{"_account_id":12175,"name":"Eli Qiao","email":"qiaoliyong@gmail.com","username":"Eli"},"change_message_id":"aeeb52ba341e942d20389e2d1f10257ad5e09a1d","unresolved":false,"context_lines":[{"line_number":37,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"* Move the V2 API code under \u0027nova/api/openstack/compute/v2\u0027. The \u0027V2\u0027 API"},{"line_number":40,"context_line":"  will be deprecated in the future."},{"line_number":41,"context_line":"* Move the V2.1 API code under \u0027nova/api/openstack/compute\u0027. The evolution"},{"line_number":42,"context_line":"  of V2.1 API is done by Microversion. So we needn\u0027t mention API version in"},{"line_number":43,"context_line":"  the code tree anymore."}],"source_content_type":"text/x-rst","patch_set":1,"id":"3afb71cf_d8c5928e","line":40,"updated":"2015-06-08 08:29:11.000000000","message":"v2 api contains some controllers in  nova/api/openstack/compute\nand also others under nova/api/openstack/compute/contrib/\n\nI think it\u0027s better to mention that move all nova/api/openstack/compute and nova/api/openstack/compute/contrib/  to\n\n\u0027nova/api/openstack/compute/v2\u0027\n\nEli.","commit_id":"5225e76b205435e97fc713cc5b5e81e605e3d919"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"a90ed33597d13258a33bc8693eccf78223da1418","unresolved":false,"context_lines":[{"line_number":37,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"* Move the V2 API code under \u0027nova/api/openstack/compute/v2\u0027. The \u0027V2\u0027 API"},{"line_number":40,"context_line":"  will be deprecated in the future."},{"line_number":41,"context_line":"* Move the V2.1 API code under \u0027nova/api/openstack/compute\u0027. The evolution"},{"line_number":42,"context_line":"  of V2.1 API is done by Microversion. So we needn\u0027t mention API version in"},{"line_number":43,"context_line":"  the code tree anymore."}],"source_content_type":"text/x-rst","patch_set":1,"id":"3afb71cf_ea6a7880","line":40,"in_reply_to":"3afb71cf_d8c5928e","updated":"2015-06-09 12:22:58.000000000","message":"yea, got it, will update ,thanks","commit_id":"5225e76b205435e97fc713cc5b5e81e605e3d919"},{"author":{"_account_id":9420,"name":"Matthew Gilliard","email":"matthew.gilliard@hp.com","username":"gilliard"},"change_message_id":"ef17fe23427cd7ba4a089238634a6bdc85fbe656","unresolved":false,"context_lines":[{"line_number":40,"context_line":"  will be deprecated in the future."},{"line_number":41,"context_line":"* Move the V2.1 API code under \u0027nova/api/openstack/compute\u0027. The evolution"},{"line_number":42,"context_line":"  of V2.1 API is done by Microversion. So we needn\u0027t mention API version in"},{"line_number":43,"context_line":"  the code tree anymore."},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"Alternatives"},{"line_number":46,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3afb71cf_b869b627","line":43,"updated":"2015-06-08 08:47:38.000000000","message":"I think it\u0027s better to keep \"v2.1\" in the path. There has been a lot of confusing changes to the API code and this would make it clear what is v2 (which is really not going away soon) and what is v2.1.","commit_id":"5225e76b205435e97fc713cc5b5e81e605e3d919"},{"author":{"_account_id":1063,"name":"Ed Leafe","email":"ed@leafe.com","username":"ed-leafe"},"change_message_id":"c38158fa6af8972f49a4a98b13cf40eceb213af5","unresolved":false,"context_lines":[{"line_number":40,"context_line":"  will be deprecated in the future."},{"line_number":41,"context_line":"* Move the V2.1 API code under \u0027nova/api/openstack/compute\u0027. The evolution"},{"line_number":42,"context_line":"  of V2.1 API is done by Microversion. So we needn\u0027t mention API version in"},{"line_number":43,"context_line":"  the code tree anymore."},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"Alternatives"},{"line_number":46,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3afb71cf_15c4cc8e","line":43,"in_reply_to":"3afb71cf_09195f22","updated":"2015-06-09 14:17:29.000000000","message":"I think removing the version from the file paths for the current work is the cleanest approach. We can then store the deprecated code in a separate v2 directory, which will make it obvious that it is not part of the standard compute. It will also make it easier to remove once the deprecation cycle is complete.","commit_id":"5225e76b205435e97fc713cc5b5e81e605e3d919"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"459f014901932276aa26b4c5824b03ce563b4c0b","unresolved":false,"context_lines":[{"line_number":40,"context_line":"  will be deprecated in the future."},{"line_number":41,"context_line":"* Move the V2.1 API code under \u0027nova/api/openstack/compute\u0027. The evolution"},{"line_number":42,"context_line":"  of V2.1 API is done by Microversion. So we needn\u0027t mention API version in"},{"line_number":43,"context_line":"  the code tree anymore."},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"Alternatives"},{"line_number":46,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3afb71cf_6488dbc5","line":43,"in_reply_to":"3afb71cf_15c4cc8e","updated":"2015-06-10 00:18:18.000000000","message":"Ed, thanks, that explain is better.","commit_id":"5225e76b205435e97fc713cc5b5e81e605e3d919"},{"author":{"_account_id":9420,"name":"Matthew Gilliard","email":"matthew.gilliard@hp.com","username":"gilliard"},"change_message_id":"242f0b9aadd8f24a85e3199df4eba975ab581c25","unresolved":false,"context_lines":[{"line_number":40,"context_line":"  will be deprecated in the future."},{"line_number":41,"context_line":"* Move the V2.1 API code under \u0027nova/api/openstack/compute\u0027. The evolution"},{"line_number":42,"context_line":"  of V2.1 API is done by Microversion. So we needn\u0027t mention API version in"},{"line_number":43,"context_line":"  the code tree anymore."},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"Alternatives"},{"line_number":46,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3afb71cf_09195f22","line":43,"in_reply_to":"3afb71cf_6ae688b1","updated":"2015-06-09 13:27:53.000000000","message":"It\u0027s confusing for sure.  And, yes, we could drop the \"vXX\" from the URL.\n\nBut, I really think we should keep v2.1 in the file paths so it\u0027s clear what is \"v2\" or \"v2.1\".  The v2 code is not going to be removed soon, is it? And the microversions, I remember from the summit, are all part of the v2.1 API, so it\u0027s not like \"2.1\" is inaccurate.","commit_id":"5225e76b205435e97fc713cc5b5e81e605e3d919"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"fc8cf5fc75076d76a373a14fa105e43b23f0e6fd","unresolved":false,"context_lines":[{"line_number":40,"context_line":"  will be deprecated in the future."},{"line_number":41,"context_line":"* Move the V2.1 API code under \u0027nova/api/openstack/compute\u0027. The evolution"},{"line_number":42,"context_line":"  of V2.1 API is done by Microversion. So we needn\u0027t mention API version in"},{"line_number":43,"context_line":"  the code tree anymore."},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"Alternatives"},{"line_number":46,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3afb71cf_c7448e75","line":43,"in_reply_to":"3afb71cf_b869b627","updated":"2015-06-08 10:33:33.000000000","message":"The directories\n\n nova/api/openstack/compute/v2\n nova/api/openstack/compute/v2.1\n\nseem readable because that is the similar to api-paste.ini :)","commit_id":"5225e76b205435e97fc713cc5b5e81e605e3d919"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"a90ed33597d13258a33bc8693eccf78223da1418","unresolved":false,"context_lines":[{"line_number":40,"context_line":"  will be deprecated in the future."},{"line_number":41,"context_line":"* Move the V2.1 API code under \u0027nova/api/openstack/compute\u0027. The evolution"},{"line_number":42,"context_line":"  of V2.1 API is done by Microversion. So we needn\u0027t mention API version in"},{"line_number":43,"context_line":"  the code tree anymore."},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"Alternatives"},{"line_number":46,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3afb71cf_6ae688b1","line":43,"in_reply_to":"3afb71cf_c7448e75","updated":"2015-06-09 12:22:58.000000000","message":"Emm....I remember we said v2.1 means the new nova rest api with microversion before. And this is only one API in nova.\n\nAfter microversion bumped, we have micoversion 2.2, 2.3. If we still call it as v2.1, it is another kind of confuse.\n\nDo we still plan to remove version from url in the future?","commit_id":"5225e76b205435e97fc713cc5b5e81e605e3d919"},{"author":{"_account_id":1063,"name":"Ed Leafe","email":"ed@leafe.com","username":"ed-leafe"},"change_message_id":"bcd4dd404ee0b69562795b1b87c0aae497d41020","unresolved":false,"context_lines":[{"line_number":97,"context_line":"-----------"},{"line_number":98,"context_line":""},{"line_number":99,"context_line":"Primary assignee:"},{"line_number":100,"context_line":"  Volunteer?"},{"line_number":101,"context_line":""},{"line_number":102,"context_line":"Work Items"},{"line_number":103,"context_line":"----------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3afb71cf_f7e24d2e","line":100,"updated":"2015-06-09 16:55:14.000000000","message":"I\u0027ll volunteer to do this.","commit_id":"5225e76b205435e97fc713cc5b5e81e605e3d919"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"4e182828a5d38dcc212531c8d256c2338dfdb453","unresolved":false,"context_lines":[{"line_number":97,"context_line":"-----------"},{"line_number":98,"context_line":""},{"line_number":99,"context_line":"Primary assignee:"},{"line_number":100,"context_line":"  Volunteer?"},{"line_number":101,"context_line":""},{"line_number":102,"context_line":"Work Items"},{"line_number":103,"context_line":"----------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3afb71cf_8ae2c12d","line":100,"in_reply_to":"3afb71cf_f7e24d2e","updated":"2015-06-10 00:29:52.000000000","message":"Emm.... huge number of cleanup. I\u0027m happy to help. We can separated tasks, then anyone of us won\u0027t feel super boring.","commit_id":"5225e76b205435e97fc713cc5b5e81e605e3d919"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"459f014901932276aa26b4c5824b03ce563b4c0b","unresolved":false,"context_lines":[{"line_number":97,"context_line":"-----------"},{"line_number":98,"context_line":""},{"line_number":99,"context_line":"Primary assignee:"},{"line_number":100,"context_line":"  Volunteer?"},{"line_number":101,"context_line":""},{"line_number":102,"context_line":"Work Items"},{"line_number":103,"context_line":"----------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3afb71cf_64afbb2a","line":100,"in_reply_to":"3afb71cf_f7e24d2e","updated":"2015-06-10 00:18:18.000000000","message":"thanks","commit_id":"5225e76b205435e97fc713cc5b5e81e605e3d919"},{"author":{"_account_id":9420,"name":"Matthew Gilliard","email":"matthew.gilliard@hp.com","username":"gilliard"},"change_message_id":"ef17fe23427cd7ba4a089238634a6bdc85fbe656","unresolved":false,"context_lines":[{"line_number":114,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":115,"context_line":""},{"line_number":116,"context_line":"This needn\u0027t any new test. Just need ensure all the tests can pass after the"},{"line_number":117,"context_line":"cleanup."},{"line_number":118,"context_line":""},{"line_number":119,"context_line":"Documentation Impact"},{"line_number":120,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3afb71cf_f8ebae76","line":117,"updated":"2015-06-08 08:47:38.000000000","message":"We can make some test that \"v3\" does not exist in the code anywhere? Or maybe that is not necessary as a unit test, but I think it should be our goal.","commit_id":"5225e76b205435e97fc713cc5b5e81e605e3d919"},{"author":{"_account_id":1063,"name":"Ed Leafe","email":"ed@leafe.com","username":"ed-leafe"},"change_message_id":"c38158fa6af8972f49a4a98b13cf40eceb213af5","unresolved":false,"context_lines":[{"line_number":114,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":115,"context_line":""},{"line_number":116,"context_line":"This needn\u0027t any new test. Just need ensure all the tests can pass after the"},{"line_number":117,"context_line":"cleanup."},{"line_number":118,"context_line":""},{"line_number":119,"context_line":"Documentation Impact"},{"line_number":120,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3afb71cf_95067c1b","line":117,"in_reply_to":"3afb71cf_2a8c10d9","updated":"2015-06-09 14:17:29.000000000","message":"There are a lot of existing tests that specify \u0027v3\u0027. Run a grep on \u0027from nova.api.openstack.compute.plugins.v3 import \u0027 and you\u0027ll find quite a few. These will all have to be updated.","commit_id":"5225e76b205435e97fc713cc5b5e81e605e3d919"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"459f014901932276aa26b4c5824b03ce563b4c0b","unresolved":false,"context_lines":[{"line_number":114,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":115,"context_line":""},{"line_number":116,"context_line":"This needn\u0027t any new test. Just need ensure all the tests can pass after the"},{"line_number":117,"context_line":"cleanup."},{"line_number":118,"context_line":""},{"line_number":119,"context_line":"Documentation Impact"},{"line_number":120,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3afb71cf_447f1fab","line":117,"in_reply_to":"3afb71cf_95067c1b","updated":"2015-06-10 00:18:18.000000000","message":"yes","commit_id":"5225e76b205435e97fc713cc5b5e81e605e3d919"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"a90ed33597d13258a33bc8693eccf78223da1418","unresolved":false,"context_lines":[{"line_number":114,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":115,"context_line":""},{"line_number":116,"context_line":"This needn\u0027t any new test. Just need ensure all the tests can pass after the"},{"line_number":117,"context_line":"cleanup."},{"line_number":118,"context_line":""},{"line_number":119,"context_line":"Documentation Impact"},{"line_number":120,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3afb71cf_2a8c10d9","line":117,"in_reply_to":"3afb71cf_f8ebae76","updated":"2015-06-09 12:22:58.000000000","message":"I think it is fine. Just some cleanup works, with v3 in the code won\u0027t hurt anything. If we missing something, just send another cleanup patch for it.","commit_id":"5225e76b205435e97fc713cc5b5e81e605e3d919"}],"specs/liberty/approved/nova-api-remove-v3.rst":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"4bff4e6383c4d0f8da6454e32ad356058281cd4d","unresolved":false,"context_lines":[{"line_number":47,"context_line":"  supported by Nova. The JSON-Schema used by v2.1 API moves into"},{"line_number":48,"context_line":"  \u0027nova/api/openstack/compute/schemas\u0027."},{"line_number":49,"context_line":"* Remove any words about \u0027v3\u0027 from the codes and configuration files. Like"},{"line_number":50,"context_line":"  APIRouterV3, and v3 endpoint entry in api-paste.ini, etc."},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"Alternatives"},{"line_number":53,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3afb71cf_0d382b3c","line":50,"updated":"2015-06-10 03:09:07.000000000","message":"Patches are there to removing v3 entry from api-paste.ini. Currently those are in Abandon state because those are dependent on test merge work. After tests merge we can restore those patches.\n\nDetails - https://etherpad.openstack.org/p/merge_sample_tests","commit_id":"a802faa02ecabf896a8935df8400eb5ae8e3a945"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"c3d53cc7f0a7a0194ef3a6409e8995f3af54eb75","unresolved":false,"context_lines":[{"line_number":47,"context_line":"  supported by Nova. The JSON-Schema used by v2.1 API moves into"},{"line_number":48,"context_line":"  \u0027nova/api/openstack/compute/schemas\u0027."},{"line_number":49,"context_line":"* Remove any words about \u0027v3\u0027 from the codes and configuration files. Like"},{"line_number":50,"context_line":"  APIRouterV3, and v3 endpoint entry in api-paste.ini, etc."},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"Alternatives"},{"line_number":53,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3afb71cf_e7df2338","line":50,"in_reply_to":"3afb71cf_0d382b3c","updated":"2015-06-10 07:05:52.000000000","message":"Gman, thanks! I forgot that. So let\u0027s leave change api-paste.ini into your work stream.","commit_id":"a802faa02ecabf896a8935df8400eb5ae8e3a945"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"4bff4e6383c4d0f8da6454e32ad356058281cd4d","unresolved":false,"context_lines":[{"line_number":113,"context_line":"* Move all V2 code under \u0027v2\u0027 directory."},{"line_number":114,"context_line":"* Move all V2.1 code to the toplevel directory."},{"line_number":115,"context_line":"* Move V2.1\u0027s json-schema out of v3 directory."},{"line_number":116,"context_line":"* Remove v3 endpoint from api-paste.ini."},{"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":2,"id":"3afb71cf_ed414faa","line":116,"updated":"2015-06-10 03:09:07.000000000","message":"ditto","commit_id":"a802faa02ecabf896a8935df8400eb5ae8e3a945"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"c3d53cc7f0a7a0194ef3a6409e8995f3af54eb75","unresolved":false,"context_lines":[{"line_number":113,"context_line":"* Move all V2 code under \u0027v2\u0027 directory."},{"line_number":114,"context_line":"* Move all V2.1 code to the toplevel directory."},{"line_number":115,"context_line":"* Move V2.1\u0027s json-schema out of v3 directory."},{"line_number":116,"context_line":"* Remove v3 endpoint from api-paste.ini."},{"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":2,"id":"3afb71cf_c7e9c7e7","line":116,"in_reply_to":"3afb71cf_ed414faa","updated":"2015-06-10 07:05:52.000000000","message":"Done","commit_id":"a802faa02ecabf896a8935df8400eb5ae8e3a945"},{"author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"change_message_id":"47450681cefa315bfba6d2834a0c9374a4f9f9ab","unresolved":false,"context_lines":[{"line_number":27,"context_line":"Use Cases"},{"line_number":28,"context_line":"----------"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"None. This is a code tree cleanup for Nova."},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"Project Priority"},{"line_number":33,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":4,"id":"1af86dd1_1e68eff4","line":30,"updated":"2015-06-12 10:06:58.000000000","message":"Nit: I would argue the use case, is to make developers less confused about what this whole v3 thing is.","commit_id":"57bde4859fb257345cf417064f788c378f731854"},{"author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"change_message_id":"47450681cefa315bfba6d2834a0c9374a4f9f9ab","unresolved":false,"context_lines":[{"line_number":40,"context_line":"* Move the V2 API code which is currently under \u0027nova/api/openstack/compute\u0027"},{"line_number":41,"context_line":"  and \u0027nova/api/openstack/compute/contrib\u0027 into"},{"line_number":42,"context_line":"  \u0027nova/api/openstack/compute/v2\u0027. The \u0027V2\u0027 API will be deprecated in the"},{"line_number":43,"context_line":"  future."},{"line_number":44,"context_line":"* The Nova V2.1 REST API refers to the new Nova REST API with Microversion."},{"line_number":45,"context_line":"  The evolution of v2.1 will be done by Microversion in the future. So the"},{"line_number":46,"context_line":"  proposal is that the API version won\u0027t be included in any code path. The V2.1"}],"source_content_type":"text/x-rst","patch_set":4,"id":"1af86dd1_1e4fcf93","line":43,"updated":"2015-06-12 10:06:58.000000000","message":"++","commit_id":"57bde4859fb257345cf417064f788c378f731854"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"8bb00885f69726711485fed96fb6f3516bb17112","unresolved":false,"context_lines":[{"line_number":47,"context_line":"  API which is now under \u0027nova/api/openstack/compute/plugins/v3\u0027 will be moved"},{"line_number":48,"context_line":"  into \u0027nova/api/openstack/compute\u0027. This means that the V2.1 API will be the"},{"line_number":49,"context_line":"  only compute API supported by Nova. The JSON-Schema used by v2.1 API moves"},{"line_number":50,"context_line":"  into \u0027nova/api/openstack/compute/schemas\u0027."},{"line_number":51,"context_line":"* Remove any reference to \u0027v3\u0027 from the code, tests, and configuration files."},{"line_number":52,"context_line":"  Examples: APIRouterV3, and v3 endpoint entry in api-paste.ini, etc."},{"line_number":53,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"1af86dd1_bed1dbfd","line":50,"updated":"2015-06-12 09:50:49.000000000","message":"While I totally understand the rationale, I think it can be confusing for new people wanting to contribute (or not following the API updates) and can also be error-prone (eg. old changes being merged which would reference openstack/compute as V2)\n\nCould we maybe consider a 2-step approach, where v3 directory would be renamed `current` and then for M, being just openstac/compute ?","commit_id":"57bde4859fb257345cf417064f788c378f731854"},{"author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"change_message_id":"47450681cefa315bfba6d2834a0c9374a4f9f9ab","unresolved":false,"context_lines":[{"line_number":47,"context_line":"  API which is now under \u0027nova/api/openstack/compute/plugins/v3\u0027 will be moved"},{"line_number":48,"context_line":"  into \u0027nova/api/openstack/compute\u0027. This means that the V2.1 API will be the"},{"line_number":49,"context_line":"  only compute API supported by Nova. The JSON-Schema used by v2.1 API moves"},{"line_number":50,"context_line":"  into \u0027nova/api/openstack/compute/schemas\u0027."},{"line_number":51,"context_line":"* Remove any reference to \u0027v3\u0027 from the code, tests, and configuration files."},{"line_number":52,"context_line":"  Examples: APIRouterV3, and v3 endpoint entry in api-paste.ini, etc."},{"line_number":53,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"1af86dd1_3e184b84","line":50,"in_reply_to":"1af86dd1_bed1dbfd","updated":"2015-06-12 10:06:58.000000000","message":"I agree there is risk of confusion here, lets make sure to make this as clear as possible in the implementation.  (Maybe via a README file in /nova/api/openstack/compute?","commit_id":"57bde4859fb257345cf417064f788c378f731854"},{"author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"change_message_id":"47450681cefa315bfba6d2834a0c9374a4f9f9ab","unresolved":false,"context_lines":[{"line_number":49,"context_line":"  only compute API supported by Nova. The JSON-Schema used by v2.1 API moves"},{"line_number":50,"context_line":"  into \u0027nova/api/openstack/compute/schemas\u0027."},{"line_number":51,"context_line":"* Remove any reference to \u0027v3\u0027 from the code, tests, and configuration files."},{"line_number":52,"context_line":"  Examples: APIRouterV3, and v3 endpoint entry in api-paste.ini, etc."},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"Alternatives"},{"line_number":55,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":4,"id":"1af86dd1_1e1d8f73","line":52,"updated":"2015-06-12 10:06:58.000000000","message":"Getting the api-paste changes working will require a cycle or two, since we don\u0027t want to require changing config (api-paste.ini) for an upgrade to work. So this will require leaving something in the api-paste.ini file for a cycle.  Along with a big note in the release notes.","commit_id":"57bde4859fb257345cf417064f788c378f731854"},{"author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"change_message_id":"47450681cefa315bfba6d2834a0c9374a4f9f9ab","unresolved":false,"context_lines":[{"line_number":57,"context_line":"Keep the V2.1 API under current directory until V2 support is removed. But"},{"line_number":58,"context_line":"since all the API improvements in the future will be for the V2.1 API, a lot of"},{"line_number":59,"context_line":"developers would be confused on how to code to the current API. We should get"},{"line_number":60,"context_line":"rid of it now, and avoid confusing developers."},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"Data model impact"},{"line_number":63,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":4,"id":"1af86dd1_09043b13","line":60,"updated":"2015-06-12 10:06:58.000000000","message":"Agreed, this isn\u0027t a good alternative, realistically we wont be removing v2 for a while anyway.","commit_id":"57bde4859fb257345cf417064f788c378f731854"}]}