)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"ac44329770423dcd76b39f1d406f8f92d74abc80","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c5a16f74_97ff7444","updated":"2021-11-09 09:15:48.000000000","message":"thanks for bringing this up, one small detail, other than that it looks ok","commit_id":"c1e7e268c370b34bd1ad826c020c3c5c4df4d90d"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"7c0d8df2dfb2302a8653290b7d0a78b1ddd7836e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"d980f7eb_64097aab","updated":"2021-11-10 12:43:46.000000000","message":"the line is too long - https://zuul.opendev.org/t/openstack/build/79bd7a0acc62473698d3bce9e5f53e6f \nafter that\u0027s fixed we can proceed with this","commit_id":"5708f173941f6e260472866470420558d8c1db1f"},{"author":{"_account_id":14624,"name":"Sam Wan","email":"sam.wan@emc.com","username":"samwan"},"change_message_id":"cfae6801fd02bef5833f319a4e711248e48f74fc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c6b06819_31075c57","in_reply_to":"d980f7eb_64097aab","updated":"2021-11-15 08:06:01.000000000","message":"fixed","commit_id":"5708f173941f6e260472866470420558d8c1db1f"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"ac06a3d2bf4f0a9b6297052016374ff825a7f5b1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"be0b3a99_9420c233","updated":"2021-11-15 10:51:24.000000000","message":"thanks","commit_id":"0bacf984313a337cdfce49d61949e585aa134e19"}],"tempest/api/volume/test_volumes_snapshots.py":[{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"ac44329770423dcd76b39f1d406f8f92d74abc80","unresolved":true,"context_lines":[{"line_number":167,"context_line":"    @decorators.idempotent_id(\u0027677863d1-3142-456d-b6ac-9924f667a7f4\u0027)"},{"line_number":168,"context_line":"    def test_volume_from_snapshot(self):"},{"line_number":169,"context_line":"        \"\"\"Test creating volume from snapshot with extending size\"\"\""},{"line_number":170,"context_line":"        self._create_volume_from_snapshot(extra_size\u003dCONF.volume.volume_size)"},{"line_number":171,"context_line":""},{"line_number":172,"context_line":"    @decorators.idempotent_id(\u0027053d8870-8282-4fff-9dbb-99cb58bb5e0a\u0027)"},{"line_number":173,"context_line":"    def test_volume_from_snapshot_no_size(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"96ab10c5_3a346370","line":170,"range":{"start_line":170,"start_character":65,"end_line":170,"end_character":76},"updated":"2021-11-09 09:15:48.000000000","message":"could you please use volume_size_extend instead - https://opendev.org/openstack/tempest/commit/91d5fe0573f8374ee7170bc5ca286a7dd8c3d07d","commit_id":"c1e7e268c370b34bd1ad826c020c3c5c4df4d90d"},{"author":{"_account_id":14624,"name":"Sam Wan","email":"sam.wan@emc.com","username":"samwan"},"change_message_id":"202c758eeb63a527427b24a9af4cd65267293e09","unresolved":false,"context_lines":[{"line_number":167,"context_line":"    @decorators.idempotent_id(\u0027677863d1-3142-456d-b6ac-9924f667a7f4\u0027)"},{"line_number":168,"context_line":"    def test_volume_from_snapshot(self):"},{"line_number":169,"context_line":"        \"\"\"Test creating volume from snapshot with extending size\"\"\""},{"line_number":170,"context_line":"        self._create_volume_from_snapshot(extra_size\u003dCONF.volume.volume_size)"},{"line_number":171,"context_line":""},{"line_number":172,"context_line":"    @decorators.idempotent_id(\u0027053d8870-8282-4fff-9dbb-99cb58bb5e0a\u0027)"},{"line_number":173,"context_line":"    def test_volume_from_snapshot_no_size(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"3189c097_89efb683","line":170,"range":{"start_line":170,"start_character":65,"end_line":170,"end_character":76},"in_reply_to":"6b8bb7d6_eb48281d","updated":"2021-11-09 10:25:19.000000000","message":"Hi Martin, thanks for the advice, yes \u0027volume_size_extend\u0027 is more appropriate.","commit_id":"c1e7e268c370b34bd1ad826c020c3c5c4df4d90d"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"d826e6ae90178cba9e6a2b2dd754690bdff7bca8","unresolved":true,"context_lines":[{"line_number":167,"context_line":"    @decorators.idempotent_id(\u0027677863d1-3142-456d-b6ac-9924f667a7f4\u0027)"},{"line_number":168,"context_line":"    def test_volume_from_snapshot(self):"},{"line_number":169,"context_line":"        \"\"\"Test creating volume from snapshot with extending size\"\"\""},{"line_number":170,"context_line":"        self._create_volume_from_snapshot(extra_size\u003dCONF.volume.volume_size)"},{"line_number":171,"context_line":""},{"line_number":172,"context_line":"    @decorators.idempotent_id(\u0027053d8870-8282-4fff-9dbb-99cb58bb5e0a\u0027)"},{"line_number":173,"context_line":"    def test_volume_from_snapshot_no_size(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"6b8bb7d6_eb48281d","line":170,"range":{"start_line":170,"start_character":65,"end_line":170,"end_character":76},"in_reply_to":"96ab10c5_3a346370","updated":"2021-11-09 09:17:09.000000000","message":"oh, i meant to send this link - https://opendev.org/openstack/tempest/commit/37c4bd3023110f2a00e55439e2086d393b44a974 , ignore the previous one","commit_id":"c1e7e268c370b34bd1ad826c020c3c5c4df4d90d"}]}
