Change in ovirt-engine[master]: Added dummy check-patch

sbonazzo at redhat.com sbonazzo at redhat.com
Fri Aug 14 12:26:43 UTC 2015


Sandro Bonazzola has submitted this change and it was merged.

Change subject: Added dummy check-patch
......................................................................


Added dummy check-patch

Because not always will a job run for ovirt-engine, this ensures that
at least one job will run and the CI flag will be set.

The flow where this scripts comes into play, is when you submit a
patch, and before jenkins had time to give a review, you submit a new
patch that does no code changes (like just changing the commit
message). At that point, gerrit has no previous review to carry over
to the new patch and jenkins will not get triggered as it does not
have any code changes, leaving that patch without CI flag at all.

With this dummy script (and the related job) in that case, this job
will do the dummy review and add the CI flag.

Change-Id: I6d636b5538fa81adc2bc0644bd20d50e2d89f82e
Signed-off-by: David Caro <dcaroest at redhat.com>
---
A automation/check-patch.sh
1 file changed, 3 insertions(+), 0 deletions(-)

Approvals:
  Sandro Bonazzola: Verified; Looks good to me, approved; Passed CI tests



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I6d636b5538fa81adc2bc0644bd20d50e2d89f82e
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: David Caro <dcaroest at redhat.com>
Gerrit-Reviewer: David Caro <dcaroest at redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list