Change in gerrit-admin[master]: Include master branch in hook consideration

eedri at redhat.com eedri at redhat.com
Sun Aug 21 14:26:09 UTC 2016


Eyal Edri has submitted this change and it was merged.

Change subject: Include master branch in hook consideration
......................................................................


Include master branch in hook consideration

	until now master branch wasn't in the logic
	of deciding if to move bug to MODIFIED or not.
	this was because we searched for branch-suffix
	matching x.y.z, while master will always have
	empty match for that.
	this patch should include master in the check
	and if there are newer branches that match
	the target milestone, the status change
	will be skipped

Change-Id: I963d5fd32f8bcf757f8608b60b84225ea87d7c08
Signed-off-by: Eyal Edri <eedri at redhat.com>
Signed-off-by: Shlomi Ben-David <sbendavi at redhat.com>
---
M hooks/custom_hooks/change-merged.set_MODIFIED
1 file changed, 18 insertions(+), 6 deletions(-)

Approvals:
  Shlomo Ben David: Verified
  Eyal Edri: Looks good to me, approved
  Jenkins CI: Passed CI tests



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I963d5fd32f8bcf757f8608b60b84225ea87d7c08
Gerrit-PatchSet: 3
Gerrit-Project: gerrit-admin
Gerrit-Branch: master
Gerrit-Owner: Eyal Edri <eedri at redhat.com>
Gerrit-Reviewer: Anton Marchukov <amarchuk at redhat.com>
Gerrit-Reviewer: Eyal Edri <eedri at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek at redhat.com>
Gerrit-Reviewer: Shlomo Ben David <sbendavi at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list