mooli tayer has uploaded a new change for review.

Change subject: configurator.py: Node unpersist must be called before removing 
a file.
......................................................................

configurator.py: Node unpersist must be called before removing a file.

Change-Id: I7226b792c904ff8ca35054e20b85be707c080b49
Signed-off-by: Mooli Tayer <[email protected]>
---
M lib/vdsm/tool/configurator.py
1 file changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/21/28821/1

diff --git a/lib/vdsm/tool/configurator.py b/lib/vdsm/tool/configurator.py
index eb9f1ca..b32e612 100644
--- a/lib/vdsm/tool/configurator.py
+++ b/lib/vdsm/tool/configurator.py
@@ -320,10 +320,10 @@
         delete a file if it exists.
         """
         try:
-            os.unlink(content['path'])
-
             if utils.isOvirtNode():
                 NodeCfg().unpersist(content['path'])
+
+            os.unlink(content['path'])
         except OSError as e:
             if e.errno != errno.ENOENT:
                 raise InvalidRun("Removing file: %s failed", content['path'])


-- 
To view, visit http://gerrit.ovirt.org/28821
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I7226b792c904ff8ca35054e20b85be707c080b49
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: mooli tayer <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to