Allon Mureinik has posted comments on this change. Change subject: safelease: Unbreak safelease on systemd ......................................................................
Patch Set 1: (1 comment) +1 on the code, please change the bug url https://gerrit.ovirt.org/#/c/46484/1//COMMIT_MSG Commit Message: Line 18: Adding another argument to a script with 9 arguments is ugly, but I Line 19: don't want to make risky changes to this delicate and critical code. Line 20: Line 21: Change-Id: I230b6909781269531eab3d71b516b28ab22de856 Line 22: Bug-Url: https://bugzilla.redhat.com/1222564 https://bugzilla.redhat.com/show_bug.cgi?id=1265177 Line 23: Signed-off-by: Nir Soffer <[email protected]> Line 24: Reviewed-on: https://gerrit.ovirt.org/45963 Line 25: Continuous-Integration: Jenkins CI Line 26: Reviewed-by: Allon Mureinik <[email protected]> -- To view, visit https://gerrit.ovirt.org/46484 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I230b6909781269531eab3d71b516b28ab22de856 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: ovirt-3.5 Gerrit-Owner: Nir Soffer <[email protected]> Gerrit-Reviewer: Adam Litke <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Eyal Edri <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
