)]}'
{"manila_tempest_tests/tests/api/base.py":[{"author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"change_message_id":"d484b81a0d0b14e6dfeacabf32756ad2a5bcedbc","unresolved":false,"context_lines":[{"line_number":267,"context_line":"    @classmethod"},{"line_number":268,"context_line":"    def setup_clients(cls):"},{"line_number":269,"context_line":"        super(BaseSharesTest, cls).setup_clients()"},{"line_number":270,"context_line":"        os \u003d getattr(cls, \u0027os_%s\u0027 % cls.credentials[0])"},{"line_number":271,"context_line":"        # Initialise share clients for test credentials"},{"line_number":272,"context_line":"        cls.shares_client \u003d os.share_v1.SharesClient()"},{"line_number":273,"context_line":"        cls.shares_v2_client \u003d os.share_v2.SharesV2Client()"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f6a8fd7_856b0aef","line":270,"range":{"start_line":270,"start_character":8,"end_line":270,"end_character":55},"updated":"2018-04-25 11:18:49.000000000","message":"Actually this needs to change, right?","commit_id":"8f4e49f20ef711fd0b3066f8899bde2c3fc78752"},{"author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"change_message_id":"7b88b9fe0638f3de2454642d83731b896b0b8a8a","unresolved":false,"context_lines":[{"line_number":280,"context_line":"            cls.subnets_client \u003d None"},{"line_number":281,"context_line":""},{"line_number":282,"context_line":"        if CONF.identity.auth_version \u003d\u003d \u0027v3\u0027:"},{"line_number":283,"context_line":"            project_id \u003d os_primary.auth_provider.auth_data[1][\u0027project\u0027][\u0027id\u0027]"},{"line_number":284,"context_line":"        else:"},{"line_number":285,"context_line":"            project_id \u003d os_primary.auth_provider.auth_data[1][\u0027token\u0027][\u0027tenant\u0027][\u0027id\u0027]"},{"line_number":286,"context_line":"        cls.tenant_id \u003d project_id"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f6a8fd7_8975c0cd","line":283,"range":{"start_line":283,"start_character":25,"end_line":283,"end_character":36},"updated":"2018-04-20 11:54:12.000000000","message":"Need to import this.","commit_id":"8f4e49f20ef711fd0b3066f8899bde2c3fc78752"},{"author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"change_message_id":"aaf995171c02044bbe1978b526c313a71dc317ba","unresolved":false,"context_lines":[{"line_number":280,"context_line":"            cls.subnets_client \u003d None"},{"line_number":281,"context_line":""},{"line_number":282,"context_line":"        if CONF.identity.auth_version \u003d\u003d \u0027v3\u0027:"},{"line_number":283,"context_line":"            project_id \u003d os_primary.auth_provider.auth_data[1][\u0027project\u0027][\u0027id\u0027]"},{"line_number":284,"context_line":"        else:"},{"line_number":285,"context_line":"            project_id \u003d os_primary.auth_provider.auth_data[1][\u0027token\u0027][\u0027tenant\u0027][\u0027id\u0027]"},{"line_number":286,"context_line":"        cls.tenant_id \u003d project_id"}],"source_content_type":"text/x-python","patch_set":2,"id":"5f7c97a3_f3c6a316","line":283,"range":{"start_line":283,"start_character":25,"end_line":283,"end_character":36},"in_reply_to":"9f6a8fd7_8975c0cd","updated":"2018-06-09 12:55:41.000000000","message":"So \u0027os\u0027 here was from line 270, where it is supposed to be set from os_primary, os_admin, etc. depending on the value of cls.credentials[0].\n\nDo you agree that this is different than the sort of \u0027os\u0027 use that is being deprecated?","commit_id":"8f4e49f20ef711fd0b3066f8899bde2c3fc78752"}]}