)]}'
{"tempest/api/compute/admin/test_live_migration.py":[{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"d88ccf8ca518dc5a544dafa54aa3e3d3a89f663b","unresolved":false,"context_lines":[{"line_number":39,"context_line":""},{"line_number":40,"context_line":"        cls.created_server_ids \u003d []"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"    def _get_compute_hostnames(self, host\u003dNone):"},{"line_number":43,"context_line":"        body \u003d self.admin_aggregates_client.list_aggregates()"},{"line_number":44,"context_line":"        # Test aggregates first."},{"line_number":45,"context_line":"        # If aggregates are enabled we should look there."}],"source_content_type":"text/x-python","patch_set":2,"id":"7a016987_50a5ab9f","line":42,"range":{"start_line":42,"start_character":15,"end_line":42,"end_character":26},"updated":"2015-05-29 09:55:50.000000000","message":"You should change the method name. It used to return all the compute hosts, it\u0027s not true anymore. I suggest \u0027get_compute_hosts_in_same_aggregate\u0027","commit_id":"92aa3684ba6dac53246cc2611e94cf43dc0c646f"},{"author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"change_message_id":"220f8742c52d1e953e8d4ec80e12428c0c4db7cd","unresolved":false,"context_lines":[{"line_number":42,"context_line":"    def _get_compute_hosts_in_same_aggregate(self, host\u003dNone):"},{"line_number":43,"context_line":"        body \u003d self.admin_aggregates_client.list_aggregates()"},{"line_number":44,"context_line":"        # Test aggregates first."},{"line_number":45,"context_line":"        # If aggregates are enabled we should look there."},{"line_number":46,"context_line":"        if body:"},{"line_number":47,"context_line":"            for aggregate in body:"},{"line_number":48,"context_line":"                if host in aggregate[\u0027hosts\u0027] and len(aggregate[\u0027hosts\u0027]) \u003e 1:"}],"source_content_type":"text/x-python","patch_set":3,"id":"3afb71cf_71f55f55","line":45,"updated":"2015-06-09 11:20:28.000000000","message":"It would be nice to better document the gotcha about migrating between incompatible hosts and the assumption we are making here.","commit_id":"ac77330ce4180ce28ae3514032813c3e7e365ff0"},{"author":{"_account_id":7872,"name":"Marc Koderer","email":"marc@koderer.com","username":"mkoderer"},"change_message_id":"be9fe9236768ca91dcbb6b78fd6fa73ce12fd8f4","unresolved":false,"context_lines":[{"line_number":52,"context_line":"                # Otherwise skip the test."},{"line_number":53,"context_line":"                if host in aggregate[\u0027hosts\u0027]:"},{"line_number":54,"context_line":"                    if len(aggregate[\u0027hosts\u0027]) \u003e 1:"},{"line_number":55,"context_line":"                        return aggregate[\u0027hosts\u0027]"},{"line_number":56,"context_line":"                    else:"},{"line_number":57,"context_line":"                        raise self.skipTest("},{"line_number":58,"context_line":"                            \"Less than 2 compute nodes,\""}],"source_content_type":"text/x-python","patch_set":6,"id":"ba3cc151_2a42e368","line":55,"updated":"2015-07-01 13:03:26.000000000","message":"Does this really solve your problem? A host can be member of several host aggregates/AZs and you are return the first matching with more than 1 member. For me this sound more like a workaround than a real fix.","commit_id":"40e3f11d166e5ac88ffa7cb3f39a640de52d2a4e"}]}