From Barak Korren <bkorren(a)redhat.com>:
Barak Korren has submitted this change and it was merged.
Change subject: Re-factored standard-ci builders
......................................................................
Re-factored standard-ci builders
- Reduced code duplicaiton
- Improved log output and error checking
- Inlined the code into YAML for easier maintainance (!include* is the
root of a lot of evil)
Change-Id: I73754db1ab0b363362a80210b8c2e312bf248541
Jira-Ticket:
https://ovirt-jira.atlassian.net/browse/OVIRT-994
Signed-off-by: Barak Korren <bkorren(a)redhat.com>
---
D jobs/confs/shell-scripts/standard_build_artifacts.sh
D jobs/confs/shell-scripts/standard_build_artifacts_manual.sh
D jobs/confs/shell-scripts/standard_check_merged.sh
D jobs/confs/shell-scripts/standard_check_patch.sh
M jobs/confs/yaml/builders/standard.yaml
M jobs/confs/yaml/templates/standard-stage.yaml
6 files changed, 44 insertions(+), 128 deletions(-)
Approvals:
Eyal Edri: Looks good to me
Barak Korren: Verified; Looks good to me, approved; Ready for merge
Gil Shinar: Looks good to me
Jenkins CI: Passed CI tests
--
To view, visit
https://gerrit.ovirt.org/69375
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I73754db1ab0b363362a80210b8c2e312bf248541
Gerrit-PatchSet: 3
Gerrit-Project: jenkins
Gerrit-Branch: master
Gerrit-Owner: Barak Korren <bkorren(a)redhat.com>
Gerrit-Reviewer: Anton Marchukov <amarchuk(a)redhat.com>
Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com>
Gerrit-Reviewer: David Caro <david(a)dcaro.es>
Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com>
Gerrit-Reviewer: Gil Shinar <gshinar(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nadav Goldin <ngoldin(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Shlomo Ben David <sbendavi(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>