)]}'
{"tempest/scenario/test_security_groups_basic_ops.py":[{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"afbf48932b78dc14a1e175e8cfc6c1442c740154","unresolved":false,"context_lines":[{"line_number":130,"context_line":"            cls.enabled \u003d False"},{"line_number":131,"context_line":"            raise cls.skipException(msg)"},{"line_number":132,"context_line":"        super(TestSecurityGroupsBasicOps, cls).check_preconditions()"},{"line_number":133,"context_line":"        if cls.check_alt_tenant_available():"},{"line_number":134,"context_line":"            msg \u003d \u0027No secondary tenant defined\u0027"},{"line_number":135,"context_line":"            cls.enabled \u003d False"},{"line_number":136,"context_line":"            raise cls.skipException(msg)"}],"source_content_type":"text/x-python","patch_set":2,"id":"9aa7fdbe_d9d64cc5","line":133,"updated":"2014-10-20 16:59:45.000000000","message":"Why not just use is_multi_user(). That checks to see if there are more than 1 unique user. Granted that\u0027s not a perfect fit for what you\u0027re checking here, (ie it\u0027s possible for a user to be provided for the same project) but the usage is similar, and will cover all the cred provider use cases. Given that for test accounts it won\u0027t work if users scoped at the same tenant are provided this will work.\n\nI think a better alternative though is that you add a patch before this one and implement is_multi_tenant() method in a similar manner to is_multi_user() and use that here.","commit_id":"57c3e3152a92dcd5f19a3bb1f8e33c741a5df11c"},{"author":{"_account_id":12412,"name":"Shang Yong","email":"yong.shang@me.com","username":"bshang"},"change_message_id":"ca324a97580beba087b64ef2455037dab05dcbd2","unresolved":false,"context_lines":[{"line_number":140,"context_line":"        # of storing them as collections on the class."},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"        # get credentials for secondary tenant"},{"line_number":143,"context_line":"        cls.alt_creds \u003d cls.alt_credentials()"},{"line_number":144,"context_line":"        cls.alt_manager \u003d clients.Manager(cls.alt_creds)"},{"line_number":145,"context_line":"        # Credentials from the manager are filled with both IDs and Names"},{"line_number":146,"context_line":"        cls.alt_creds \u003d cls.alt_manager.credentials"}],"source_content_type":"text/x-python","patch_set":4,"id":"9aa7fdbe_ab5d4417","line":143,"updated":"2014-10-23 14:10:39.000000000","message":"It\u0027s better to call cls.isolated_creds.get_alt_creds() to keep consistency with cls.isolated_creds.is_multi_tenant().","commit_id":"4d5efb37416e1ec495f574038bf361771c4dd37a"},{"author":{"_account_id":8576,"name":"Yair Fried","email":"yfried@redhat.com","username":"yfried"},"change_message_id":"d7a50eb37968835bebf6778f1e61862f08b0cd86","unresolved":false,"context_lines":[{"line_number":140,"context_line":"        # of storing them as collections on the class."},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"        # get credentials for secondary tenant"},{"line_number":143,"context_line":"        cls.alt_creds \u003d cls.alt_credentials()"},{"line_number":144,"context_line":"        cls.alt_manager \u003d clients.Manager(cls.alt_creds)"},{"line_number":145,"context_line":"        # Credentials from the manager are filled with both IDs and Names"},{"line_number":146,"context_line":"        cls.alt_creds \u003d cls.alt_manager.credentials"}],"source_content_type":"text/x-python","patch_set":4,"id":"9aa7fdbe_85e4bbe1","line":143,"in_reply_to":"9aa7fdbe_ab5d4417","updated":"2014-10-26 08:02:15.000000000","message":"Done","commit_id":"4d5efb37416e1ec495f574038bf361771c4dd37a"}]}