)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"40e9ab65bd540948c520263d839fe647866d5d16","unresolved":false,"context_lines":[{"line_number":18,"context_line":"block_device_info parameter. Which also looks consistent within other"},{"line_number":19,"context_line":"driver methods."},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Also it fixes LP #1592015"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Related-Bug: 1592015"},{"line_number":24,"context_line":"Related-Bug: 1580625"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"fa6399be_a4780491","line":21,"range":{"start_line":21,"start_character":0,"end_line":21,"end_character":25},"updated":"2016-10-18 13:30:31.000000000","message":"nit: You can remove this - it\u0027s indicated below","commit_id":"e764bfcb5d618b57959b1197f1f195421cda95ff"},{"author":{"_account_id":10224,"name":"Feodor Tersin","email":"ftersin@hotmail.com","username":"ftersin"},"change_message_id":"7f7bb5d1469af0d2a458979e6e12f3b4c6b01641","unresolved":false,"context_lines":[{"line_number":18,"context_line":"block_device_info parameter. Which also looks consistent within other"},{"line_number":19,"context_line":"driver methods."},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Also it fixes LP #1592015"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Related-Bug: 1592015"},{"line_number":24,"context_line":"Related-Bug: 1580625"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"7a77a97e_50f071fe","line":21,"range":{"start_line":21,"start_character":0,"end_line":21,"end_character":25},"in_reply_to":"fa6399be_a4780491","updated":"2016-11-17 06:25:34.000000000","message":"Done","commit_id":"e764bfcb5d618b57959b1197f1f195421cda95ff"}],"nova/compute/manager.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"40e9ab65bd540948c520263d839fe647866d5d16","unresolved":false,"context_lines":[{"line_number":3429,"context_line":"            bdms \u003d objects.BlockDeviceMappingList.get_by_instance_uuid("},{"line_number":3430,"context_line":"                    context, instance.uuid)"},{"line_number":3431,"context_line":"            block_device_info \u003d self._get_instance_block_device_info("},{"line_number":3432,"context_line":"                                context, instance, bdms\u003dbdms)"},{"line_number":3433,"context_line":"            self.driver.confirm_migration(migration, instance,"},{"line_number":3434,"context_line":"                                          network_info, block_device_info)"},{"line_number":3435,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"fa6399be_24acf4ec","line":3432,"range":{"start_line":3432,"start_character":32,"end_line":3432,"end_character":61},"updated":"2016-10-18 13:30:31.000000000","message":"nit: could indent this the same way as the line above?","commit_id":"e764bfcb5d618b57959b1197f1f195421cda95ff"},{"author":{"_account_id":10224,"name":"Feodor Tersin","email":"ftersin@hotmail.com","username":"ftersin"},"change_message_id":"8cc693a6466fe714494d8c00d1593a3c1deb8518","unresolved":false,"context_lines":[{"line_number":3429,"context_line":"            bdms \u003d objects.BlockDeviceMappingList.get_by_instance_uuid("},{"line_number":3430,"context_line":"                    context, instance.uuid)"},{"line_number":3431,"context_line":"            block_device_info \u003d self._get_instance_block_device_info("},{"line_number":3432,"context_line":"                                context, instance, bdms\u003dbdms)"},{"line_number":3433,"context_line":"            self.driver.confirm_migration(migration, instance,"},{"line_number":3434,"context_line":"                                          network_info, block_device_info)"},{"line_number":3435,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"7a77a97e_54913e46","line":3432,"range":{"start_line":3432,"start_character":32,"end_line":3432,"end_character":61},"in_reply_to":"7a77a97e_70c80da9","updated":"2016-11-17 09:40:01.000000000","message":"wrong comment","commit_id":"e764bfcb5d618b57959b1197f1f195421cda95ff"},{"author":{"_account_id":10224,"name":"Feodor Tersin","email":"ftersin@hotmail.com","username":"ftersin"},"change_message_id":"8cc693a6466fe714494d8c00d1593a3c1deb8518","unresolved":false,"context_lines":[{"line_number":3429,"context_line":"            bdms \u003d objects.BlockDeviceMappingList.get_by_instance_uuid("},{"line_number":3430,"context_line":"                    context, instance.uuid)"},{"line_number":3431,"context_line":"            block_device_info \u003d self._get_instance_block_device_info("},{"line_number":3432,"context_line":"                                context, instance, bdms\u003dbdms)"},{"line_number":3433,"context_line":"            self.driver.confirm_migration(migration, instance,"},{"line_number":3434,"context_line":"                                          network_info, block_device_info)"},{"line_number":3435,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"7a77a97e_149ee662","line":3432,"range":{"start_line":3432,"start_character":32,"end_line":3432,"end_character":61},"in_reply_to":"fa6399be_24acf4ec","updated":"2016-11-17 09:40:01.000000000","message":"Done","commit_id":"e764bfcb5d618b57959b1197f1f195421cda95ff"},{"author":{"_account_id":10224,"name":"Feodor Tersin","email":"ftersin@hotmail.com","username":"ftersin"},"change_message_id":"7f7bb5d1469af0d2a458979e6e12f3b4c6b01641","unresolved":false,"context_lines":[{"line_number":3429,"context_line":"            bdms \u003d objects.BlockDeviceMappingList.get_by_instance_uuid("},{"line_number":3430,"context_line":"                    context, instance.uuid)"},{"line_number":3431,"context_line":"            block_device_info \u003d self._get_instance_block_device_info("},{"line_number":3432,"context_line":"                                context, instance, bdms\u003dbdms)"},{"line_number":3433,"context_line":"            self.driver.confirm_migration(migration, instance,"},{"line_number":3434,"context_line":"                                          network_info, block_device_info)"},{"line_number":3435,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"7a77a97e_70c80da9","line":3432,"range":{"start_line":3432,"start_character":32,"end_line":3432,"end_character":61},"in_reply_to":"fa6399be_24acf4ec","updated":"2016-11-17 06:25:34.000000000","message":"changed by using improved _is_booted_from_volume","commit_id":"e764bfcb5d618b57959b1197f1f195421cda95ff"}],"nova/virt/hyperv/driver.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"40e9ab65bd540948c520263d839fe647866d5d16","unresolved":false,"context_lines":[{"line_number":310,"context_line":"                                                             retry_interval)"},{"line_number":311,"context_line":""},{"line_number":312,"context_line":"    def confirm_migration(self, migration, instance, network_info,"},{"line_number":313,"context_line":"                          block_device_info\u003dNone):"},{"line_number":314,"context_line":"        self._migrationops.confirm_migration(migration, instance, network_info)"},{"line_number":315,"context_line":""},{"line_number":316,"context_line":"    def finish_revert_migration(self, context, instance, network_info,"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa6399be_04dd3070","line":313,"range":{"start_line":313,"start_character":26,"end_line":313,"end_character":48},"updated":"2016-10-18 13:30:31.000000000","message":"You should probably use:\n\n    *args, **kwargs):\n\nhere instead, seeing as you\u0027re not using this variable","commit_id":"e764bfcb5d618b57959b1197f1f195421cda95ff"},{"author":{"_account_id":10224,"name":"Feodor Tersin","email":"ftersin@hotmail.com","username":"ftersin"},"change_message_id":"4c3b54d0f085ad4f412f333ecb776afbb015c27c","unresolved":false,"context_lines":[{"line_number":310,"context_line":"                                                             retry_interval)"},{"line_number":311,"context_line":""},{"line_number":312,"context_line":"    def confirm_migration(self, migration, instance, network_info,"},{"line_number":313,"context_line":"                          block_device_info\u003dNone):"},{"line_number":314,"context_line":"        self._migrationops.confirm_migration(migration, instance, network_info)"},{"line_number":315,"context_line":""},{"line_number":316,"context_line":"    def finish_revert_migration(self, context, instance, network_info,"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa6399be_a71646a5","line":313,"range":{"start_line":313,"start_character":26,"end_line":313,"end_character":48},"in_reply_to":"fa6399be_04dd3070","updated":"2016-10-18 14:13:06.000000000","message":"Look at L275-278 in this file.\n\nAnd i think it\u0027s more right to specify all parameters of overridden methods even if not all of them are used. This is a part of an abstract interface, it must stay the same for all implementation.","commit_id":"e764bfcb5d618b57959b1197f1f195421cda95ff"}]}