)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":28609,"name":"Arkady Shtempler","email":"ashtempl@redhat.com","username":"ashtempl"},"change_message_id":"f24c12d539e86859134049bfcfeb463816ca8d96","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"2128679d_057ac458","updated":"2022-05-01 11:48:55.000000000","message":"recheck","commit_id":"f9de863f8f0c18fd7a9f729d7d8c63f7982c3a5c"},{"author":{"_account_id":28609,"name":"Arkady Shtempler","email":"ashtempl@redhat.com","username":"ashtempl"},"change_message_id":"2d37477b775563258ee11b1cd97a9f857b375d6c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"61029396_0490d91b","updated":"2022-04-27 12:24:28.000000000","message":"recheck","commit_id":"f9de863f8f0c18fd7a9f729d7d8c63f7982c3a5c"},{"author":{"_account_id":5572,"name":"Don Kehn","display_name":"DEKehn","email":"dekehn@gmail.com","username":"dekehn"},"change_message_id":"0528c5371c8d0b45f13c0704cc112195d5f0b604","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"61fc6c91_ff7dce02","updated":"2022-05-11 21:02:55.000000000","message":"LGTM","commit_id":"df81fe6228b8f29bf84504add0d340a90dca568d"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"6c4df17e959d103c6e17af4b7a195f810c61d01d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"840c1e0c_abe47b92","updated":"2022-05-09 22:11:28.000000000","message":"LGTM","commit_id":"df81fe6228b8f29bf84504add0d340a90dca568d"},{"author":{"_account_id":31664,"name":"Omer Schwartz","email":"oschwart@redhat.com","username":"oschwart"},"change_message_id":"83da9c65906a31d829748099453e552ffbd9a811","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"584435c9_30faab89","updated":"2022-05-09 09:59:56.000000000","message":"Python LGTM, left a few nits.","commit_id":"df81fe6228b8f29bf84504add0d340a90dca568d"},{"author":{"_account_id":28609,"name":"Arkady Shtempler","email":"ashtempl@redhat.com","username":"ashtempl"},"change_message_id":"b6e5e37e0754237fed7bf0dcb1b149a3789b69c6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"50984a85_a9d0f97d","updated":"2022-05-08 14:12:48.000000000","message":"recheck","commit_id":"df81fe6228b8f29bf84504add0d340a90dca568d"}],"designate_tempest_plugin/tests/api/v2/test_quotas.py":[{"author":{"_account_id":31664,"name":"Omer Schwartz","email":"oschwart@redhat.com","username":"oschwart"},"change_message_id":"83da9c65906a31d829748099453e552ffbd9a811","unresolved":true,"context_lines":[{"line_number":87,"context_line":"            for quota_type, quota_value in body.items():"},{"line_number":88,"context_line":"                self.assertTrue("},{"line_number":89,"context_line":"                    isinstance(quota_value, int),"},{"line_number":90,"context_line":"                    \u0027Failed, the value of:{} is:{}, expected integer\u0027.format("},{"line_number":91,"context_line":"                        quota_type, quota_value))"},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"    @decorators.idempotent_id(\u00270448b089-5803-4ce3-8a6c-5c15ff75a2cc\u0027)"}],"source_content_type":"text/x-python","patch_set":7,"id":"45f9e730_46fd82dc","line":90,"updated":"2022-05-09 09:59:56.000000000","message":"nit: maybe you could also log type(quota_value) in case it fails? Seems like you are asserting its type.","commit_id":"df81fe6228b8f29bf84504add0d340a90dca568d"},{"author":{"_account_id":31664,"name":"Omer Schwartz","email":"oschwart@redhat.com","username":"oschwart"},"change_message_id":"83da9c65906a31d829748099453e552ffbd9a811","unresolved":true,"context_lines":[{"line_number":297,"context_line":""},{"line_number":298,"context_line":"        LOG.info(\u0027Try to set quota using not existing quota type in its body\u0027)"},{"line_number":299,"context_line":"        primary_project_id \u003d self.quotas_client.project_id"},{"line_number":300,"context_line":"        quota \u003d dns_data_utils.rand_quotas()"},{"line_number":301,"context_line":"        quota[tempest_data_utils.rand_name()] \u003d 777"},{"line_number":302,"context_line":""},{"line_number":303,"context_line":"        with self.assertRaisesDns("}],"source_content_type":"text/x-python","patch_set":7,"id":"85a778d6_97ef3d79","line":300,"range":{"start_line":300,"start_character":8,"end_line":300,"end_character":13},"updated":"2022-05-09 09:59:56.000000000","message":"nit: maybe you could call that variable rand_quotas? Or any multiple named variable? IMO quota (single) might sound a little confusing on lines 301, 307.","commit_id":"df81fe6228b8f29bf84504add0d340a90dca568d"}]}