[Gerrit] NACR is not set

Bug #1591005 reported by Ivan Udovichenko
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Committed
High
Andrey Nikitin

Bug Description

For some reason Non-Author-Code-Review can't be set, even after 2 non-author reviews:
https://review.fuel-infra.org/#/c/21678/

Project: openstack-build/oslo.concurrency-build

Tags: area-devops
Changed in fuel:
status: New → Confirmed
Igor Shishkin (teran)
tags: added: area-devops
removed: area-ci
Revision history for this message
Andrey Nikitin (heos) wrote :

Ivan, as workaround: try to create another request and upload the code. I'm not sure, but I think, it should helps you.

Revision history for this message
Ivan Udovichenko (iudovichenko) wrote :
Changed in fuel:
importance: High → Critical
Revision history for this message
Ivan Udovichenko (iudovichenko) wrote :

And request was not merged even after set Verified (+2):
https://review.fuel-infra.org/#/c/21678/
Anyway this ^^^ particular requests we don't need anymore.

We need just this one:
https://review.fuel-infra.org/#/c/22359/

Which will unblock our further work on packages.

Revision history for this message
Andrey Nikitin (heos) wrote :

The root cause of the bug is incorrect rules in NACR-related file.

That file had the following rule "gerrit:commit_label(label('Code-Review', 1), R)," in line 9 instead of "gerrit:commit_label(label('Code-Review', 2), R),".

The first rule means, that a request should has one CR+1, as minimum, to allow merge it with a lot of CR+2 scores.

I've updated the rules.pl file with proper rule.

Please check and let me know, if you will have a problem with merging.

Changed in fuel:
assignee: Fuel DevOps (fuel-devops) → Andrey Nikitin (heos)
importance: Critical → High
status: Confirmed → In Progress
status: In Progress → Fix Committed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.