Dan Kenigsberg has posted comments on this change.
Change subject: Fixed %post script error during upgrade ......................................................................
Patch Set 1: Code-Review-1
(1 comment)
http://gerrit.ovirt.org/#/c/35739/1/init/vdsmd_init_common.sh.in File init/vdsmd_init_common.sh.in:
Line 255: # of added bonding options or tweaked kernel defaults) it has to be run Line 256: # after load_needed_modules (modprobe bonding). Line 257: # NetworkManager must not crash on nmcli conn delete. Line 258: run_tasks " \ Line 259: set_sasl_password \ why should we run this on every vdsm startup? calling it from within `vdsm-tool configure` makes more sense to me. Line 260: mkdirs \ Line 261: configure_coredump \ Line 262: configure_vdsm_logs \ Line 263: run_init_hooks \