)]}'
{"tempest/scenario/test_server_multinode.py":[{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"909395480819f0fa3cdb2227e4793bad87b96082","unresolved":false,"context_lines":[{"line_number":38,"context_line":"    def setup_clients(cls):"},{"line_number":39,"context_line":"        super(TestServerMultinode, cls).setup_clients()"},{"line_number":40,"context_line":"        # Use admin client by default"},{"line_number":41,"context_line":"        cls.manager \u003d cls.admin_manager"},{"line_number":42,"context_line":"        # this is needed so that we can use the availability_zone:host"},{"line_number":43,"context_line":"        # scheduler hint, which is admin_only by default"},{"line_number":44,"context_line":"        cls.servers_client \u003d cls.admin_manager.servers_client"}],"source_content_type":"text/x-python","patch_set":2,"id":"da85f559_4aec6e8f","line":41,"updated":"2015-11-13 14:52:36.000000000","message":"You don\u0027t really need to do this, actually, you could leave L51 below as it was, and drop L40/41.","commit_id":"9b115183f45fbcbb2544711dd4c66c21ba558f12"},{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"909395480819f0fa3cdb2227e4793bad87b96082","unresolved":false,"context_lines":[{"line_number":41,"context_line":"        cls.manager \u003d cls.admin_manager"},{"line_number":42,"context_line":"        # this is needed so that we can use the availability_zone:host"},{"line_number":43,"context_line":"        # scheduler hint, which is admin_only by default"},{"line_number":44,"context_line":"        cls.servers_client \u003d cls.admin_manager.servers_client"},{"line_number":45,"context_line":"        super(TestServerMultinode, cls).resource_setup()"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"    @test.idempotent_id(\u00279cecbe35-b9d4-48da-a37e-7ce70aa43d30\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"da85f559_2a59ea8e","line":44,"updated":"2015-11-13 14:52:36.000000000","message":"note to reviewer: setting the manager is not enough because the create_server helper relies on self.servers_client - which is pretty bad as it does not allow for using a different tenant unless cls.servers_client is overwritten like this. This has to be fixed in scenarios, but until then this is the only way to boot a server as admin.","commit_id":"9b115183f45fbcbb2544711dd4c66c21ba558f12"}]}