From Yaniv Bronhaim <ybron...@redhat.com>: Yaniv Bronhaim has posted comments on this change.
Change subject: check_merged: remove redundant vars ...................................................................... Patch Set 2: (1 comment) https://gerrit.ovirt.org/#/c/70573/2/automation/check-merged.sh File automation/check-merged.sh: Line 104: Line 105: run_infra_tests | tee "$EXPORTS/functional_tests_stdout.$DISTRO.log" Line 106: failed="${PIPESTATUS[0]}" Line 107: Line 108: run_functional_tests | tee "$EXPORTS/functional_tests_stdout.$DISTRO.log" > stale function call. how did that happen?!@# and wasn't fail in the run? Line 109: failed="${PIPESTATUS[0]}" Line 110: Line 111: run_network_tests | tee -a "$EXPORTS/functional_tests_stdout.$DISTRO.log" Line 112: res="${PIPESTATUS[0]}" -- To view, visit https://gerrit.ovirt.org/70573 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Icc55a9eca21d96476cc09e743eab099cfc653c9f Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Yaniv Bronhaim <ybron...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Irit Goihman <igoih...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Leon Goldberg <leon.ot...@gmail.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczew...@gmail.com> Gerrit-Reviewer: Yaniv Bronhaim <ybron...@redhat.com> Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org> Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org