Alon Bar-Lev has uploaded a new change for review. Change subject: bootstrap: cleanup command used to gather node id ......................................................................
bootstrap: cleanup command used to gather node id 1. There is no apparent reason to encapsulate within echo. 2. There is no reason to explicit execute bash. 3. There is no reason to use full path for system utilities, utilities may reside at different locations at differnt setups. If an issue will be found, it should be clearly documented. Change-Id: Iace5593ab5585d1c8e25dcf76882480f3b6676d0 Signed-off-by: Alon Bar-Lev <[email protected]> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsInstaller.java 1 file changed, 2 insertions(+), 1 deletion(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/20/6720/1 -- To view, visit http://gerrit.ovirt.org/6720 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Iace5593ab5585d1c8e25dcf76882480f3b6676d0 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
