Dan Kenigsberg has uploaded a new change for review. Change subject: vdsm startup: fix assert statement ......................................................................
vdsm startup: fix assert statement SyntaxWarning: assertion is always true, perhaps remove parentheses? Change-Id: I0b8cb905494b0719147bf299bb6411c3c4616407 Signed-off-by: Dan Kenigsberg <[email protected]> --- M vdsm/vdsm 1 file changed, 3 insertions(+), 3 deletions(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/45/9745/1 diff --git a/vdsm/vdsm b/vdsm/vdsm index 035ea5d..5c81a40 100755 --- a/vdsm/vdsm +++ b/vdsm/vdsm @@ -118,9 +118,9 @@ "to run from user %s" % (constants.VDSM_USER, username) ) group = grp.getgrnam(constants.VDSM_GROUP) - assert ((constants.VDSM_USER in group.gr_mem - or pwd.getpwnam(constants.VDSM_USER).pw_gid == group.gr_gid), - "VDSM failed to start: vdsm user is not in KVM group") + assert (constants.VDSM_USER in group.gr_mem + or pwd.getpwnam(constants.VDSM_USER).pw_gid == group.gr_gid), \ + "VDSM failed to start: vdsm user is not in KVM group" if __name__ == '__main__': __assertVdsmUser() -- To view, visit http://gerrit.ovirt.org/9745 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I0b8cb905494b0719147bf299bb6411c3c4616407 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Dan Kenigsberg <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
