)]}'
{"manila_tempest_tests/tests/api/admin/test_quotas_negative.py":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"212e886112f89fd3ea25ada83a928666553f00b3","unresolved":false,"context_lines":[{"line_number":33,"context_line":"@ddt.ddt"},{"line_number":34,"context_line":"class SharesAdminQuotasNegativeTest(base.BaseSharesAdminTest):"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"    force_tenant_isolation \u003d True"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"    @classmethod"},{"line_number":39,"context_line":"    def resource_setup(cls):"}],"source_content_type":"text/x-python","patch_set":1,"id":"bf51134e_ee1152ae","line":36,"range":{"start_line":36,"start_character":4,"end_line":36,"end_character":33},"updated":"2020-06-19 23:13:54.000000000","message":"It appears that when this is set to True, we\u0027re going to get brand new creds - so you need to skip the test class. \n\nOrtherwise:        self.get_client_with_isolated_creds() will only get you a client for a pre-existing tenant if dynamic credentials is set to False. So we don\u0027t need to skip test cases where \"force_tenant_isolation\" isn\u0027t True.","commit_id":"a38cde6a8327617bde1b7b969432ec59a2845c70"},{"author":{"_account_id":18058,"name":"Lucio Seki","email":"lseki@redhat.com","username":"lseki"},"change_message_id":"47d34a9b964820f71109636b8eddb51b6498f235","unresolved":false,"context_lines":[{"line_number":33,"context_line":"@ddt.ddt"},{"line_number":34,"context_line":"class SharesAdminQuotasNegativeTest(base.BaseSharesAdminTest):"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"    force_tenant_isolation \u003d True"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"    @classmethod"},{"line_number":39,"context_line":"    def resource_setup(cls):"}],"source_content_type":"text/x-python","patch_set":1,"id":"bf51134e_a48db932","line":36,"range":{"start_line":36,"start_character":4,"end_line":36,"end_character":33},"in_reply_to":"bf51134e_ee1152ae","updated":"2020-06-20 00:03:26.000000000","message":"I see... So this [0] might be where all the magic happens.\n\n[0] https://github.com/openstack/tempest/blob/b5fcce38124ff4bd6eb3a654101d5edd9a40f661/tempest/common/credentials_factory.py#L134_#L147","commit_id":"a38cde6a8327617bde1b7b969432ec59a2845c70"}],"manila_tempest_tests/tests/api/admin/test_share_networks.py":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"212e886112f89fd3ea25ada83a928666553f00b3","unresolved":false,"context_lines":[{"line_number":26,"context_line":"        base.BaseSharesAdminTest,"},{"line_number":27,"context_line":"        test_share_networks.ShareNetworkListMixin):"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"    @classmethod"},{"line_number":30,"context_line":"    def skip_checks(cls):"},{"line_number":31,"context_line":"        super(ShareNetworkAdminTest, cls).skip_checks()"},{"line_number":32,"context_line":"        if not CONF.auth.use_dynamic_credentials:"},{"line_number":33,"context_line":"            raise cls.skipException(\u0027Dynamic credentials are disabled\u0027)"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    @classmethod"},{"line_number":36,"context_line":"    def resource_setup(cls):"}],"source_content_type":"text/x-python","patch_set":1,"id":"bf51134e_4e283ee1","line":33,"range":{"start_line":29,"start_character":16,"end_line":33,"end_character":71},"updated":"2020-06-19 23:13:54.000000000","message":"No need to skip this, \"force_tenant_isolation\" isn\u0027t set to True","commit_id":"a38cde6a8327617bde1b7b969432ec59a2845c70"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"1aecced2219fa7bd499077741007a92723f4386d","unresolved":false,"context_lines":[{"line_number":65,"context_line":"            ss_type\u003d\u0027kerberos\u0027,"},{"line_number":66,"context_line":"            **cls.data_sn_with_ldap_ss)"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"        cls.sn_with_kerberos_ss \u003d cls.alt_shares_v2_client.create_share_network("},{"line_number":69,"context_line":"            cleanup_in_class\u003dTrue,"},{"line_number":70,"context_line":"            **cls.data_sn_with_kerberos_ss)"},{"line_number":71,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"bf51134e_5f50e6d4","line":68,"updated":"2020-06-20 02:10:41.000000000","message":"pep8: E501 line too long (80 \u003e 79 characters)","commit_id":"1e6d7ad2c72948f4d6fc86e6e303f0f4e8bb4f4c"}],"manila_tempest_tests/tests/api/admin/test_user_messages_negative.py":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"212e886112f89fd3ea25ada83a928666553f00b3","unresolved":false,"context_lines":[{"line_number":36,"context_line":"        super(UserMessageNegativeTest, self).setUp()"},{"line_number":37,"context_line":"        self.message \u003d self.create_user_message()"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"    @decorators.attr(type\u003d[base.TAG_NEGATIVE, base.TAG_API])"},{"line_number":40,"context_line":"    @testtools.skipUnless("},{"line_number":41,"context_line":"        CONF.auth.use_dynamic_credentials,"},{"line_number":42,"context_line":"        \u0027Dynamic credentials are disabled\u0027)"},{"line_number":43,"context_line":"    def test_show_message_of_other_tenants(self):"},{"line_number":44,"context_line":"        isolated_client \u003d self.get_client_with_isolated_creds("},{"line_number":45,"context_line":"            type_of_creds\u003d\u0027alt\u0027, client_version\u003d\u00272\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"bf51134e_ce79eefc","line":42,"range":{"start_line":39,"start_character":60,"end_line":42,"end_character":43},"updated":"2020-06-19 23:13:54.000000000","message":"No need to skip this and the test case below, \"force_tenant_isloation\" isn\u0027t set","commit_id":"a38cde6a8327617bde1b7b969432ec59a2845c70"}],"manila_tempest_tests/tests/api/test_shares_actions.py":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"212e886112f89fd3ea25ada83a928666553f00b3","unresolved":false,"context_lines":[{"line_number":403,"context_line":"        self.assertGreater(shares[\"count\"], 0)"},{"line_number":404,"context_line":""},{"line_number":405,"context_line":"    @tc.attr(base.TAG_POSITIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":406,"context_line":"    @testtools.skipUnless("},{"line_number":407,"context_line":"        CONF.auth.use_dynamic_credentials,"},{"line_number":408,"context_line":"        \u0027Dynamic credentials are disabled\u0027)"},{"line_number":409,"context_line":"    def test_list_shares_public_with_detail(self):"},{"line_number":410,"context_line":"        public_share \u003d self.create_share("},{"line_number":411,"context_line":"            name\u003d\u0027public_share\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"bf51134e_ae7c7aeb","line":408,"range":{"start_line":406,"start_character":0,"end_line":408,"end_character":43},"updated":"2020-06-19 23:13:54.000000000","message":"No need to skip this","commit_id":"a38cde6a8327617bde1b7b969432ec59a2845c70"}],"manila_tempest_tests/tests/api/test_shares_negative.py":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"212e886112f89fd3ea25ada83a928666553f00b3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"bf51134e_8e77f602","updated":"2020-06-19 23:13:54.000000000","message":"No need to skip tests in this class if dynamic creds aren\u0027t turned on.","commit_id":"a38cde6a8327617bde1b7b969432ec59a2845c70"}]}