)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"321f715bdfb5c18cca36874ba06f8c4c9c006c1a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"9ec32d32_a9426025","updated":"2024-01-29 22:24:18.000000000","message":"Hi! It seems we have a CI issue here. Zuul is not on the reviewers list, therefore gate jobs do not start. I can see different Zuuls in a list of possible reviewers, but none looks right. Correct Zuul account is one without the e-mail.","commit_id":"34e657c3041f825eac1c730f867ec022b15f2df9"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"c274f0e323c8907621df98eceb204800f4d692a4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"e6b77224_b1fae9f0","updated":"2024-11-11 23:15:06.000000000","message":"recheck","commit_id":"28bc6699233f1c1c6fc9d8a6432d1430e7846390"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"0c1996c4dc3c0a17eab176402d9d0f4dd5086f73","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"28e9c45c_e8df2978","updated":"2024-02-05 19:47:08.000000000","message":"recheck fresh results","commit_id":"28bc6699233f1c1c6fc9d8a6432d1430e7846390"}],"ironicclient/tests/functional/osc/v1/test_baremetal_chassis_create.py":[{"author":{"_account_id":13719,"name":"Naohiro Tamura","email":"naohirot.openstack@gmail.com","username":"nao"},"change_message_id":"04fbcf635df8dce6f23135889b607d84657048bb","unresolved":false,"context_lines":[{"line_number":33,"context_line":"        return params_str"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    def assertEqualDict(self, params, object_params):"},{"line_number":36,"context_line":"        object_params \u003d ast.literal_eval(str(object_params))"},{"line_number":37,"context_line":"        self.assertEqual(params, object_params)"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"    def test_description(self):"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a77a97e_d1593c90","line":36,"range":{"start_line":36,"start_character":41,"end_line":36,"end_character":58},"updated":"2016-11-22 05:57:48.000000000","message":"I just wondered if we need to call str() for \"object_params\".\nBecause if \"object_params\" is not string, ast.literal_eval() is not necessary, I think.\nDo we need to call str()?","commit_id":"1561cdc81defa9c780ebbf7e7762c1dd52092bbd"},{"author":{"_account_id":13719,"name":"Naohiro Tamura","email":"naohirot.openstack@gmail.com","username":"nao"},"change_message_id":"04fbcf635df8dce6f23135889b607d84657048bb","unresolved":false,"context_lines":[{"line_number":33,"context_line":"        return params_str"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    def assertEqualDict(self, params, object_params):"},{"line_number":36,"context_line":"        object_params \u003d ast.literal_eval(str(object_params))"},{"line_number":37,"context_line":"        self.assertEqual(params, object_params)"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"    def test_description(self):"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a77a97e_912a64e6","line":36,"range":{"start_line":36,"start_character":8,"end_line":36,"end_character":21},"updated":"2016-11-22 05:57:48.000000000","message":"It is better to change this variable name to different name from \"object_params\" which is the third argument of the method.","commit_id":"1561cdc81defa9c780ebbf7e7762c1dd52092bbd"},{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"e9f4b69930baf334a867d1e89497319c4d28e229","unresolved":false,"context_lines":[{"line_number":33,"context_line":"        return params_str"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    def assertEqualDict(self, params, object_params):"},{"line_number":36,"context_line":"        object_params \u003d ast.literal_eval(str(object_params))"},{"line_number":37,"context_line":"        self.assertEqual(params, object_params)"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"    def test_description(self):"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a77a97e_41aa96d3","line":36,"range":{"start_line":36,"start_character":41,"end_line":36,"end_character":58},"in_reply_to":"7a77a97e_d1593c90","updated":"2016-11-22 16:59:48.000000000","message":"Let\u0027s check, it looks, possibly, that we can assert equality of dicts directly.","commit_id":"1561cdc81defa9c780ebbf7e7762c1dd52092bbd"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"66ad9d5dfeb8b564c4e3abb6043c1f8b67defa9b","unresolved":false,"context_lines":[{"line_number":21,"context_line":"    \"\"\"Functional tests for baremetal chassis create command with options.\"\"\""},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"    def setUp(self):"},{"line_number":24,"context_line":"        super(BaremetalChassisCreate, self).setUp()"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"    @staticmethod"},{"line_number":27,"context_line":"    def generate_params(argument, params):"}],"source_content_type":"text/x-python","patch_set":10,"id":"9f560f44_e5f9534e","line":24,"updated":"2020-09-08 08:35:19.000000000","message":"nit: not needed","commit_id":"34e657c3041f825eac1c730f867ec022b15f2df9"}]}