Change in jenkins[master]: lago-pipeline: refactoring

[Code Review]

>From Eyal Edri <eedri at redhat.com>:

Eyal Edri has submitted this change and it was merged.

Change subject: lago-pipeline: refactoring
......................................................................


lago-pipeline: refactoring

1. Assert 'sha1' and 'REFSPEC' parameters are passed to the job,
otherwise fail. This will be done inside a preliminary stage, thus
allowing to use the proper Jenkins stage failure mechanism, and avoid
messing with buildResult parameters. Did not include assertion for
gerrit parameters, as we use this flow for github only at the moment.

2. Use 'params' instead of 'env' where applicable.

3. Use the new 'stage' syntax.

4. Remove code duplication and unify functions.

5. Use the new milestone syntax to guarantee only one job gets to the
deploy stage.

6. Moved main() to the top of the code.

7. JJB variables will be substituted into the main function call.

Change-Id: I13f3b9ac30266a6372e9b0e34de8511629f9453c
Signed-off-by: Nadav Goldin <ngoldin at redhat.com>
---
M jobs/confs/groovy-scripts/standard-pipeline.groovy
1 file changed, 70 insertions(+), 139 deletions(-)

Approvals:
  Nadav Goldin: Verified; Ready for review
  Eyal Edri: Looks good to me, approved; Ready for merge
  Barak Korren: Looks good to me
  Jenkins CI: Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/72648
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I13f3b9ac30266a6372e9b0e34de8511629f9453c
Gerrit-PatchSet: 7
Gerrit-Project: jenkins
Gerrit-Branch: master
Gerrit-Owner: Nadav Goldin <ngoldin at redhat.com>
Gerrit-Reviewer: Anton Marchukov <amarchuk at redhat.com>
Gerrit-Reviewer: Barak Korren <bkorren at redhat.com>
Gerrit-Reviewer: Daniel Belenky <dbelenky at redhat.com>
Gerrit-Reviewer: Eyal Edri <eedri at redhat.com>
Gerrit-Reviewer: Gal Ben Haim <galbh2 at gmail.com>
Gerrit-Reviewer: Gil Shinar <gshinar at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nadav Goldin <ngoldin at redhat.com>
Gerrit-Reviewer: Shlomo Ben David <sbendavi at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>