Change in jenkins[master]: mock_runner fix empty mrmap bug

Code Review gerrit at ovirt.org
Wed Aug 23 13:29:55 UTC 2017


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

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

Change subject: mock_runner fix empty mrmap bug
......................................................................


mock_runner fix empty mrmap bug

Mock runner would wait for input forever
because it tried to `cat` ovirt-ci.mrmap.
If the path to mock-confs-dir was wrong,
or the file doesn't exists it would cause
mock runner to wait forever without any
meaningful message written to the screen.

Change-Id: Ic2fe06037dbf3b398b56361c783042b39efd579f
Signed-off-by: Daniel Belenky <dbelenky at redhat.com>
---
M mock_configs/mock_runner.sh
1 file changed, 1 insertion(+), 1 deletion(-)

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



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic2fe06037dbf3b398b56361c783042b39efd579f
Gerrit-PatchSet: 1
Gerrit-Project: jenkins
Gerrit-Branch: master
Gerrit-Owner: Daniel Belenky <dbelenky 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>


More information about the Engine-commits mailing list