[node-patches] Change in ovirt-node[ovirt-3.6]: kdump expects more and more -- create /var/crash on nfs ours...

[fabiand at redhat.com]

Fabian Deutsch has uploaded a new change for review.

Change subject: kdump expects more and more -- create /var/crash on nfs ourselves
......................................................................

kdump expects more and more -- create /var/crash on nfs ourselves

kdumpctl now expects the nfs path to be mounted *and* for the path
it expects to use to already exist, otherwise kdumpctl fails.

So, check whether path is set, and assume /var/crash if not, then
create it.

Change-Id: Ifa5c90a8bbbb116f0715b3a58fc8e0efa4da6933
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1257857
Signed-off-by: Ryan Barry <rbarry at redhat.com>
---
M src/ovirt/node/config/defaults.py
1 file changed, 7 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-node refs/changes/08/46108/1

diff --git a/src/ovirt/node/config/defaults.py b/src/ovirt/node/config/defaults.py
index 7f6542d..428579d 100755
--- a/src/ovirt/node/config/defaults.py
+++ b/src/ovirt/node/config/defaults.py
@@ -1153,6 +1153,13 @@
                         os.makedirs(nfs_path)
                     system.Mount(nfs_path, nfs, "nfs").mount()
 
+                    crash_path = aug.get("/files/etc/kdump.conf/path") or \
+                        "var/crash"
+
+                    crash_path = "%s/%s" % (nfs_path, crash_path)
+                    if not os.path.isdir(crash_path):
+                        os.makedirs(crash_path)
+
                     File("/etc/fstab").write(
                         "\n%s\t%s\tnfs\tdefaults\t0 0" % (nfs, nfs_path),
                         "a")


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ifa5c90a8bbbb116f0715b3a58fc8e0efa4da6933
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-node
Gerrit-Branch: ovirt-3.6
Gerrit-Owner: Fabian Deutsch <fabiand at redhat.com>
Gerrit-Reviewer: Fabian Deutsch <fabiand at redhat.com>