)]}'
{"specs/mitaka/approved/return_HTTPForbidden_if_over_quota.rst":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"d9e87aa063080f85468f870018217c17d818fb25","unresolved":false,"context_lines":[{"line_number":8,"context_line":"return_httpforbidden_if_over_quota"},{"line_number":9,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"https://blueprints.launchpad.net/nova/+spec/return_HTTPForbidden_if_over_quota"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Return HTTPForbidden if over quota"},{"line_number":14,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_01e562da","line":11,"updated":"2015-10-08 20:30:01.000000000","message":"This doesn\u0027t exist.","commit_id":"21164af18eeaa7cfa697b3fc782561cf7fbc2517"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"d9e87aa063080f85468f870018217c17d818fb25","unresolved":false,"context_lines":[{"line_number":21,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Nova API returns HTTPBadRequest instead of HTTPForbidden when over quota"},{"line_number":24,"context_line":"in tenant_networks.py"},{"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":1,"id":"7a2fa921_61f53655","line":24,"updated":"2015-10-08 20:30:01.000000000","message":"Are you only changing this API or are there others?  If it\u0027s only this one, I\u0027d probably have that in the commit message.\n\nShould probably also refer to this in the References section:\n\nhttps://github.com/openstack/nova/blob/12.0.0.0rc2/nova/api/openstack/compute/tenant_networks.py#L169-L171","commit_id":"21164af18eeaa7cfa697b3fc782561cf7fbc2517"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"d9e87aa063080f85468f870018217c17d818fb25","unresolved":false,"context_lines":[{"line_number":29,"context_line":"End user use this API check return error code when over quota."},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"Project Priority"},{"line_number":32,"context_line":"-----------------"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"N/A"},{"line_number":35,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_e18866c7","line":32,"updated":"2015-10-08 20:30:01.000000000","message":"Remove the priority section since it\u0027s been dropped from the template.","commit_id":"21164af18eeaa7cfa697b3fc782561cf7fbc2517"},{"author":{"_account_id":6062,"name":"jichenjc","email":"jichenjc@cn.ibm.com","username":"jichenjc"},"change_message_id":"6b5bcbbdb5f391d7989c324909c8ccc294f28bea","unresolved":false,"context_lines":[{"line_number":29,"context_line":"End user use this API check return error code when over quota."},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"Project Priority"},{"line_number":32,"context_line":"-----------------"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"N/A"},{"line_number":35,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_bce11b2c","line":32,"in_reply_to":"7a2fa921_e18866c7","updated":"2015-10-09 11:32:43.000000000","message":"Done","commit_id":"21164af18eeaa7cfa697b3fc782561cf7fbc2517"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"d9e87aa063080f85468f870018217c17d818fb25","unresolved":false,"context_lines":[{"line_number":36,"context_line":"Proposed change"},{"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":"Bump a microverison to change the return value from"},{"line_number":40,"context_line":"HTTPBadRequest to HTTPForbidden."},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"Alternatives"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_0c0cb939","line":39,"updated":"2015-10-08 20:30:01.000000000","message":"I don\u0027t think we need a microversion for this given:\n\nhttp://docs.openstack.org/developer/nova/api_microversion_dev.html#f2\n\n403 is implied and os-tenant-networks create already has 403 in it\u0027s list of expected errors:\n\nhttps://github.com/openstack/nova/blob/12.0.0.0rc2/nova/api/openstack/compute/tenant_networks.py#L141\n\nSo I think you could just change this to raise the proper exception and be done with it:\n\nhttps://github.com/openstack/nova/blob/12.0.0.0rc2/nova/api/openstack/compute/tenant_networks.py#L171","commit_id":"21164af18eeaa7cfa697b3fc782561cf7fbc2517"},{"author":{"_account_id":6062,"name":"jichenjc","email":"jichenjc@cn.ibm.com","username":"jichenjc"},"change_message_id":"6b5bcbbdb5f391d7989c324909c8ccc294f28bea","unresolved":false,"context_lines":[{"line_number":36,"context_line":"Proposed change"},{"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":"Bump a microverison to change the return value from"},{"line_number":40,"context_line":"HTTPBadRequest to HTTPForbidden."},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"Alternatives"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_1cafef7d","line":39,"in_reply_to":"7a2fa921_0c0cb939","updated":"2015-10-09 11:32:43.000000000","message":"ok, I think we should abandon this unless the original code patch is not accepted","commit_id":"21164af18eeaa7cfa697b3fc782561cf7fbc2517"}]}