Dan Kenigsberg has submitted this change and it was merged. Change subject: Adding configure sanlock on force ......................................................................
Adding configure sanlock on force If sanlock is down we cannot check sanlock process, so isconfigure return that sanlock process is already related to the supplementary group. Configure verb won't configure sanlock if isconfigure return True currently. Therefore, this patch change the ConfigureOnForce to True so 'vdsm-tool configure --force' will perform the usermod action as the user expected. Change-Id: I0bb3125e64cf02e55738f5978fd81d605dd84d38 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1057225 Signed-off-by: Yaniv Bronhaim <[email protected]> Reviewed-on: http://gerrit.ovirt.org/24007 Reviewed-by: Sandro Bonazzola <[email protected]> Reviewed-by: Dan Kenigsberg <[email protected]> Reviewed-on: http://gerrit.ovirt.org/24153 --- M lib/vdsm/tool/configurator.py 1 file changed, 4 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Yaniv Bronhaim: Verified Dan Kenigsberg: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/24153 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0bb3125e64cf02e55738f5978fd81d605dd84d38 Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: ovirt-3.4 Gerrit-Owner: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Douglas Schilling Landgraf <[email protected]> Gerrit-Reviewer: Sandro Bonazzola <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: [email protected] _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
