)]}'
{"patrole_tempest_plugin/tests/api/volume/test_volumes_backup_rbac.py":[{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"fdd7223b44ea68db59cd71b51c42a45b47f67175","unresolved":false,"context_lines":[{"line_number":100,"context_line":"            volume_id\u003dself.volume[\u0027id\u0027])[\u0027backup\u0027]"},{"line_number":101,"context_line":"        self.addCleanup(test_utils.call_and_ignore_notfound_exc,"},{"line_number":102,"context_line":"                        self.backups_client.delete_backup, backup[\u0027id\u0027])"},{"line_number":103,"context_line":"        waiters.wait_for_volume_resource_status(self.backups_client,"},{"line_number":104,"context_line":"                                                backup[\u0027id\u0027], \u0027available\u0027)"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"        self.rbac_utils.switch_role(self, toggle_rbac_role\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7f231b9d_808a5abc","line":103,"range":{"start_line":103,"start_character":53,"end_line":103,"end_character":67},"updated":"2017-06-14 16:24:26.000000000","message":"nit: better if it\u0027s os_admin.backups_client\n\nI believe we should add a hacking rule around this in a separate commit because I\u0027m sure there are plenty of offending areas where os_primary creds are used for show.","commit_id":"9cc8d6f4a5caa06286538f41b36b78a5dab9721d"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"fcc1c1f4c01a960acd6e19118f467465dbd2843e","unresolved":false,"context_lines":[{"line_number":100,"context_line":"            volume_id\u003dself.volume[\u0027id\u0027])[\u0027backup\u0027]"},{"line_number":101,"context_line":"        self.addCleanup(test_utils.call_and_ignore_notfound_exc,"},{"line_number":102,"context_line":"                        self.backups_client.delete_backup, backup[\u0027id\u0027])"},{"line_number":103,"context_line":"        waiters.wait_for_volume_resource_status(self.os_admin.backups_client,"},{"line_number":104,"context_line":"                                                backup[\u0027id\u0027], \u0027available\u0027)"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"        self.rbac_utils.switch_role(self, toggle_rbac_role\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":3,"id":"7f231b9d_16988672","line":103,"range":{"start_line":103,"start_character":53,"end_line":103,"end_character":61},"updated":"2017-06-16 21:31:22.000000000","message":"This could be os_primary, but doesn\u0027t matter.","commit_id":"93dae2a4487e67473d57420ebbde3afd4c142d84"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"fcc1c1f4c01a960acd6e19118f467465dbd2843e","unresolved":false,"context_lines":[{"line_number":106,"context_line":"        self.rbac_utils.switch_role(self, toggle_rbac_role\u003dTrue)"},{"line_number":107,"context_line":"        self.backups_client.delete_backup(backup[\u0027id\u0027])"},{"line_number":108,"context_line":"        # Wait for deletion so error isn\u0027t thrown during clean up."},{"line_number":109,"context_line":"        self.backups_client.wait_for_resource_deletion(backup[\u0027id\u0027])"},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"    @test.attr(type\u003d[\"slow\"])"},{"line_number":112,"context_line":"    @rbac_rule_validation.action(service\u003d\"cinder\","}],"source_content_type":"text/x-python","patch_set":3,"id":"7f231b9d_56927e58","line":109,"range":{"start_line":109,"start_character":13,"end_line":109,"end_character":27},"updated":"2017-06-16 21:31:22.000000000","message":"This sort of thing needs to os_admin.backups_client","commit_id":"93dae2a4487e67473d57420ebbde3afd4c142d84"}]}