)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":35834,"name":"Yosi Ben Shimon","display_name":"Yosi Ben Shimon","email":"ybenshim@redhat.com","username":"ybenshim"},"change_message_id":"a779a0f0398b6949ad57e6c59248f6252e6232ba","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c8bc46b1_acc976fa","updated":"2024-06-17 11:31:46.000000000","message":"Thank you Liron!","commit_id":"c4f384b0901f71e743b7f3d938a3dadc66d7a61d"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"45704b857ece0eab6b2bd87ca85feed28b48ec23","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d6c593fa_4a4a6935","updated":"2024-06-17 10:01:44.000000000","message":"Thanks Yosi!\nI noticed there are three failures in the Reader tests. It seems that the Reader can perform actions that are not allowed.\n\nIn addition, please fix the pep8 error.","commit_id":"c4f384b0901f71e743b7f3d938a3dadc66d7a61d"},{"author":{"_account_id":35834,"name":"Yosi Ben Shimon","display_name":"Yosi Ben Shimon","email":"ybenshim@redhat.com","username":"ybenshim"},"change_message_id":"a779a0f0398b6949ad57e6c59248f6252e6232ba","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"47ab0dbc_9c052c1b","in_reply_to":"d6c593fa_4a4a6935","updated":"2024-06-17 11:31:46.000000000","message":"submitted a new BZ for it:\nhttps://bugs.launchpad.net/cinder/+bug/2069605","commit_id":"c4f384b0901f71e743b7f3d938a3dadc66d7a61d"},{"author":{"_account_id":35834,"name":"Yosi Ben Shimon","display_name":"Yosi Ben Shimon","email":"ybenshim@redhat.com","username":"ybenshim"},"change_message_id":"0c5b09f2617ab03b0cf408b284248ff003fb0243","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"41555ef1_3a043377","updated":"2025-01-13 10:02:26.000000000","message":"recheck\n\nunrelated issue in\ncinder-tempest-plugin-lvm-lio-barbican\ncinder-tempest-plugin-lvm-nvmet-barbican\ncinder-tempest-plugin-cbak-ceph","commit_id":"bca10db6f28ca31b256b75e336d48e92ffb1602f"}],"cinder_tempest_plugin/rbac/v3/test_volume_transfer.py":[{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"45704b857ece0eab6b2bd87ca85feed28b48ec23","unresolved":true,"context_lines":[{"line_number":32,"context_line":"    @classmethod"},{"line_number":33,"context_line":"    def resource_setup(cls):"},{"line_number":34,"context_line":"        super(RbacV3VolumeTransferTests, cls).resource_setup()"},{"line_number":35,"context_line":"        cls.volume_id \u003d cls.create_volume(client\u003dcls.admin_volume_client)"},{"line_number":36,"context_line":"        cls.volume_transfer \u003d cls.admin_transfer_client.create_volume_transfer("},{"line_number":37,"context_line":"            volume_id\u003dcls.volume_id"},{"line_number":38,"context_line":"        )[\u0027transfer\u0027]"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"        cls.addClassResourceCleanup("},{"line_number":41,"context_line":"            waiters.wait_for_volume_resource_status,"},{"line_number":42,"context_line":"            cls.admin_volume_client, cls.volume_id, \u0027available\u0027"},{"line_number":43,"context_line":"        )"},{"line_number":44,"context_line":"        cls.addClassResourceCleanup("},{"line_number":45,"context_line":"            cls.admin_transfer_client.delete_volume_transfer,"},{"line_number":46,"context_line":"            cls.volume_transfer[\u0027id\u0027]"},{"line_number":47,"context_line":"        )"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"    def _accept_volume_transfer(self, expected_status):"},{"line_number":50,"context_line":"        \"\"\"Accept volume transfer\"\"\""}],"source_content_type":"text/x-python","patch_set":1,"id":"db158504_98d0adbd","line":47,"range":{"start_line":35,"start_character":0,"end_line":47,"end_character":9},"updated":"2024-06-17 10:01:44.000000000","message":"I don\u0027t see a reason to place it here if only one test uses it.","commit_id":"c4f384b0901f71e743b7f3d938a3dadc66d7a61d"},{"author":{"_account_id":35834,"name":"Yosi Ben Shimon","display_name":"Yosi Ben Shimon","email":"ybenshim@redhat.com","username":"ybenshim"},"change_message_id":"a779a0f0398b6949ad57e6c59248f6252e6232ba","unresolved":true,"context_lines":[{"line_number":32,"context_line":"    @classmethod"},{"line_number":33,"context_line":"    def resource_setup(cls):"},{"line_number":34,"context_line":"        super(RbacV3VolumeTransferTests, cls).resource_setup()"},{"line_number":35,"context_line":"        cls.volume_id \u003d cls.create_volume(client\u003dcls.admin_volume_client)"},{"line_number":36,"context_line":"        cls.volume_transfer \u003d cls.admin_transfer_client.create_volume_transfer("},{"line_number":37,"context_line":"            volume_id\u003dcls.volume_id"},{"line_number":38,"context_line":"        )[\u0027transfer\u0027]"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"        cls.addClassResourceCleanup("},{"line_number":41,"context_line":"            waiters.wait_for_volume_resource_status,"},{"line_number":42,"context_line":"            cls.admin_volume_client, cls.volume_id, \u0027available\u0027"},{"line_number":43,"context_line":"        )"},{"line_number":44,"context_line":"        cls.addClassResourceCleanup("},{"line_number":45,"context_line":"            cls.admin_transfer_client.delete_volume_transfer,"},{"line_number":46,"context_line":"            cls.volume_transfer[\u0027id\u0027]"},{"line_number":47,"context_line":"        )"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"    def _accept_volume_transfer(self, expected_status):"},{"line_number":50,"context_line":"        \"\"\"Accept volume transfer\"\"\""}],"source_content_type":"text/x-python","patch_set":1,"id":"8c4437e0_25d206e6","line":47,"range":{"start_line":35,"start_character":0,"end_line":47,"end_character":9},"in_reply_to":"db158504_98d0adbd","updated":"2024-06-17 11:31:46.000000000","message":"There are 3 actually. list, list-with-detail and show","commit_id":"c4f384b0901f71e743b7f3d938a3dadc66d7a61d"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"45704b857ece0eab6b2bd87ca85feed28b48ec23","unresolved":true,"context_lines":[{"line_number":79,"context_line":""},{"line_number":80,"context_line":"        if expected_status !\u003d exceptions.Forbidden:"},{"line_number":81,"context_line":"            vol_transfer \u003d vol_transfer[\u0027transfer\u0027]"},{"line_number":82,"context_line":"            vol_transfer_show \u003d ("},{"line_number":83,"context_line":"                self.admin_transfer_client.show_volume_transfer("},{"line_number":84,"context_line":"                    vol_transfer[\u0027id\u0027]"},{"line_number":85,"context_line":"                )"},{"line_number":86,"context_line":"            )[\u0027transfer\u0027]"},{"line_number":87,"context_line":"            self.assertEqual(volume_id, vol_transfer_show[\u0027volume_id\u0027])"},{"line_number":88,"context_line":"            self.admin_transfer_client.delete_volume_transfer("},{"line_number":89,"context_line":"                transfer_id\u003dvol_transfer[\u0027id\u0027]"},{"line_number":90,"context_line":"            )"}],"source_content_type":"text/x-python","patch_set":1,"id":"849c3754_06a2d8d0","line":87,"range":{"start_line":82,"start_character":0,"end_line":87,"end_character":71},"updated":"2024-06-17 10:01:44.000000000","message":"This API call is unnecessary. \nYou should only check the permissions for the action","commit_id":"c4f384b0901f71e743b7f3d938a3dadc66d7a61d"},{"author":{"_account_id":35834,"name":"Yosi Ben Shimon","display_name":"Yosi Ben Shimon","email":"ybenshim@redhat.com","username":"ybenshim"},"change_message_id":"a779a0f0398b6949ad57e6c59248f6252e6232ba","unresolved":false,"context_lines":[{"line_number":79,"context_line":""},{"line_number":80,"context_line":"        if expected_status !\u003d exceptions.Forbidden:"},{"line_number":81,"context_line":"            vol_transfer \u003d vol_transfer[\u0027transfer\u0027]"},{"line_number":82,"context_line":"            vol_transfer_show \u003d ("},{"line_number":83,"context_line":"                self.admin_transfer_client.show_volume_transfer("},{"line_number":84,"context_line":"                    vol_transfer[\u0027id\u0027]"},{"line_number":85,"context_line":"                )"},{"line_number":86,"context_line":"            )[\u0027transfer\u0027]"},{"line_number":87,"context_line":"            self.assertEqual(volume_id, vol_transfer_show[\u0027volume_id\u0027])"},{"line_number":88,"context_line":"            self.admin_transfer_client.delete_volume_transfer("},{"line_number":89,"context_line":"                transfer_id\u003dvol_transfer[\u0027id\u0027]"},{"line_number":90,"context_line":"            )"}],"source_content_type":"text/x-python","patch_set":1,"id":"ef103572_1af90165","line":87,"range":{"start_line":82,"start_character":0,"end_line":87,"end_character":71},"in_reply_to":"849c3754_06a2d8d0","updated":"2024-06-17 11:31:46.000000000","message":"Acknowledged","commit_id":"c4f384b0901f71e743b7f3d938a3dadc66d7a61d"}]}