8:59 p.m.
repo: git.baserock.org:baserock/baserock/morph
branch: danielfirth/S9121
sha1: 8048f4fde73fa3a3a9efcde40620d87dc3061325
Dan Firth (1):
Deployment failures will now remove the disk image
morphlib/plugins/deploy_plugin.py | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
--
1.8.1.2
8:59 p.m.
New subject: [PATCH] Deployment failures will now remove the disk image
From: Dan Firth <dan.firth(a)codethink.co.uk>
---
morphlib/plugins/deploy_plugin.py | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/morphlib/plugins/deploy_plugin.py b/morphlib/plugins/deploy_plugin.py
index dc9f515..17b7e02 100644
--- a/morphlib/plugins/deploy_plugin.py
+++ b/morphlib/plugins/deploy_plugin.py
@@ -416,7 +416,11 @@ class DeployPlugin(cliapp.Plugin):
'.write',
[system_tree, location],
env)
-
+ except Exception as e:
+ os.remove(location)
+ self.app.status(msg='FAILED to deploy %s, reason: %s' %
+ (location, e))
+ self.app.status(msg='Removing disk image')
finally:
# Cleanup.
self.app.status(msg='Cleaning up')
--
1.8.1.2
9:36 p.m.
New subject: [PATCH] Deployment failures will now remove the disk image
On Tue, Oct 08, 2013 at 12:59:25PM +0100, Daniel Firth wrote:
+ except Exception as e:
+ os.remove(location)
+ self.app.status(msg='FAILED to deploy %s, reason: %s' %
+ (location, e))
+ self.app.status(msg='Removing disk image')
finally:
# Cleanup.
self.app.status(msg='Cleaning up')
Location is not always a file path. This will not work in general. -1.
Cleanup needs to be handled on a per-write-extension basis.
3637
days inactive
3637
days old
2 comments
2 participants
participants (2)
-
Daniel Firth -
Richard Maw