Change in jenkins[master]: Separate jenkins diffs folders

Code Review gerrit at ovirt.org
Sun Aug 27 09:33:41 UTC 2017


>From Barak Korren <bkorren at redhat.com>:

Barak Korren has submitted this change and it was merged. ( https://gerrit.ovirt.org/80161 )

Change subject: Separate jenkins diffs folders
......................................................................


Separate jenkins diffs folders

This patch fixes two main issues:
1) We do not want to have git changes in the original
   workspace. For that we will create a temporary
   folder for running the JJB in order to create the
   old_xmls
2) Jenkins check-patch diffed the current patch with
   the current master which was wrong. We actually want
   to diff the current patch with the master it was on.

Some additional changes:
1) Removed execution of check_publishers_not_deleted.py
   from check-patch.sh and added it to jenkins_check_yaml.sh
2) Changed the code of jenkins_check_yaml.sh to be a bit
   more modular. Without global parameters and with functions

Change-Id: Idb16bb01e07f952a00ed88ea2ea6bdc4b835f73d
Signed-off-by: Gil Shinar <gshinar at redhat.com>
---
M automation/check-patch.sh
M automation/check_publishers_not_deleted.py
M automation/jenkins_check_yaml.sh
3 files changed, 89 insertions(+), 68 deletions(-)

Approvals:
  Barak Korren: Looks good to me, approved; Ready for merge
  Gil Shinar: Verified; Ready for review
  Jenkins CI: Passed CI tests



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Idb16bb01e07f952a00ed88ea2ea6bdc4b835f73d
Gerrit-PatchSet: 9
Gerrit-Project: jenkins
Gerrit-Branch: master
Gerrit-Owner: Gil Shinar <gshinar at redhat.com>
Gerrit-Reviewer: Anton Marchukov <amarchuk at redhat.com>
Gerrit-Reviewer: Barak Korren <bkorren at redhat.com>
Gerrit-Reviewer: Dafna Ron <dron at redhat.com>
Gerrit-Reviewer: Daniel Belenky <dbelenky at redhat.com>
Gerrit-Reviewer: Evgheni Dereveanchin <ederevea at redhat.com>
Gerrit-Reviewer: Eyal Edri <eedri at redhat.com>
Gerrit-Reviewer: Gil Shinar <gshinar at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nadav Goldin <ngoldin at redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Shlomo Ben David <sbendavi at redhat.com>
Gerrit-Reviewer: Staging Jenkins Server


More information about the Engine-commits mailing list