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

piotr.kliczewski at gmail.com piotr.kliczewski at gmail.com
Fri Aug 14 12:56:20 UTC 2015


Piotr Kliczewski 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/44840
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I6d636b5538fa81adc2bc0644bd20d50e2d89f82e
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.6
Gerrit-Owner: David Caro <dcaroest at redhat.com>
Gerrit-Reviewer: David Caro <dcaroest at redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>



More information about the Engine-commits mailing list