)]}'
{"nova/virt/libvirt/driver.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"59ad49daab0b4a4b606de86c563be0b2f1cbb949","unresolved":false,"context_lines":[{"line_number":2946,"context_line":"                                      \u0027with error: %s\u0027),"},{"line_number":2947,"context_line":"                                  e, instance\u003dinstance)"},{"line_number":2948,"context_line":""},{"line_number":2949,"context_line":"            config_drive_image \u003d self.image_backend.image("},{"line_number":2950,"context_line":"                instance, \u0027disk.config\u0027 + suffix,"},{"line_number":2951,"context_line":"                self._get_disk_config_image_type())"},{"line_number":2952,"context_line":"            try:"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a2fa921_44e22cd7","line":2949,"updated":"2015-10-05 20:30:03.000000000","message":"So this obviously fails and that\u0027s how you hit the regression introduced here:\n\nhttps://review.openstack.org/#/c/220018/2/nova/virt/libvirt/driver.py\n\nSo I\u0027d declare a config_drive_image variable outside the try/except and check if it\u0027s set before calling import_file_cleanup, like:\n\nconfig_drive_image \u003d None\ntry:\n    ...\nfinally:\n    if config_drive_image:\n        config_drive_image.import_file_cleanup(...)","commit_id":"bb738bc41e873b77617699f7782b2fe006e80610"}]}
