)]}'
{"tempest/api/compute/admin/test_live_migration.py":[{"author":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"change_message_id":"55821601b6c1adc168896e818c9500dd01157fec","unresolved":false,"context_lines":[{"line_number":92,"context_line":"    max_microversion \u003d \u00272.24\u0027"},{"line_number":93,"context_line":"    block_migration \u003d None"},{"line_number":94,"context_line":""},{"line_number":95,"context_line":"    def _test_live_migration(self, state\u003d\u0027ACTIVE\u0027, volume_backed\u003dFalse,config_drive\u003dFalse):"},{"line_number":96,"context_line":"        \"\"\"Tests live migration between two hosts."},{"line_number":97,"context_line":""},{"line_number":98,"context_line":"        Requires CONF.compute_feature_enabled.live_migration to be True."}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_4c10c159","line":95,"updated":"2019-11-04 21:17:00.000000000","message":"whitespace needed","commit_id":"ff52d41572e6c24e1a674ff59bc71546e856916e"},{"author":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"change_message_id":"55821601b6c1adc168896e818c9500dd01157fec","unresolved":false,"context_lines":[{"line_number":102,"context_line":"                      and \u0027PAUSED\u0027."},{"line_number":103,"context_line":"        :param volume_backed: If the instance is volume backed or not. If"},{"line_number":104,"context_line":"                              volume_backed, *block* migration is not used."},{"line_number":105,"context_line":"        : param config_drive: If config_drive is enabled , it will store meta data"},{"line_number":106,"context_line":"                                on a configuration drive (iso9660-default, if you want to use vfat "},{"line_number":107,"context_line":"                                as driver_type then you can explicilty mention that in /etc/nova.conf) "},{"line_number":108,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_a702ae07","line":105,"range":{"start_line":105,"start_character":9,"end_line":105,"end_character":10},"updated":"2019-11-04 21:17:00.000000000","message":"nit: no space needed","commit_id":"ff52d41572e6c24e1a674ff59bc71546e856916e"},{"author":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"change_message_id":"55821601b6c1adc168896e818c9500dd01157fec","unresolved":false,"context_lines":[{"line_number":104,"context_line":"                              volume_backed, *block* migration is not used."},{"line_number":105,"context_line":"        : param config_drive: If config_drive is enabled , it will store meta data"},{"line_number":106,"context_line":"                                on a configuration drive (iso9660-default, if you want to use vfat "},{"line_number":107,"context_line":"                                as driver_type then you can explicilty mention that in /etc/nova.conf) "},{"line_number":108,"context_line":"        \"\"\""},{"line_number":109,"context_line":"        # Live migrate an instance to another host"},{"line_number":110,"context_line":"        server_id \u003d self.create_test_server(wait_until\u003d\"ACTIVE\","}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_87e9b21a","line":107,"range":{"start_line":107,"start_character":87,"end_line":107,"end_character":102},"updated":"2019-11-04 21:17:00.000000000","message":"move it to a next line, max chars reached","commit_id":"ff52d41572e6c24e1a674ff59bc71546e856916e"},{"author":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"change_message_id":"55821601b6c1adc168896e818c9500dd01157fec","unresolved":false,"context_lines":[{"line_number":103,"context_line":"        :param volume_backed: If the instance is volume backed or not. If"},{"line_number":104,"context_line":"                              volume_backed, *block* migration is not used."},{"line_number":105,"context_line":"        : param config_drive: If config_drive is enabled , it will store meta data"},{"line_number":106,"context_line":"                                on a configuration drive (iso9660-default, if you want to use vfat "},{"line_number":107,"context_line":"                                as driver_type then you can explicilty mention that in /etc/nova.conf) "},{"line_number":108,"context_line":"        \"\"\""},{"line_number":109,"context_line":"        # Live migrate an instance to another host"},{"line_number":110,"context_line":"        server_id \u003d self.create_test_server(wait_until\u003d\"ACTIVE\","}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_c7180a77","line":107,"range":{"start_line":106,"start_character":31,"end_line":107,"end_character":34},"updated":"2019-11-04 21:17:00.000000000","message":"nit: indent according to above line","commit_id":"ff52d41572e6c24e1a674ff59bc71546e856916e"},{"author":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"change_message_id":"55821601b6c1adc168896e818c9500dd01157fec","unresolved":false,"context_lines":[{"line_number":104,"context_line":"                              volume_backed, *block* migration is not used."},{"line_number":105,"context_line":"        : param config_drive: If config_drive is enabled , it will store meta data"},{"line_number":106,"context_line":"                                on a configuration drive (iso9660-default, if you want to use vfat "},{"line_number":107,"context_line":"                                as driver_type then you can explicilty mention that in /etc/nova.conf) "},{"line_number":108,"context_line":"        \"\"\""},{"line_number":109,"context_line":"        # Live migrate an instance to another host"},{"line_number":110,"context_line":"        server_id \u003d self.create_test_server(wait_until\u003d\"ACTIVE\","}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_e788a66b","line":107,"range":{"start_line":107,"start_character":102,"end_line":107,"end_character":103},"updated":"2019-11-04 21:17:00.000000000","message":"remove whitespace","commit_id":"ff52d41572e6c24e1a674ff59bc71546e856916e"},{"author":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"change_message_id":"55821601b6c1adc168896e818c9500dd01157fec","unresolved":false,"context_lines":[{"line_number":108,"context_line":"        \"\"\""},{"line_number":109,"context_line":"        # Live migrate an instance to another host"},{"line_number":110,"context_line":"        server_id \u003d self.create_test_server(wait_until\u003d\"ACTIVE\","},{"line_number":111,"context_line":"                                            volume_backed\u003dvolume_backed,config_drive\u003dconfig_drive)[\u0027id\u0027]"},{"line_number":112,"context_line":"        source_host \u003d self.get_host_for_server(server_id)"},{"line_number":113,"context_line":"        destination_host \u003d self.get_host_other_than(server_id)"},{"line_number":114,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_67f27607","line":111,"range":{"start_line":111,"start_character":72,"end_line":111,"end_character":104},"updated":"2019-11-04 21:17:00.000000000","message":"move it to the next line, max chars reached","commit_id":"ff52d41572e6c24e1a674ff59bc71546e856916e"},{"author":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"change_message_id":"55821601b6c1adc168896e818c9500dd01157fec","unresolved":false,"context_lines":[{"line_number":126,"context_line":"            # ways, so live migrate VM back to the source host"},{"line_number":127,"context_line":"            LOG.info(\"Live migrate back to source %s\", source_host)"},{"line_number":128,"context_line":"            self._live_migrate(server_id, source_host, state, volume_backed)"},{"line_number":129,"context_line":"    "},{"line_number":130,"context_line":"    "},{"line_number":131,"context_line":"    @decorators.idempotent_id(\u00271dce86b8-eb04-4c03-a9d8-9c1dc3ee0c7b\u0027)"},{"line_number":132,"context_line":"    def test_live_block_migration(self):"}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_a72b8e7f","line":129,"range":{"start_line":129,"start_character":0,"end_line":129,"end_character":4},"updated":"2019-11-04 21:17:00.000000000","message":"remove the whitespace chars","commit_id":"ff52d41572e6c24e1a674ff59bc71546e856916e"},{"author":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"change_message_id":"55821601b6c1adc168896e818c9500dd01157fec","unresolved":false,"context_lines":[{"line_number":127,"context_line":"            LOG.info(\"Live migrate back to source %s\", source_host)"},{"line_number":128,"context_line":"            self._live_migrate(server_id, source_host, state, volume_backed)"},{"line_number":129,"context_line":"    "},{"line_number":130,"context_line":"    "},{"line_number":131,"context_line":"    @decorators.idempotent_id(\u00271dce86b8-eb04-4c03-a9d8-9c1dc3ee0c7b\u0027)"},{"line_number":132,"context_line":"    def test_live_block_migration(self):"},{"line_number":133,"context_line":"        self._test_live_migration()"}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_8728d285","line":130,"range":{"start_line":130,"start_character":0,"end_line":130,"end_character":4},"updated":"2019-11-04 21:17:00.000000000","message":"methods should be separated by one blank line, so probably delete this one","commit_id":"ff52d41572e6c24e1a674ff59bc71546e856916e"},{"author":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"change_message_id":"55821601b6c1adc168896e818c9500dd01157fec","unresolved":false,"context_lines":[{"line_number":131,"context_line":"    @decorators.idempotent_id(\u00271dce86b8-eb04-4c03-a9d8-9c1dc3ee0c7b\u0027)"},{"line_number":132,"context_line":"    def test_live_block_migration(self):"},{"line_number":133,"context_line":"        self._test_live_migration()"},{"line_number":134,"context_line":"    "},{"line_number":135,"context_line":"    "},{"line_number":136,"context_line":"    @decorators.idempotent_id(\u0027f11bf4cb-494b-4f7e-98dd-ff5800adb0e7\u0027)"},{"line_number":137,"context_line":"    @testtools.skipUnless(CONF.compute_feature_enabled."},{"line_number":138,"context_line":"                          config_drive,"}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_471e5a5f","line":135,"range":{"start_line":134,"start_character":4,"end_line":135,"end_character":3},"updated":"2019-11-04 21:17:00.000000000","message":"same as above line 129/130","commit_id":"ff52d41572e6c24e1a674ff59bc71546e856916e"},{"author":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"change_message_id":"55821601b6c1adc168896e818c9500dd01157fec","unresolved":false,"context_lines":[{"line_number":136,"context_line":"    @decorators.idempotent_id(\u0027f11bf4cb-494b-4f7e-98dd-ff5800adb0e7\u0027)"},{"line_number":137,"context_line":"    @testtools.skipUnless(CONF.compute_feature_enabled."},{"line_number":138,"context_line":"                          config_drive,"},{"line_number":139,"context_line":"                          \u0027config_drive is not enabled\u0027)   "},{"line_number":140,"context_line":"    def test_live_block_migration_with_config_drive(self):"},{"line_number":141,"context_line":"        self._test_live_migration(config_drive\u003dTrue)"},{"line_number":142,"context_line":"       "}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_6721169e","line":139,"range":{"start_line":139,"start_character":56,"end_line":139,"end_character":59},"updated":"2019-11-04 21:17:00.000000000","message":"whitespace remove","commit_id":"ff52d41572e6c24e1a674ff59bc71546e856916e"},{"author":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"change_message_id":"55821601b6c1adc168896e818c9500dd01157fec","unresolved":false,"context_lines":[{"line_number":139,"context_line":"                          \u0027config_drive is not enabled\u0027)   "},{"line_number":140,"context_line":"    def test_live_block_migration_with_config_drive(self):"},{"line_number":141,"context_line":"        self._test_live_migration(config_drive\u003dTrue)"},{"line_number":142,"context_line":"       "},{"line_number":143,"context_line":"    @decorators.idempotent_id(\u00271e107f21-61b2-4988-8f22-b196e938ab88\u0027)"},{"line_number":144,"context_line":"    @testtools.skipUnless(CONF.compute_feature_enabled.pause,"},{"line_number":145,"context_line":"                          \u0027Pause is not available.\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_073ce2c5","line":142,"range":{"start_line":142,"start_character":0,"end_line":142,"end_character":7},"updated":"2019-11-04 21:17:00.000000000","message":"whitespace remove again","commit_id":"ff52d41572e6c24e1a674ff59bc71546e856916e"},{"author":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"change_message_id":"55821601b6c1adc168896e818c9500dd01157fec","unresolved":false,"context_lines":[{"line_number":153,"context_line":"    @utils.services(\u0027volume\u0027)"},{"line_number":154,"context_line":"    def test_volume_backed_live_migration(self):"},{"line_number":155,"context_line":"        self._test_live_migration(volume_backed\u003dTrue)"},{"line_number":156,"context_line":"    @decorators.idempotent_id(\u0027e19c0cc6-6720-4ed8-be83-b6603ed5c812\u0027)"},{"line_number":157,"context_line":"    @testtools.skipIf(not CONF.compute_feature_enabled."},{"line_number":158,"context_line":"                      block_migration_for_live_migration,"},{"line_number":159,"context_line":"                      \u0027Block Live migration not available\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_e72c2694","line":156,"updated":"2019-11-04 21:17:00.000000000","message":"add an extra blank line above","commit_id":"ff52d41572e6c24e1a674ff59bc71546e856916e"}],"tempest/api/compute/base.py":[{"author":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"change_message_id":"55821601b6c1adc168896e818c9500dd01157fec","unresolved":false,"context_lines":[{"line_number":208,"context_line":"                raise"},{"line_number":209,"context_line":""},{"line_number":210,"context_line":"    @classmethod"},{"line_number":211,"context_line":"    def create_test_server(cls, validatable\u003dFalse, volume_backed\u003dFalse,config_drive\u003dFalse,"},{"line_number":212,"context_line":"                           validation_resources\u003dNone, **kwargs):"},{"line_number":213,"context_line":"        \"\"\"Wrapper utility that returns a test server."},{"line_number":214,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_477ada76","line":211,"updated":"2019-11-04 21:17:00.000000000","message":"add space after ,","commit_id":"ff52d41572e6c24e1a674ff59bc71546e856916e"}],"tempest/common/compute.py":[{"author":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"change_message_id":"55821601b6c1adc168896e818c9500dd01157fec","unresolved":false,"context_lines":[{"line_number":64,"context_line":"def create_test_server(clients, validatable\u003dFalse, validation_resources\u003dNone,"},{"line_number":65,"context_line":"                       tenant_network\u003dNone, wait_until\u003dNone,"},{"line_number":66,"context_line":"                       volume_backed\u003dFalse, name\u003dNone, flavor\u003dNone,"},{"line_number":67,"context_line":"                       image_id\u003dNone,config_drive\u003dFalse, **kwargs):"},{"line_number":68,"context_line":"    \"\"\"Common wrapper utility returning a test server."},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"    This method is a common wrapper returning a test server that can be"}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_677d966b","line":67,"updated":"2019-11-04 21:17:00.000000000","message":"add space after ,","commit_id":"ff52d41572e6c24e1a674ff59bc71546e856916e"},{"author":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"change_message_id":"55821601b6c1adc168896e818c9500dd01157fec","unresolved":false,"context_lines":[{"line_number":200,"context_line":"        kwargs.setdefault(\u0027availability_zone\u0027,"},{"line_number":201,"context_line":"                          CONF.compute.compute_volume_common_az)"},{"line_number":202,"context_line":"    body \u003d clients.servers_client.create_server(name\u003dname, imageRef\u003dimage_id,"},{"line_number":203,"context_line":"                                                flavorRef\u003dflavor,config_drive\u003dconfig_drive,"},{"line_number":204,"context_line":"                                                **kwargs)"},{"line_number":205,"context_line":""},{"line_number":206,"context_line":"    # handle the case of multiple servers"}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_079862bd","line":203,"updated":"2019-11-04 21:17:00.000000000","message":"add space after ,","commit_id":"ff52d41572e6c24e1a674ff59bc71546e856916e"}]}
