)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":18573,"name":"Petri Kemppainen","email":"petri.kemppainen@nokia.com","username":"petrkemp"},"change_message_id":"8a62c0894e0f116140ac92fed2a348baaad02116","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"If a tenant quota is not found, the delete operation should fail"},{"line_number":10,"context_line":"with a 404 error. It currently returns 204 even if the delete"},{"line_number":11,"context_line":"operation actually does not delete anything which is inconsistenct"},{"line_number":12,"context_line":"with other delete operations in the Neutron API."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I1cd91b5e06bd17f9aac97bba71228f2e5c48879b"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5a5ae5dd_3554ca1f","line":11,"range":{"start_line":11,"start_character":53,"end_line":11,"end_character":66},"updated":"2016-02-04 10:45:19.000000000","message":"inconsistent","commit_id":"90b903a63cc478dcab9543fb086827b5160a7de1"},{"author":{"_account_id":14605,"name":"vikram.choudhary","email":"vikschw@gmail.com","username":"vikram.choudhary"},"change_message_id":"b1776000ed69a2cfd93c6bf6ef1c9512ad16f789","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"If a tenant quota is not found, the delete operation should fail"},{"line_number":10,"context_line":"with a 404 error. It currently returns 204 even if the delete"},{"line_number":11,"context_line":"operation actually does not delete anything which is inconsistenct"},{"line_number":12,"context_line":"with other delete operations in the Neutron API."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I1cd91b5e06bd17f9aac97bba71228f2e5c48879b"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"bae84128_8c063a1b","line":11,"range":{"start_line":11,"start_character":53,"end_line":11,"end_character":66},"updated":"2016-02-23 08:21:55.000000000","message":"inconsistent","commit_id":"2739c737ed847e65ea0bdbd0797f9dfe4ba5c930"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"c550a4c2f4eccbed4444e8b6450e8535346e9c74","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"If a tenant quota is not found, the delete operation should fail"},{"line_number":10,"context_line":"with a 404 error. It currently returns 204 even if the delete"},{"line_number":11,"context_line":"operation actually does not delete anything which is inconsistenct"},{"line_number":12,"context_line":"with other delete operations in the Neutron API."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I1cd91b5e06bd17f9aac97bba71228f2e5c48879b"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"5aef4532_0bba9be1","line":11,"range":{"start_line":11,"start_character":53,"end_line":11,"end_character":66},"updated":"2016-03-07 12:47:48.000000000","message":"inconsistent?  inconsistency?","commit_id":"fca5c8dc35634d07cbbad62332685e157ab55768"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"fa9ab3742442756bd7b4259d1b35afa4d6c3bfbe","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"If a tenant quota is not found, the delete operation should fail"},{"line_number":10,"context_line":"with a 404 error. It currently returns 204 even if the delete"},{"line_number":11,"context_line":"operation actually does not delete anything which is inconsistenct"},{"line_number":12,"context_line":"with other delete operations in the Neutron API."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I1cd91b5e06bd17f9aac97bba71228f2e5c48879b"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"5aef4532_ee865d64","line":11,"range":{"start_line":11,"start_character":53,"end_line":11,"end_character":66},"in_reply_to":"5aef4532_0bba9be1","updated":"2016-03-07 12:58:38.000000000","message":"Damn it....","commit_id":"fca5c8dc35634d07cbbad62332685e157ab55768"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"d45e0cd45e685374759fcfe28afa5df846581b26","unresolved":false,"context_lines":[{"line_number":10,"context_line":"with a 404 error. It currently returns 204 even if the delete"},{"line_number":11,"context_line":"operation actually does not delete anything which is inconsistent"},{"line_number":12,"context_line":"with other delete operations in the Neutron API."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I1cd91b5e06bd17f9aac97bba71228f2e5c48879b"},{"line_number":15,"context_line":"Closes-Bug: 1307506"},{"line_number":16,"context_line":"Co-Authored-By: Salvatore Orlando \u003csalv.orlando@gmail.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"9a061dce_2abe0f65","line":13,"updated":"2016-04-06 09:44:43.000000000","message":"Can you please add a APIImpact flag here","commit_id":"705dd88932b3e01c8a6204786eebaa819669d007"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"8a73373efa66af48fbdfe39181e3568d51526bc9","unresolved":false,"context_lines":[{"line_number":10,"context_line":"with a 404 error. It currently returns 204 even if the delete"},{"line_number":11,"context_line":"operation actually does not delete anything which is inconsistent"},{"line_number":12,"context_line":"with other delete operations in the Neutron API."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I1cd91b5e06bd17f9aac97bba71228f2e5c48879b"},{"line_number":15,"context_line":"Closes-Bug: 1307506"},{"line_number":16,"context_line":"Co-Authored-By: Salvatore Orlando \u003csalv.orlando@gmail.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"1a122d0e_95606937","line":13,"in_reply_to":"9a061dce_2abe0f65","updated":"2016-04-18 05:12:01.000000000","message":"Done","commit_id":"705dd88932b3e01c8a6204786eebaa819669d007"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"05e181220e20c4bcdb0ab21e84892d33df19be86","unresolved":false,"context_lines":[{"line_number":11,"context_line":"operation actually does not delete anything which is inconsistent"},{"line_number":12,"context_line":"with other delete operations in the Neutron API."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"APIImpact"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I1cd91b5e06bd17f9aac97bba71228f2e5c48879b"},{"line_number":17,"context_line":"Closes-Bug: 1307506"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"1a122d0e_28181c1d","line":14,"updated":"2016-04-18 06:00:38.000000000","message":"I think that we need to add a release not for this","commit_id":"eb5ae91e872fbf82270177d4798d43e2ffab6515"}],"neutron/db/quota/driver.py":[{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"8ae3caea51224dcf98693c10e6478befff098ea7","unresolved":false,"context_lines":[{"line_number":25,"context_line":"LOG \u003d log.getLogger(__name__)"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"class TenantQuotaNotFound(exceptions.NotFound):"},{"line_number":29,"context_line":"    message \u003d _(\"Quota for tenant %(tenant_id)s could not be found.\")"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"class DbQuotaDriver(object):"}],"source_content_type":"text/x-python","patch_set":4,"id":"bae84128_3d5116d9","line":29,"range":{"start_line":28,"start_character":0,"end_line":29,"end_character":69},"updated":"2016-02-23 09:41:36.000000000","message":"I feel strange a little that the exception is defined in DB file. It\u0027s better to move extension file or other place.","commit_id":"2739c737ed847e65ea0bdbd0797f9dfe4ba5c930"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"d45e0cd45e685374759fcfe28afa5df846581b26","unresolved":false,"context_lines":[{"line_number":67,"context_line":"        with context.session.begin():"},{"line_number":68,"context_line":"            tenant_quotas \u003d context.session.query(quota_models.Quota)"},{"line_number":69,"context_line":"            tenant_quotas \u003d tenant_quotas.filter_by(tenant_id\u003dtenant_id)"},{"line_number":70,"context_line":"            if not tenant_quotas.delete():"},{"line_number":71,"context_line":"                # No record deleted means the quota was not found"},{"line_number":72,"context_line":"                raise exceptions.TenantQuotaNotFound(tenant_id\u003dtenant_id)"},{"line_number":73,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"9a061dce_aa285fe3","line":70,"updated":"2016-04-06 09:44:43.000000000","message":"could tenant_quotas be None here?","commit_id":"705dd88932b3e01c8a6204786eebaa819669d007"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"1687008fe278b11abbd683b75ba993cf4a544f91","unresolved":false,"context_lines":[{"line_number":67,"context_line":"        with context.session.begin():"},{"line_number":68,"context_line":"            tenant_quotas \u003d context.session.query(quota_models.Quota)"},{"line_number":69,"context_line":"            tenant_quotas \u003d tenant_quotas.filter_by(tenant_id\u003dtenant_id)"},{"line_number":70,"context_line":"            if not tenant_quotas.delete():"},{"line_number":71,"context_line":"                # No record deleted means the quota was not found"},{"line_number":72,"context_line":"                raise exceptions.TenantQuotaNotFound(tenant_id\u003dtenant_id)"},{"line_number":73,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"1a122d0e_b58805e9","line":70,"in_reply_to":"9a061dce_aa285fe3","updated":"2016-04-18 05:11:33.000000000","message":"You mean tenant_quota in the DB is None?\nIf it is none, then also delete should work, as it is an entry , isnt it??","commit_id":"705dd88932b3e01c8a6204786eebaa819669d007"}],"neutron/tests/api/admin/test_quotas.py":[{"author":{"_account_id":18573,"name":"Petri Kemppainen","email":"petri.kemppainen@nokia.com","username":"petrkemp"},"change_message_id":"a8c01ac5001c0fdfbdb889558e7dd5d26e1d829e","unresolved":false,"context_lines":[{"line_number":88,"context_line":"            self.assertNotEqual(tenant_id, q[\u0027tenant_id\u0027])"},{"line_number":89,"context_line":""},{"line_number":90,"context_line":"    def _cleanup_quotas(self, project_id):"},{"line_number":91,"context_line":"        # try to clean up the resources.If it fails, then"},{"line_number":92,"context_line":"        # assume that everything was already deleted, so"},{"line_number":93,"context_line":"        # it is OK to continue."},{"line_number":94,"context_line":"        try:"}],"source_content_type":"text/x-python","patch_set":3,"id":"dae33548_844b322b","line":91,"range":{"start_line":91,"start_character":0,"end_line":91,"end_character":57},"updated":"2016-02-15 08:17:55.000000000","message":"minor nit: try should start with capital T","commit_id":"f0ad22dba4aa9c0d3a29d91477501027eb6e1f3c"},{"author":{"_account_id":11159,"name":"Hong Hui Xiao","email":"honghui_xiao@yeah.net","username":"HongHuiXiao"},"change_message_id":"cab7d70de17ec461f074288f8dbd9d2ca1ec789b","unresolved":false,"context_lines":[{"line_number":88,"context_line":"            self.assertNotEqual(tenant_id, q[\u0027tenant_id\u0027])"},{"line_number":89,"context_line":""},{"line_number":90,"context_line":"    def _cleanup_quotas(self, project_id):"},{"line_number":91,"context_line":"        # try to clean up the resources.If it fails, then"},{"line_number":92,"context_line":"        # assume that everything was already deleted, so"},{"line_number":93,"context_line":"        # it is OK to continue."},{"line_number":94,"context_line":"        try:"}],"source_content_type":"text/x-python","patch_set":6,"id":"5aef4532_68de8640","line":91,"range":{"start_line":91,"start_character":10,"end_line":91,"end_character":13},"updated":"2016-03-08 12:42:11.000000000","message":"If this patch gets another version, try-\u003eTry and add a space before \"If\"","commit_id":"a334fdf0d1ef8b292109083f70c0b31d9281ec69"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"2b04af8f132d04e6c1b937e8474ddb134a8cd23e","unresolved":false,"context_lines":[{"line_number":88,"context_line":"            self.assertNotEqual(tenant_id, q[\u0027tenant_id\u0027])"},{"line_number":89,"context_line":""},{"line_number":90,"context_line":"    def _cleanup_quotas(self, project_id):"},{"line_number":91,"context_line":"        # try to clean up the resources.If it fails, then"},{"line_number":92,"context_line":"        # assume that everything was already deleted, so"},{"line_number":93,"context_line":"        # it is OK to continue."},{"line_number":94,"context_line":"        try:"}],"source_content_type":"text/x-python","patch_set":6,"id":"9a061dce_62d15a57","line":91,"range":{"start_line":91,"start_character":10,"end_line":91,"end_character":13},"in_reply_to":"5aef4532_68de8640","updated":"2016-04-06 06:05:13.000000000","message":"Done","commit_id":"a334fdf0d1ef8b292109083f70c0b31d9281ec69"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"d45e0cd45e685374759fcfe28afa5df846581b26","unresolved":false,"context_lines":[{"line_number":60,"context_line":""},{"line_number":61,"context_line":"        # Change quotas for tenant"},{"line_number":62,"context_line":"        quota_set \u003d self.admin_client.update_quotas(tenant_id,"},{"line_number":63,"context_line":"                                                    **new_quotas)"},{"line_number":64,"context_line":"        self.addCleanup(self._cleanup_quotas, tenant_id)"},{"line_number":65,"context_line":"        for key, value in six.iteritems(new_quotas):"},{"line_number":66,"context_line":"            self.assertEqual(value, quota_set[key])"}],"source_content_type":"text/x-python","patch_set":7,"id":"9a061dce_4a5d4378","line":63,"updated":"2016-04-06 09:44:43.000000000","message":"i think that we need a specific unit test","commit_id":"705dd88932b3e01c8a6204786eebaa819669d007"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"1687008fe278b11abbd683b75ba993cf4a544f91","unresolved":false,"context_lines":[{"line_number":60,"context_line":""},{"line_number":61,"context_line":"        # Change quotas for tenant"},{"line_number":62,"context_line":"        quota_set \u003d self.admin_client.update_quotas(tenant_id,"},{"line_number":63,"context_line":"                                                    **new_quotas)"},{"line_number":64,"context_line":"        self.addCleanup(self._cleanup_quotas, tenant_id)"},{"line_number":65,"context_line":"        for key, value in six.iteritems(new_quotas):"},{"line_number":66,"context_line":"            self.assertEqual(value, quota_set[key])"}],"source_content_type":"text/x-python","patch_set":7,"id":"1a122d0e_f56a7d19","line":63,"in_reply_to":"9a061dce_4a5d4378","updated":"2016-04-18 05:11:33.000000000","message":"Wouldnt this suffice?\nhttps://review.openstack.org/#/c/276096/7/neutron/tests/unit/extensions/test_quotasv2.py","commit_id":"705dd88932b3e01c8a6204786eebaa819669d007"}],"neutron/tests/unit/extensions/test_quotasv2.py":[{"author":{"_account_id":20128,"name":"Han Chao","email":"hanchao7@lenovo.com","username":"HanChao"},"change_message_id":"86a1ed9de4108dae8cdcec3dc212be3a00a2527e","unresolved":false,"context_lines":[{"line_number":297,"context_line":"        self.assertEqual(403, res.status_int)"},{"line_number":298,"context_line":""},{"line_number":299,"context_line":"    def test_delete_quota_with_unknown_tenant_returns_404(self):"},{"line_number":300,"context_line":"        tenant_id \u003d \u0027idonotexist\u0027"},{"line_number":301,"context_line":"        env \u003d {\u0027neutron.context\u0027: context.Context(\u0027\u0027, tenant_id + \u00272\u0027,"},{"line_number":302,"context_line":"                                                  is_admin\u003dTrue)}"},{"line_number":303,"context_line":"        res \u003d self.api.delete(_get_path(\u0027quotas\u0027, id\u003dtenant_id, fmt\u003dself.fmt),"}],"source_content_type":"text/x-python","patch_set":6,"id":"5aef4532_3e3fa0d7","line":300,"range":{"start_line":300,"start_character":21,"end_line":300,"end_character":32},"updated":"2016-03-08 10:27:30.000000000","message":"Even though this could be anything, but to be strict, I suppose you would give the name as \u0027idnotexist\u0027?","commit_id":"a334fdf0d1ef8b292109083f70c0b31d9281ec69"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"2b04af8f132d04e6c1b937e8474ddb134a8cd23e","unresolved":false,"context_lines":[{"line_number":297,"context_line":"        self.assertEqual(403, res.status_int)"},{"line_number":298,"context_line":""},{"line_number":299,"context_line":"    def test_delete_quota_with_unknown_tenant_returns_404(self):"},{"line_number":300,"context_line":"        tenant_id \u003d \u0027idonotexist\u0027"},{"line_number":301,"context_line":"        env \u003d {\u0027neutron.context\u0027: context.Context(\u0027\u0027, tenant_id + \u00272\u0027,"},{"line_number":302,"context_line":"                                                  is_admin\u003dTrue)}"},{"line_number":303,"context_line":"        res \u003d self.api.delete(_get_path(\u0027quotas\u0027, id\u003dtenant_id, fmt\u003dself.fmt),"}],"source_content_type":"text/x-python","patch_set":6,"id":"9a061dce_a27d3270","line":300,"range":{"start_line":300,"start_character":21,"end_line":300,"end_character":32},"in_reply_to":"5aef4532_3e3fa0d7","updated":"2016-04-06 06:05:13.000000000","message":"Well, ok... :)","commit_id":"a334fdf0d1ef8b292109083f70c0b31d9281ec69"},{"author":{"_account_id":11159,"name":"Hong Hui Xiao","email":"honghui_xiao@yeah.net","username":"HongHuiXiao"},"change_message_id":"cab7d70de17ec461f074288f8dbd9d2ca1ec789b","unresolved":false,"context_lines":[{"line_number":302,"context_line":"                                                  is_admin\u003dTrue)}"},{"line_number":303,"context_line":"        res \u003d self.api.delete(_get_path(\u0027quotas\u0027, id\u003dtenant_id, fmt\u003dself.fmt),"},{"line_number":304,"context_line":"                              extra_environ\u003denv, expect_errors\u003dTrue)"},{"line_number":305,"context_line":"        self.assertEqual(404, res.status_int)"},{"line_number":306,"context_line":""},{"line_number":307,"context_line":"    def test_quotas_loaded_bad_returns_404(self):"},{"line_number":308,"context_line":"        try:"}],"source_content_type":"text/x-python","patch_set":6,"id":"5aef4532_e8c5763c","line":305,"range":{"start_line":305,"start_character":25,"end_line":305,"end_character":28},"updated":"2016-03-08 12:42:11.000000000","message":"Given the fact that webob.exc has been imported. How about using \"exc.HTTPNotFound.code\" here?","commit_id":"a334fdf0d1ef8b292109083f70c0b31d9281ec69"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"2b04af8f132d04e6c1b937e8474ddb134a8cd23e","unresolved":false,"context_lines":[{"line_number":302,"context_line":"                                                  is_admin\u003dTrue)}"},{"line_number":303,"context_line":"        res \u003d self.api.delete(_get_path(\u0027quotas\u0027, id\u003dtenant_id, fmt\u003dself.fmt),"},{"line_number":304,"context_line":"                              extra_environ\u003denv, expect_errors\u003dTrue)"},{"line_number":305,"context_line":"        self.assertEqual(404, res.status_int)"},{"line_number":306,"context_line":""},{"line_number":307,"context_line":"    def test_quotas_loaded_bad_returns_404(self):"},{"line_number":308,"context_line":"        try:"}],"source_content_type":"text/x-python","patch_set":6,"id":"5aef4532_e8829693","line":305,"range":{"start_line":305,"start_character":25,"end_line":305,"end_character":28},"in_reply_to":"5aef4532_e8c5763c","updated":"2016-04-06 06:05:13.000000000","message":"Ok, let me do it in the next patch","commit_id":"a334fdf0d1ef8b292109083f70c0b31d9281ec69"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"b5f3fa361aa35033a871f54fdc386b8112163d80","unresolved":false,"context_lines":[{"line_number":298,"context_line":""},{"line_number":299,"context_line":"    def test_delete_quota_with_unknown_tenant_returns_404(self):"},{"line_number":300,"context_line":"        tenant_id \u003d \u0027idnotexist\u0027"},{"line_number":301,"context_line":"        env \u003d {\u0027neutron.context\u0027: context.Context(\u0027\u0027, tenant_id + \u00272\u0027,"},{"line_number":302,"context_line":"                                                  is_admin\u003dTrue)}"},{"line_number":303,"context_line":"        res \u003d self.api.delete(_get_path(\u0027quotas\u0027, id\u003dtenant_id, fmt\u003dself.fmt),"},{"line_number":304,"context_line":"                              extra_environ\u003denv, expect_errors\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":10,"id":"1a122d0e_4d5a9450","line":301,"range":{"start_line":301,"start_character":63,"end_line":301,"end_character":69},"updated":"2016-04-19 03:29:03.000000000","message":"Why is \u00272\u0027 added to tenant_id? I guess that it\u0027s unnecessary.","commit_id":"915352725296de45a74ffd2ea1edc4c498eb4a8f"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"c515567d837de33cdc8ccc32d637b8e52136ad27","unresolved":false,"context_lines":[{"line_number":298,"context_line":""},{"line_number":299,"context_line":"    def test_delete_quota_with_unknown_tenant_returns_404(self):"},{"line_number":300,"context_line":"        tenant_id \u003d \u0027idnotexist\u0027"},{"line_number":301,"context_line":"        env \u003d {\u0027neutron.context\u0027: context.Context(\u0027\u0027, tenant_id + \u00272\u0027,"},{"line_number":302,"context_line":"                                                  is_admin\u003dTrue)}"},{"line_number":303,"context_line":"        res \u003d self.api.delete(_get_path(\u0027quotas\u0027, id\u003dtenant_id, fmt\u003dself.fmt),"},{"line_number":304,"context_line":"                              extra_environ\u003denv, expect_errors\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":10,"id":"1a122d0e_ad16c852","line":301,"range":{"start_line":301,"start_character":63,"end_line":301,"end_character":69},"in_reply_to":"1a122d0e_4d5a9450","updated":"2016-04-19 03:31:52.000000000","message":"The env is set for tenant_id + 2\nbut the delete is called for tenant id\ntenant_id wont exist in the env\nso the test \"should\" pass...","commit_id":"915352725296de45a74ffd2ea1edc4c498eb4a8f"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"b078aad3a5c062893c456c520066d237dba45d59","unresolved":false,"context_lines":[{"line_number":298,"context_line":""},{"line_number":299,"context_line":"    def test_delete_quota_with_unknown_tenant_returns_404(self):"},{"line_number":300,"context_line":"        tenant_id \u003d \u0027idnotexist\u0027"},{"line_number":301,"context_line":"        env \u003d {\u0027neutron.context\u0027: context.Context(\u0027\u0027, tenant_id + \u00272\u0027,"},{"line_number":302,"context_line":"                                                  is_admin\u003dTrue)}"},{"line_number":303,"context_line":"        res \u003d self.api.delete(_get_path(\u0027quotas\u0027, id\u003dtenant_id, fmt\u003dself.fmt),"},{"line_number":304,"context_line":"                              extra_environ\u003denv, expect_errors\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":10,"id":"1a122d0e_f1425a3d","line":301,"range":{"start_line":301,"start_character":63,"end_line":301,"end_character":69},"in_reply_to":"1a122d0e_ad16c852","updated":"2016-04-19 03:36:26.000000000","message":"I see.","commit_id":"915352725296de45a74ffd2ea1edc4c498eb4a8f"}],"releasenotes/notes/404-for-quota-tenant-2c09c16759269b21.yaml":[{"author":{"_account_id":14605,"name":"vikram.choudhary","email":"vikschw@gmail.com","username":"vikram.choudhary"},"change_message_id":"d4eafb18842726243a7b65c6e3a730a85baa8c0f","unresolved":false,"context_lines":[{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Return code for `quota delete` for a tenant whose"},{"line_number":5,"context_line":"    quota has not been previously defined has been "},{"line_number":6,"context_line":"    changed from 204 to 404."}],"source_content_type":"text/x-yaml","patch_set":9,"id":"1a122d0e_de0839dc","line":5,"updated":"2016-04-18 09:21:59.000000000","message":"nit: white-spaces","commit_id":"22fe5262b25295416acf2c51adf5029ba6239f87"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"b97fe93426685cf1df86a9ef27ae27aa665c0549","unresolved":false,"context_lines":[{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Return code for `quota delete` for a tenant whose"},{"line_number":5,"context_line":"    quota has not been previously defined has been "},{"line_number":6,"context_line":"    changed from 204 to 404."}],"source_content_type":"text/x-yaml","patch_set":9,"id":"1a122d0e_01009600","line":5,"updated":"2016-04-18 09:21:15.000000000","message":"whitespace","commit_id":"22fe5262b25295416acf2c51adf5029ba6239f87"}]}