)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"444c8ebd1cc1fe45ad8c91204750a9911d85c9a3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"4f5ca7f4_8be74835","updated":"2021-10-25 23:23:19.000000000","message":"A couple of questions here, otherwise looks good to me","commit_id":"ce22e48b5484aebe198df8fff906a69058be28ed"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"2def6d7a8c6e664b3121b9179db34c3747b10962","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"46d6ce57_56d477e7","updated":"2021-10-28 17:18:16.000000000","message":"LGTM","commit_id":"a8a8eaf69417437c56e889950dde8ca2429eae1c"}],"designate_tempest_plugin/tests/scenario/v2/test_quotas.py":[{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"444c8ebd1cc1fe45ad8c91204750a9911d85c9a3","unresolved":true,"context_lines":[{"line_number":60,"context_line":""},{"line_number":61,"context_line":"        alt_project_id \u003d self.alt_client.project_id"},{"line_number":62,"context_line":"        http_header \u003d {\u0027x-auth-sudo-project-id\u0027: alt_project_id}"},{"line_number":63,"context_line":"        limit_zones_quota \u003d 5"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"        LOG.info(\u0027As Admin user set Zones quota for Alt user \u0027"},{"line_number":66,"context_line":"                 \u0027to:{} \u0027.format(limit_zones_quota))"}],"source_content_type":"text/x-python","patch_set":4,"id":"f8d18966_6989d19b","line":63,"updated":"2021-10-25 23:23:19.000000000","message":"Why 5?\nCan we use a lower number to save test run time?","commit_id":"ce22e48b5484aebe198df8fff906a69058be28ed"},{"author":{"_account_id":28609,"name":"Arkady Shtempler","email":"ashtempl@redhat.com","username":"ashtempl"},"change_message_id":"109bf3fecd364d781ee5ae80374b3b1628dec0c9","unresolved":true,"context_lines":[{"line_number":60,"context_line":""},{"line_number":61,"context_line":"        alt_project_id \u003d self.alt_client.project_id"},{"line_number":62,"context_line":"        http_header \u003d {\u0027x-auth-sudo-project-id\u0027: alt_project_id}"},{"line_number":63,"context_line":"        limit_zones_quota \u003d 5"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"        LOG.info(\u0027As Admin user set Zones quota for Alt user \u0027"},{"line_number":66,"context_line":"                 \u0027to:{} \u0027.format(limit_zones_quota))"}],"source_content_type":"text/x-python","patch_set":4,"id":"e9b8e6a5_df69a567","line":63,"in_reply_to":"f8d18966_6989d19b","updated":"2021-10-26 11:45:55.000000000","message":"Changed to 3","commit_id":"ce22e48b5484aebe198df8fff906a69058be28ed"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"444c8ebd1cc1fe45ad8c91204750a9911d85c9a3","unresolved":true,"context_lines":[{"line_number":85,"context_line":"                over_quota_error \u003d False"},{"line_number":86,"context_line":"            except Exception as err:"},{"line_number":87,"context_line":"                raised_error \u003d str(err).replace(\u0027 \u0027, \u0027\u0027)"},{"line_number":88,"context_line":"                if \"\u0027code\u0027:413\" and \"\u0027type\u0027:\u0027over_quota\u0027\" in raised_error:"},{"line_number":89,"context_line":"                    over_quota_error \u003d True"},{"line_number":90,"context_line":"        self.assertTrue("},{"line_number":91,"context_line":"            over_quota_error,"}],"source_content_type":"text/x-python","patch_set":4,"id":"3803f55a_abad2e0a","line":88,"updated":"2021-10-25 23:23:19.000000000","message":"What if the API had a bug that caused it to return 413 at iteration 2 instead of at 6?\nIt seems like we can remove the \"over_quota_error\" boolean if we just check the attempt_number count is the expected zone creation that should trigger the failure.","commit_id":"ce22e48b5484aebe198df8fff906a69058be28ed"},{"author":{"_account_id":28609,"name":"Arkady Shtempler","email":"ashtempl@redhat.com","username":"ashtempl"},"change_message_id":"109bf3fecd364d781ee5ae80374b3b1628dec0c9","unresolved":true,"context_lines":[{"line_number":85,"context_line":"                over_quota_error \u003d False"},{"line_number":86,"context_line":"            except Exception as err:"},{"line_number":87,"context_line":"                raised_error \u003d str(err).replace(\u0027 \u0027, \u0027\u0027)"},{"line_number":88,"context_line":"                if \"\u0027code\u0027:413\" and \"\u0027type\u0027:\u0027over_quota\u0027\" in raised_error:"},{"line_number":89,"context_line":"                    over_quota_error \u003d True"},{"line_number":90,"context_line":"        self.assertTrue("},{"line_number":91,"context_line":"            over_quota_error,"}],"source_content_type":"text/x-python","patch_set":4,"id":"b62602d5_45e4fde0","line":88,"in_reply_to":"3803f55a_abad2e0a","updated":"2021-10-26 11:45:55.000000000","message":"Fixed, thanks!","commit_id":"ce22e48b5484aebe198df8fff906a69058be28ed"}]}