[node-patches] Change in ovirt-node[master]: Remove hooks/on-shutdown/01-save-virtwho, add persistence to...

rbarry at redhat.com rbarry at redhat.com
Fri Dec 11 18:03:17 UTC 2015


Ryan Barry has uploaded a new change for review.

Change subject: Remove hooks/on-shutdown/01-save-virtwho, add persistence to inst
......................................................................

Remove hooks/on-shutdown/01-save-virtwho, add persistence to inst

Testing reveals that systemd's reverse ordering of services means
that shutdown hooks execute after /config is already umounted.

This appeared to work during development because the script was
tested (and triggered) before actually rebooting, so
/config/etc/virt-who.d/... already existed when persisting
individual (globbed) files.

Since /config is umounted, we need to just add this to the list
of files/folders which are persisted at install time.

Change-Id: I75df21a0e59fe8591a9989262344d1e872b278e2
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1209356
Signed-off-by: Ryan Barry <rbarry at redhat.com>
---
M src/ovirtnode/ovirtfunctions.py
1 file changed, 1 insertion(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-node refs/changes/44/50344/1

diff --git a/src/ovirtnode/ovirtfunctions.py b/src/ovirtnode/ovirtfunctions.py
index d07eeb8..1194dde 100644
--- a/src/ovirtnode/ovirtfunctions.py
+++ b/src/ovirtnode/ovirtfunctions.py
@@ -84,6 +84,7 @@
                        "/var/lib/glusterd",
                        "/var/lib/rhsm",
                        "/etc/sysconfig/virt-who",
+                       "/etc/virt-who.d/",
                        "/etc/virt-who.conf"]
 
 OVIRT_VARS = {}


-- 
To view, visit https://gerrit.ovirt.org/50344
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I75df21a0e59fe8591a9989262344d1e872b278e2
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Ryan Barry <rbarry at redhat.com>



More information about the node-patches mailing list