)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":32755,"name":"Christian Rohmann","email":"christian.rohmann@inovex.de","username":"frittentheke"},"change_message_id":"6bc60051ca16013939113c7e2210e932048f768e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ab0d4222_ebad86e6","updated":"2021-10-15 20:53:20.000000000","message":"Artem, I did cherry-pick this from Simons change https://review.opendev.org/c/openstack/openstacksdk/+/814179 as you asked to commit to feature/r1 first.\n","commit_id":"4aaadeac03d4deaccc08f747cb0308a735489278"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"5b76c81bb5fc8bb41f2c74107b912b401db9a3b2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d053ca85_1ae0e639","updated":"2021-10-27 11:40:24.000000000","message":"Left a comment about openstack.tests.unit.cloud.test_volume.TestVolume.test_get_volume_by_id, the other failures are probably due to the same issue.","commit_id":"4aaadeac03d4deaccc08f747cb0308a735489278"},{"author":{"_account_id":32755,"name":"Christian Rohmann","email":"christian.rohmann@inovex.de","username":"frittentheke"},"change_message_id":"f582b9ce3161dd88d51b273ebd3585a36e51db5e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"000624c1_ea5a073e","updated":"2021-11-04 12:01:18.000000000","message":"Thanks Kendall for the +2. Should I add anybody else to the reviewers list to get this merged?\nMaybe Brian, you could take a peek? We chatted about this change (read: the issue with the tests) via IRC the other day.","commit_id":"9564085691d838cd5c250f2f93db503c46ead8be"},{"author":{"_account_id":32755,"name":"Christian Rohmann","email":"christian.rohmann@inovex.de","username":"frittentheke"},"change_message_id":"8990df3c5d26bb3d2e06ec6f349f400cf7e34c40","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"85a7bd77_41040751","updated":"2021-10-28 07:39:39.000000000","message":"recheck","commit_id":"9564085691d838cd5c250f2f93db503c46ead8be"},{"author":{"_account_id":32755,"name":"Christian Rohmann","email":"christian.rohmann@inovex.de","username":"frittentheke"},"change_message_id":"8b598316f8157bc38658a133fafb490f0138b8df","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"8e117f4c_953a209f","updated":"2021-10-28 09:54:28.000000000","message":"recheck","commit_id":"9564085691d838cd5c250f2f93db503c46ead8be"},{"author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"change_message_id":"51ed3e8cfe0c513b10340781b0b8f4a433af6072","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c3f2145b_8ec49a9a","updated":"2021-11-16 12:10:24.000000000","message":"recheck","commit_id":"9564085691d838cd5c250f2f93db503c46ead8be"}],"openstack/tests/fakes.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"5b76c81bb5fc8bb41f2c74107b912b401db9a3b2","unresolved":true,"context_lines":[{"line_number":346,"context_line":"    }))"},{"line_number":347,"context_line":""},{"line_number":348,"context_line":""},{"line_number":349,"context_line":"class FakeVolume:"},{"line_number":350,"context_line":"    def __init__("},{"line_number":351,"context_line":"            self, id, status, name, attachments\u003d[],"},{"line_number":352,"context_line":"            size\u003d75):"}],"source_content_type":"text/x-python","patch_set":1,"id":"bd8f9187_9433d779","line":349,"updated":"2021-10-27 11:40:24.000000000","message":"This is a fake volume response.  Your patch is adding the updated_at field, which isn\u0027t listed below, and so it won\u0027t show up in the expected result when you create vol1 at line https://review.opendev.org/c/openstack/openstacksdk/+/814232/1/openstack/tests/unit/cloud/test_volume.py#383","commit_id":"4aaadeac03d4deaccc08f747cb0308a735489278"},{"author":{"_account_id":32755,"name":"Christian Rohmann","email":"christian.rohmann@inovex.de","username":"frittentheke"},"change_message_id":"7ed2e7846f3d5689bc1ab925985292ec03dbf541","unresolved":false,"context_lines":[{"line_number":346,"context_line":"    }))"},{"line_number":347,"context_line":""},{"line_number":348,"context_line":""},{"line_number":349,"context_line":"class FakeVolume:"},{"line_number":350,"context_line":"    def __init__("},{"line_number":351,"context_line":"            self, id, status, name, attachments\u003d[],"},{"line_number":352,"context_line":"            size\u003d75):"}],"source_content_type":"text/x-python","patch_set":1,"id":"de5f9d71_26e2d8ac","line":349,"in_reply_to":"bd8f9187_9433d779","updated":"2022-09-16 15:41:56.000000000","message":"Done","commit_id":"4aaadeac03d4deaccc08f747cb0308a735489278"}],"openstack/tests/unit/cloud/test_volume.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"5b76c81bb5fc8bb41f2c74107b912b401db9a3b2","unresolved":true,"context_lines":[{"line_number":390,"context_line":"                 json\u003d{\u0027volume\u0027: vol1}"},{"line_number":391,"context_line":"                 )"},{"line_number":392,"context_line":"        ])"},{"line_number":393,"context_line":"        self._compare_volumes(vol1, self.cloud.get_volume_by_id(\u002701\u0027))"},{"line_number":394,"context_line":"        self.assert_calls()"},{"line_number":395,"context_line":""},{"line_number":396,"context_line":"    def test_create_volume(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"c389e9d8_bdedd63f","line":393,"updated":"2021-10-27 11:40:24.000000000","message":"See comment in fakes.py","commit_id":"4aaadeac03d4deaccc08f747cb0308a735489278"}]}