[node-patches] Change in ovirt-node[master]: mount only files and directories listed in /etc/files

mburns at redhat.com mburns at redhat.com
Thu Oct 4 13:12:37 UTC 2012


Michael Burns has posted comments on this change.

Change subject: mount only files and directories listed in /etc/files
......................................................................


Patch Set 1: (1 inline comment)

....................................................
File scripts/ovirt-functions.in
Line 382:             cp -rv --update /live/config/* /config
Line 383:         fi
Line 384:         # bind mount all persisted configs to rootfs
Line 385:         for f in $(cat /config/files); do
Line 386:             if grep -q " $f ext[234]" /proc/mounts ; then
In this case, it's only attempting to filter out pre-existing filesystem devices like /var/log /config, etc which are things that are created by ovirt-node and known. 

Thanks for the feedback though
Line 387:                 # skip if already bind-mounted
Line 388:                 true
Line 389:             else
Line 390:                 mkdir -p "$(dirname "$f")"


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I6dce8d619b8a6a51d636974eb34141f8aac9bae7
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Michael Burns <mburns at redhat.com>
Gerrit-Reviewer: Alan Pevec <apevec at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Joey Boggs <jboggs at redhat.com>
Gerrit-Reviewer: Michael Burns <mburns at redhat.com>



More information about the node-patches mailing list