Alon Bar-Lev has uploaded a new change for review.

Change subject: node: persist: minor cleanup
......................................................................

node: persist: minor cleanup

Change-Id: Ic89be93394823c5d41b49de9a3500360b1a74e72
Signed-off-by: Alon Bar-Lev <alo...@redhat.com>
---
M src/plugins/ovirt-host-deploy/node/persist.py
1 file changed, 4 insertions(+), 6 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-host-deploy 
refs/changes/11/10111/1

diff --git a/src/plugins/ovirt-host-deploy/node/persist.py 
b/src/plugins/ovirt-host-deploy/node/persist.py
index c8ab376..d109af4 100644
--- a/src/plugins/ovirt-host-deploy/node/persist.py
+++ b/src/plugins/ovirt-host-deploy/node/persist.py
@@ -50,11 +50,9 @@
     def _closeup(self):
         from ovirtnode import ovirtfunctions
 
-        # always persist id, as user
-        # may change this manually
-        ovirtfunctions.ovirt_store_config(odeploycons.Const.VDSM_ID_FILE)
-        self.logger.debug('persisting: %s' % odeploycons.Const.VDSM_ID_FILE)
-
-        for f in self.environment[otopicons.CoreEnv.MODIFIED_FILES]:
+        for f in (
+            [odeploycons.Const.VDSM_ID_FILE] +
+            self.environment[otopicons.CoreEnv.MODIFIED_FILES]
+        ):
             self.logger.debug('persisting: %s' % f)
             ovirtfunctions.ovirt_store_config(f)


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic89be93394823c5d41b49de9a3500360b1a74e72
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-host-deploy
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev <alo...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to