)]}'
{"specs/mitaka/approved/change-return-code-api-20x.rst":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"5db8c32ff5ed5e294ecafc8dbfa9f2f8d9ebe81e","unresolved":false,"context_lines":[{"line_number":24,"context_line":"As a end user, I expect API response to be align to HTTP definition"},{"line_number":25,"context_line":"as much as nova can."},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"Project Priority"},{"line_number":28,"context_line":"-----------------"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"None"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_389777ed","line":27,"updated":"2015-10-08 19:29:45.000000000","message":"Drop the project priority section since that\u0027s been dropped from the template.","commit_id":"16d00cc3a675350d29e3be42997b53f13b6147f5"},{"author":{"_account_id":6062,"name":"jichenjc","email":"jichenjc@cn.ibm.com","username":"jichenjc"},"change_message_id":"78a57120b4d7daf63c1e78a62274917e776f7173","unresolved":false,"context_lines":[{"line_number":24,"context_line":"As a end user, I expect API response to be align to HTTP definition"},{"line_number":25,"context_line":"as much as nova can."},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"Project Priority"},{"line_number":28,"context_line":"-----------------"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"None"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_021f8408","line":27,"in_reply_to":"7a2fa921_389777ed","updated":"2015-10-09 12:31:40.000000000","message":"Done","commit_id":"16d00cc3a675350d29e3be42997b53f13b6147f5"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"5db8c32ff5ed5e294ecafc8dbfa9f2f8d9ebe81e","unresolved":false,"context_lines":[{"line_number":33,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"Check nova/api/openstack/compute folder and update v2.1 API code to"},{"line_number":36,"context_line":"return 201-204 for all occurance."},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"Alternatives"},{"line_number":39,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_d888c3c7","line":36,"updated":"2015-10-08 19:29:45.000000000","message":"I think it would be useful to list the impacted APIs here so we can get an idea for the size of this change.\n\n--\n\nI guess that\u0027s in work items below...","commit_id":"16d00cc3a675350d29e3be42997b53f13b6147f5"},{"author":{"_account_id":6062,"name":"jichenjc","email":"jichenjc@cn.ibm.com","username":"jichenjc"},"change_message_id":"78a57120b4d7daf63c1e78a62274917e776f7173","unresolved":false,"context_lines":[{"line_number":33,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"Check nova/api/openstack/compute folder and update v2.1 API code to"},{"line_number":36,"context_line":"return 201-204 for all occurance."},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"Alternatives"},{"line_number":39,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_e2ea88da","line":36,"in_reply_to":"7a2fa921_d888c3c7","updated":"2015-10-09 12:31:40.000000000","message":"yes, in the work items","commit_id":"16d00cc3a675350d29e3be42997b53f13b6147f5"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"5db8c32ff5ed5e294ecafc8dbfa9f2f8d9ebe81e","unresolved":false,"context_lines":[{"line_number":38,"context_line":"Alternatives"},{"line_number":39,"context_line":"------------"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"No change, use 200 instead of HTTP definition 201-204"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"Data model impact"},{"line_number":44,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_78916ff0","line":41,"updated":"2015-10-08 19:29:45.000000000","message":"Just put None in this section.","commit_id":"16d00cc3a675350d29e3be42997b53f13b6147f5"},{"author":{"_account_id":6062,"name":"jichenjc","email":"jichenjc@cn.ibm.com","username":"jichenjc"},"change_message_id":"78a57120b4d7daf63c1e78a62274917e776f7173","unresolved":false,"context_lines":[{"line_number":38,"context_line":"Alternatives"},{"line_number":39,"context_line":"------------"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"No change, use 200 instead of HTTP definition 201-204"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"Data model impact"},{"line_number":44,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_82c07451","line":41,"in_reply_to":"7a2fa921_78916ff0","updated":"2015-10-09 12:31:40.000000000","message":"Done","commit_id":"16d00cc3a675350d29e3be42997b53f13b6147f5"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"5db8c32ff5ed5e294ecafc8dbfa9f2f8d9ebe81e","unresolved":false,"context_lines":[{"line_number":85,"context_line":"Other end user impact"},{"line_number":86,"context_line":"---------------------"},{"line_number":87,"context_line":""},{"line_number":88,"context_line":"None (python-novaclient don\u0027t need change)"},{"line_number":89,"context_line":""},{"line_number":90,"context_line":"Performance Impact"},{"line_number":91,"context_line":"------------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_f841ff63","line":88,"updated":"2015-10-08 19:29:45.000000000","message":"How about other services using the API, like Tempest?  I think Heat and Ceilometer are using python-novaclient so probably not an issue there.  What about python-openstackclient or shade?","commit_id":"16d00cc3a675350d29e3be42997b53f13b6147f5"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"5db8c32ff5ed5e294ecafc8dbfa9f2f8d9ebe81e","unresolved":false,"context_lines":[{"line_number":111,"context_line":"None"},{"line_number":112,"context_line":""},{"line_number":113,"context_line":"Work Items"},{"line_number":114,"context_line":"----------"},{"line_number":115,"context_line":""},{"line_number":116,"context_line":"1) change os-agents/delete success code from 200 to 204"},{"line_number":117,"context_line":"2) change os-agents/create success code from 200 to 201"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_fb3ed989","line":114,"updated":"2015-10-08 19:29:45.000000000","message":"Did you forget keypairs here?","commit_id":"16d00cc3a675350d29e3be42997b53f13b6147f5"},{"author":{"_account_id":6062,"name":"jichenjc","email":"jichenjc@cn.ibm.com","username":"jichenjc"},"change_message_id":"78a57120b4d7daf63c1e78a62274917e776f7173","unresolved":false,"context_lines":[{"line_number":111,"context_line":"None"},{"line_number":112,"context_line":""},{"line_number":113,"context_line":"Work Items"},{"line_number":114,"context_line":"----------"},{"line_number":115,"context_line":""},{"line_number":116,"context_line":"1) change os-agents/delete success code from 200 to 204"},{"line_number":117,"context_line":"2) change os-agents/create success code from 200 to 201"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_2243a06d","line":114,"in_reply_to":"7a2fa921_fb3ed989","updated":"2015-10-09 12:31:40.000000000","message":"os-keypairs? seems the original code covered already?","commit_id":"16d00cc3a675350d29e3be42997b53f13b6147f5"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"7212946d26ddcf7b5724bf274e647865778e3878","unresolved":false,"context_lines":[{"line_number":116,"context_line":"1) change os-agents/delete success code from 200 to 204"},{"line_number":117,"context_line":"2) change os-agents/create success code from 200 to 201"},{"line_number":118,"context_line":"3) change os-quota-sets/delete success code from 202 to 204"},{"line_number":119,"context_line":"4) change os-rescue/rescue success code from 200 to 202"},{"line_number":120,"context_line":"5) change os-admin-password/changePassword success code from 202 to 204"},{"line_number":121,"context_line":""},{"line_number":122,"context_line":"All items will be done in one patch so only one microversion"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_6f6c58d0","line":119,"updated":"2015-10-08 13:19:30.000000000","message":"Search the todo, evacuate need change also.","commit_id":"16d00cc3a675350d29e3be42997b53f13b6147f5"},{"author":{"_account_id":6062,"name":"jichenjc","email":"jichenjc@cn.ibm.com","username":"jichenjc"},"change_message_id":"78a57120b4d7daf63c1e78a62274917e776f7173","unresolved":false,"context_lines":[{"line_number":116,"context_line":"1) change os-agents/delete success code from 200 to 204"},{"line_number":117,"context_line":"2) change os-agents/create success code from 200 to 201"},{"line_number":118,"context_line":"3) change os-quota-sets/delete success code from 202 to 204"},{"line_number":119,"context_line":"4) change os-rescue/rescue success code from 200 to 202"},{"line_number":120,"context_line":"5) change os-admin-password/changePassword success code from 202 to 204"},{"line_number":121,"context_line":""},{"line_number":122,"context_line":"All items will be done in one patch so only one microversion"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_229e4007","line":119,"in_reply_to":"7a2fa921_1bf635f9","updated":"2015-10-09 12:31:40.000000000","message":"Done","commit_id":"16d00cc3a675350d29e3be42997b53f13b6147f5"},{"author":{"_account_id":6062,"name":"jichenjc","email":"jichenjc@cn.ibm.com","username":"jichenjc"},"change_message_id":"78a57120b4d7daf63c1e78a62274917e776f7173","unresolved":false,"context_lines":[{"line_number":116,"context_line":"1) change os-agents/delete success code from 200 to 204"},{"line_number":117,"context_line":"2) change os-agents/create success code from 200 to 201"},{"line_number":118,"context_line":"3) change os-quota-sets/delete success code from 202 to 204"},{"line_number":119,"context_line":"4) change os-rescue/rescue success code from 200 to 202"},{"line_number":120,"context_line":"5) change os-admin-password/changePassword success code from 202 to 204"},{"line_number":121,"context_line":""},{"line_number":122,"context_line":"All items will be done in one patch so only one microversion"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_42a1fc44","line":119,"in_reply_to":"7a2fa921_6f6c58d0","updated":"2015-10-09 12:31:40.000000000","message":"Done","commit_id":"16d00cc3a675350d29e3be42997b53f13b6147f5"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"5db8c32ff5ed5e294ecafc8dbfa9f2f8d9ebe81e","unresolved":false,"context_lines":[{"line_number":116,"context_line":"1) change os-agents/delete success code from 200 to 204"},{"line_number":117,"context_line":"2) change os-agents/create success code from 200 to 201"},{"line_number":118,"context_line":"3) change os-quota-sets/delete success code from 202 to 204"},{"line_number":119,"context_line":"4) change os-rescue/rescue success code from 200 to 202"},{"line_number":120,"context_line":"5) change os-admin-password/changePassword success code from 202 to 204"},{"line_number":121,"context_line":""},{"line_number":122,"context_line":"All items will be done in one patch so only one microversion"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_1bf635f9","line":119,"in_reply_to":"7a2fa921_6f6c58d0","updated":"2015-10-08 19:29:45.000000000","message":"There is also a TODO in os-volumes-attachments:create:\n\n# TODO(justinsb): How do I return \"accepted\" here?\n\nSince attach is an async process it seems that should be a 202.","commit_id":"16d00cc3a675350d29e3be42997b53f13b6147f5"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"5db8c32ff5ed5e294ecafc8dbfa9f2f8d9ebe81e","unresolved":false,"context_lines":[{"line_number":130,"context_line":"Testing"},{"line_number":131,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"Unit test only."},{"line_number":134,"context_line":""},{"line_number":135,"context_line":"Documentation Impact"},{"line_number":136,"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":"7a2fa921_dbdc3d75","line":133,"updated":"2015-10-08 19:29:45.000000000","message":"What about API samples functional testing?","commit_id":"16d00cc3a675350d29e3be42997b53f13b6147f5"},{"author":{"_account_id":6062,"name":"jichenjc","email":"jichenjc@cn.ibm.com","username":"jichenjc"},"change_message_id":"78a57120b4d7daf63c1e78a62274917e776f7173","unresolved":false,"context_lines":[{"line_number":130,"context_line":"Testing"},{"line_number":131,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"Unit test only."},{"line_number":134,"context_line":""},{"line_number":135,"context_line":"Documentation Impact"},{"line_number":136,"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":"7a2fa921_62a438b9","line":133,"in_reply_to":"7a2fa921_dbdc3d75","updated":"2015-10-09 12:31:40.000000000","message":"yes, it should be updated as well, will add it","commit_id":"16d00cc3a675350d29e3be42997b53f13b6147f5"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"5db8c32ff5ed5e294ecafc8dbfa9f2f8d9ebe81e","unresolved":false,"context_lines":[{"line_number":135,"context_line":"Documentation Impact"},{"line_number":136,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":137,"context_line":""},{"line_number":138,"context_line":"None"},{"line_number":139,"context_line":""},{"line_number":140,"context_line":"References"},{"line_number":141,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_fb5579c1","line":138,"updated":"2015-10-08 19:29:45.000000000","message":"We wouldn\u0027t need to update the API docs for this?  We should scrub through:\n\nhttp://developer.openstack.org/api-ref-compute-v2.1.html\n\nBut it actually looks like the docs have some of this correct (what you\u0027d be changing to) rather than what is returned today.","commit_id":"16d00cc3a675350d29e3be42997b53f13b6147f5"},{"author":{"_account_id":6062,"name":"jichenjc","email":"jichenjc@cn.ibm.com","username":"jichenjc"},"change_message_id":"78a57120b4d7daf63c1e78a62274917e776f7173","unresolved":false,"context_lines":[{"line_number":135,"context_line":"Documentation Impact"},{"line_number":136,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":137,"context_line":""},{"line_number":138,"context_line":"None"},{"line_number":139,"context_line":""},{"line_number":140,"context_line":"References"},{"line_number":141,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_02946440","line":138,"in_reply_to":"7a2fa921_fb5579c1","updated":"2015-10-09 12:31:40.000000000","message":"right \nsome doc and code has inconsistency here\nwill mark DocImpact","commit_id":"16d00cc3a675350d29e3be42997b53f13b6147f5"}]}