)]}'
{"patrole_tempest_plugin/tests/api/image/v2/test_image_namespace_rbac.py":[{"author":{"_account_id":23185,"name":"Samantha Blanco","email":"samantha.blanco@att.com","username":"sblanco"},"change_message_id":"5e7b80b8ac6a991ec91c0cd7850d362d66aaabf2","unresolved":false,"context_lines":[{"line_number":32,"context_line":"        super(ImageNamespacesRbacTest, self).tearDown()"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"    def _cleanup_namespace(self, namespace_name):"},{"line_number":35,"context_line":"        body \u003d self.namespaces_client.show_namespace(namespace_name)"},{"line_number":36,"context_line":"        self.assertEqual(namespace_name, body[\u0027namespace\u0027])"},{"line_number":37,"context_line":"        body \u003d self.namespaces_client.update_namespace("},{"line_number":38,"context_line":"            namespace\u003dnamespace_name,"},{"line_number":39,"context_line":"            description\u003d\u0027Tempest\u0027,"},{"line_number":40,"context_line":"            visibility\u003d\u0027private\u0027,"},{"line_number":41,"context_line":"            display_name\u003dnamespace_name,"},{"line_number":42,"context_line":"            protected\u003dFalse)"},{"line_number":43,"context_line":"        self.namespaces_client.delete_namespace(namespace_name)"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"    @rbac_rule_validation.action(service\u003d\"glance\","}],"source_content_type":"text/x-python","patch_set":2,"id":"3a461143_531d730e","line":42,"range":{"start_line":35,"start_character":0,"end_line":42,"end_character":28},"updated":"2017-01-25 15:37:52.000000000","message":"Why is all of this in a cleanup?","commit_id":"65be429d1b492e09ea94714af53ad3bef3c95809"},{"author":{"_account_id":23184,"name":"DavidPurcell","email":"d.purcell222@gmail.com","username":"DavidPurcell"},"change_message_id":"7efc95cea4abc9f1fc6b147c5b6becc6a8e16654","unresolved":false,"context_lines":[{"line_number":32,"context_line":"        super(ImageNamespacesRbacTest, self).tearDown()"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"    def _cleanup_namespace(self, namespace_name):"},{"line_number":35,"context_line":"        body \u003d self.namespaces_client.show_namespace(namespace_name)"},{"line_number":36,"context_line":"        self.assertEqual(namespace_name, body[\u0027namespace\u0027])"},{"line_number":37,"context_line":"        body \u003d self.namespaces_client.update_namespace("},{"line_number":38,"context_line":"            namespace\u003dnamespace_name,"},{"line_number":39,"context_line":"            description\u003d\u0027Tempest\u0027,"},{"line_number":40,"context_line":"            visibility\u003d\u0027private\u0027,"},{"line_number":41,"context_line":"            display_name\u003dnamespace_name,"},{"line_number":42,"context_line":"            protected\u003dFalse)"},{"line_number":43,"context_line":"        self.namespaces_client.delete_namespace(namespace_name)"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"    @rbac_rule_validation.action(service\u003d\"glance\","}],"source_content_type":"text/x-python","patch_set":2,"id":"3a461143_964b49b0","line":42,"range":{"start_line":35,"start_character":0,"end_line":42,"end_character":28},"in_reply_to":"3a461143_531d730e","updated":"2017-01-25 16:03:35.000000000","message":"Trying to directly delete namespaces can results in errors if they are not properly configured.  It is a bit excessive (the assert could probably be removed).  Based on the _cleanup_namespace from the corresponding Tempest api test.  https://github.com/openstack/tempest/blob/master/tempest/api/image/v2/test_images_metadefs_namespaces.py","commit_id":"65be429d1b492e09ea94714af53ad3bef3c95809"},{"author":{"_account_id":23185,"name":"Samantha Blanco","email":"samantha.blanco@att.com","username":"sblanco"},"change_message_id":"e8e20e82abfef79bff09c21baf834aeff7ec536f","unresolved":false,"context_lines":[{"line_number":31,"context_line":"        rbac_utils.switch_role(self, switchToRbacRole\u003dFalse)"},{"line_number":32,"context_line":"        super(ImageNamespacesRbacTest, self).tearDown()"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"    def _cleanup_namespace(self, namespace_name):"},{"line_number":35,"context_line":"        self.namespaces_client.delete_namespace(namespace_name)"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"    @rbac_rule_validation.action(service\u003d\"glance\","},{"line_number":38,"context_line":"                                 rule\u003d\"add_metadef_namespace\")"}],"source_content_type":"text/x-python","patch_set":3,"id":"3a461143_7ef6afd8","line":35,"range":{"start_line":34,"start_character":0,"end_line":35,"end_character":63},"updated":"2017-01-25 18:14:17.000000000","message":"If this is all that\u0027s in cleanup, do we really need this as a separate function?","commit_id":"b6d9e30aaf30de38e4647d9d50cf972439c99699"}]}