)]}'
{"novaclient/tests/functional/test_quotas.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"49741d74ac936ebd006fbfc83836488d7decae71","unresolved":false,"context_lines":[{"line_number":47,"context_line":"        for quota_name in self._quota_resources:"},{"line_number":48,"context_line":"            params.append(\"--%(name)s %(value)s\" % {"},{"line_number":49,"context_line":"                \"name\": quota_name.replace(\"_\", \"-\"),"},{"line_number":50,"context_line":"                \"value\": getattr(original_quotas, quota_name) + 1})"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"        self.nova(\"quota-update\", params\u003d\" \".join(params))"},{"line_number":53,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"da20952f_2b1731ef","line":50,"updated":"2015-09-04 13:27:32.000000000","message":"You have to increment higher than 1 I think, we had a similar issue in Tempest:\n\ne3e9da70c611d686a62c630fa9a2ddfb420ae8c4\n\nBecause fixed_ips defaults to -1 I believe so incrementing by 1 caused that to be 0 and then over quota for anything booting instances against that tenant.","commit_id":"a5f8ff3c22d6c015705a5e88bef190c0455ac772"},{"author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"change_message_id":"4ef09952d7a09dc2143eb67fb2a07563892230b2","unresolved":false,"context_lines":[{"line_number":47,"context_line":"        for quota_name in self._quota_resources:"},{"line_number":48,"context_line":"            params.append(\"--%(name)s %(value)s\" % {"},{"line_number":49,"context_line":"                \"name\": quota_name.replace(\"_\", \"-\"),"},{"line_number":50,"context_line":"                \"value\": getattr(original_quotas, quota_name) + 1})"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"        self.nova(\"quota-update\", params\u003d\" \".join(params))"},{"line_number":53,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"da20952f_abe4a1a7","line":50,"in_reply_to":"da20952f_2b1731ef","updated":"2015-09-04 13:38:51.000000000","message":"tox runs functional tests serially(https://review.openstack.org/#/c/217768/), so this tests should not affect other tests.","commit_id":"a5f8ff3c22d6c015705a5e88bef190c0455ac772"},{"author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"change_message_id":"971e4d4d498ae09b0c75b8867be1a260fdf2a193","unresolved":false,"context_lines":[{"line_number":47,"context_line":"        for quota_name in self._quota_resources:"},{"line_number":48,"context_line":"            params.append(\"--%(name)s %(value)s\" % {"},{"line_number":49,"context_line":"                \"name\": quota_name.replace(\"_\", \"-\"),"},{"line_number":50,"context_line":"                \"value\": getattr(original_quotas, quota_name) + 1})"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"        self.nova(\"quota-update\", params\u003d\" \".join(params))"},{"line_number":53,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"da20952f_2b2c11ab","line":50,"in_reply_to":"da20952f_ab402160","updated":"2015-09-04 13:42:04.000000000","message":"sure, I\u0027ll change it(already canhged locally). Just want to wait until functional tests finished for this patch-set.","commit_id":"a5f8ff3c22d6c015705a5e88bef190c0455ac772"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"5eca541f4035922a7c1344f4848a3bf1538c8cc7","unresolved":false,"context_lines":[{"line_number":47,"context_line":"        for quota_name in self._quota_resources:"},{"line_number":48,"context_line":"            params.append(\"--%(name)s %(value)s\" % {"},{"line_number":49,"context_line":"                \"name\": quota_name.replace(\"_\", \"-\"),"},{"line_number":50,"context_line":"                \"value\": getattr(original_quotas, quota_name) + 1})"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"        self.nova(\"quota-update\", params\u003d\" \".join(params))"},{"line_number":53,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"da20952f_ab402160","line":50,"in_reply_to":"da20952f_abe4a1a7","updated":"2015-09-04 13:40:11.000000000","message":"Until we change that and introduce the bug - it doesn\u0027t hurt to just change the 1 to 100 now like we have in tempest.  I\u0027m sure sdague will be +2 on it again and then I\u0027ll +W.","commit_id":"a5f8ff3c22d6c015705a5e88bef190c0455ac772"}]}