)]}'
{"tempest/api/object_storage/test_container_acl_negative.py":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"95bc30bad157c46f851e039df88e6f46db3722ed","unresolved":false,"context_lines":[{"line_number":42,"context_line":"        self.container_client.update_container(self.container_name)"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"    def tearDown(self):"},{"line_number":45,"context_line":"        base.delete_containers([self.container_name],"},{"line_number":46,"context_line":"                               self.container_client, self.object_client)"},{"line_number":47,"context_line":"        super(ObjectACLsNegativeTest, self).tearDown()"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"    @decorators.attr(type\u003d[\u0027negative\u0027])"}],"source_content_type":"text/x-python","patch_set":2,"id":"5f7c97a3_4650405f","line":46,"range":{"start_line":45,"start_character":0,"end_line":46,"end_character":73},"updated":"2018-05-20 23:06:57.000000000","message":"did not get the reason. self.delete_containers is just a wrapper of base.delete_containers by passing the same clients.","commit_id":"f49fe7034bd861cca8b81a76b2e21da9b35dd64a"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"66773fcffa27f6de915da1a9e06459788c00077a","unresolved":false,"context_lines":[{"line_number":42,"context_line":"        self.container_client.update_container(self.container_name)"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"    def tearDown(self):"},{"line_number":45,"context_line":"        base.delete_containers([self.container_name],"},{"line_number":46,"context_line":"                               self.container_client, self.object_client)"},{"line_number":47,"context_line":"        super(ObjectACLsNegativeTest, self).tearDown()"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"    @decorators.attr(type\u003d[\u0027negative\u0027])"}],"source_content_type":"text/x-python","patch_set":2,"id":"5f7c97a3_313da42b","line":46,"range":{"start_line":45,"start_character":0,"end_line":46,"end_character":73},"in_reply_to":"5f7c97a3_4650405f","updated":"2018-07-09 18:42:40.000000000","message":"Yeah, Ghanshyam is right here.\nself.delete_containers() calls base.delete_containers() like\n\ntempest/api/object_storage/base.py\n 133     @classmethod\n 134     def delete_containers(cls, container_client\u003dNone, object_client\u003dNone):\n 135         if container_client is None:\n 136             container_client \u003d cls.container_client\n 137         if object_client is None:\n 138             object_client \u003d cls.object_client\n 139         delete_containers(cls.containers, container_client, object_client)\n\nSo this patch doesn\u0027t make any changes actually.","commit_id":"f49fe7034bd861cca8b81a76b2e21da9b35dd64a"}]}