Nir Soffer has posted comments on this change.
Change subject: fcp: Deactivate vdsm volume groups during boot ......................................................................
Patch Set 1:
(3 comments)
Ayal, the current implementation would not cut the storage under a running VM, as it makes sure deactivate_vdsm_vgs runs only once since boot.
Moving the code to a pre-task would save you the need to duplicate this service for upstart and systemd.
Using pre-start will work for install or when upgrading from fixed version to newer one, but is not correct when upgrading unfixed version, because there may be running vms using activated lvs. For this kind of upgrade we may either require a reboot, or detect which lvs are used and deactivate the rest.
Nir, please add the new file to the Makefile, spec and deb definitions.
I'm working an alternative fix. I'll continue in this direction if the alternative fix is rejected.
.................................................... Commit Message Line 17: This script must also be used during installation or ugprade. It is safe Line 18: to invoke it multiple times; it will modify volume groups only on the Line 19: first run. Line 20: Line 21: On RHEL 6.5 we can use new activation skipping option instead of this This will work for Fedora, with the proper systemd configuration file.
Will fix RHEL trademark issue. Line 22: script. I'll address this in a separate patch. Line 23: Line 24: Change-Id: I8f72a68ad09566ba222aa45448c78d1577c40d21 Line 25: Bug-Url: https://bugzilla.redhat.com/1009812
.................................................... File init/sysvinit/vdsm-deactivate-vgs.init Line 22: ### END INIT INFO Line 23: Line 24: . /etc/init.d/functions Line 25: Line 26: run_file="var/run/vdsm/lvm/deactivate-vgs" Will fix. Line 27: prog="vdsm-deactivate-vgs" Line 28: retval=0 Line 29: Line 30: log_failure_msg()
Line 38: } Line 39: Line 40: is_first_run() Line 41: { Line 42: test ! -f $run_file Will fix. Line 43: } Line 44: Line 45: set_was_run() Line 46: {