)]}'
{"nova/virt/libvirt/driver.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"84d0d05f4d4361c43c9fe5160a5bf661bcc5739b","unresolved":false,"context_lines":[{"line_number":2959,"context_line":"                config_drive_image.import_file("},{"line_number":2960,"context_line":"                    instance, configdrive_path, \u0027disk.config\u0027 + suffix)"},{"line_number":2961,"context_line":"            finally:"},{"line_number":2962,"context_line":"                config_drive_image.import_file_cleanup(configdrive_path)"},{"line_number":2963,"context_line":""},{"line_number":2964,"context_line":"        # File injection only if needed"},{"line_number":2965,"context_line":"        elif inject_files and CONF.libvirt.inject_partition !\u003d -2:"}],"source_content_type":"text/x-python","patch_set":2,"id":"7a2fa921_e4aa5885","line":2962,"updated":"2015-10-05 20:27:44.000000000","message":"You can an UnboundLocalError here if self.image_backend.image failed, see bug 1502961.","commit_id":"9ba70756de326ffaa8be43acfde12cad04ed0af2"}],"nova/virt/libvirt/imagebackend.py":[{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"cee15c14b3aef4314233e23dedbaf7ea65e67bb8","unresolved":false,"context_lines":[{"line_number":408,"context_line":"        # we should talk about if we want this functionality for everything."},{"line_number":409,"context_line":"        pass"},{"line_number":410,"context_line":""},{"line_number":411,"context_line":"    def import_file_cleanup(self, instance, path):"},{"line_number":412,"context_line":"        \"\"\"Once an import has run, we many no longer need the local file."},{"line_number":413,"context_line":""},{"line_number":414,"context_line":"        :param local_file: path to the file that was imported."}],"source_content_type":"text/x-python","patch_set":1,"id":"da20952f_263008e8","line":411,"updated":"2015-09-03 07:28:32.000000000","message":"here should be local_file instead of path because of the following docstring.","commit_id":"02dbd7621a010869d0f34833d7841bc0ba572ba7"},{"author":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"change_message_id":"ed0677ed51727914a837a677c8f44e0cab53d1db","unresolved":false,"context_lines":[{"line_number":408,"context_line":"        # we should talk about if we want this functionality for everything."},{"line_number":409,"context_line":"        pass"},{"line_number":410,"context_line":""},{"line_number":411,"context_line":"    def import_file_cleanup(self, instance, path):"},{"line_number":412,"context_line":"        \"\"\"Once an import has run, we many no longer need the local file."},{"line_number":413,"context_line":""},{"line_number":414,"context_line":"        :param local_file: path to the file that was imported."}],"source_content_type":"text/x-python","patch_set":1,"id":"da20952f_1c96704c","line":411,"in_reply_to":"da20952f_263008e8","updated":"2015-09-03 11:00:58.000000000","message":"Done","commit_id":"02dbd7621a010869d0f34833d7841bc0ba572ba7"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"cee15c14b3aef4314233e23dedbaf7ea65e67bb8","unresolved":false,"context_lines":[{"line_number":844,"context_line":"            self.driver.remove_image(name)"},{"line_number":845,"context_line":"        self.driver.import_image(local_file, name)"},{"line_number":846,"context_line":""},{"line_number":847,"context_line":"    def import_file_cleanup(self, instance, local_file):"},{"line_number":848,"context_line":"        if os.path.exists(local_file):"},{"line_number":849,"context_line":"            os.unlink(local_file)"},{"line_number":850,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"da20952f_a60c183c","line":847,"updated":"2015-09-03 07:28:32.000000000","message":"instance is not used in both import_file_cleanup()s, so we can remove it.","commit_id":"02dbd7621a010869d0f34833d7841bc0ba572ba7"},{"author":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"change_message_id":"ed0677ed51727914a837a677c8f44e0cab53d1db","unresolved":false,"context_lines":[{"line_number":844,"context_line":"            self.driver.remove_image(name)"},{"line_number":845,"context_line":"        self.driver.import_image(local_file, name)"},{"line_number":846,"context_line":""},{"line_number":847,"context_line":"    def import_file_cleanup(self, instance, local_file):"},{"line_number":848,"context_line":"        if os.path.exists(local_file):"},{"line_number":849,"context_line":"            os.unlink(local_file)"},{"line_number":850,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"da20952f_dc9f985d","line":847,"in_reply_to":"da20952f_a60c183c","updated":"2015-09-03 11:00:58.000000000","message":"Done","commit_id":"02dbd7621a010869d0f34833d7841bc0ba572ba7"}]}