Adopt the idea that a review label name should be as
widely-applicable as possible for a given concept so that different
repos with similar needs will end up sharing columns in the user
interface. Since the TC and infra-specs repos both need "votes from
the people who are members of the committee" as a review label
(while preserving the ability for non-members to leave their feedback
as +/- entries in code-review), have them share the name "Rollcall
Vote". This will cause them both to appear in the "RV" column in
Gerrit change lists. Simply "Vote" is an option worth considering,
but it is not clear how "Vote" is different from Code-Review.
Change-Id: Id53d1bea7f53b92009ba6e9274162b8ffa3b2b92
The voting configuration for infra-specs produces a red-x and
green-check for +/-1 code reviews now, which is confusing to people
since they normally associate those icons with +/-2 votes. This
change expands the vote range to include +/-2 but gives no one
permission to vote at those levels. This should cause Code-Review
votes to appear as +/-1, while the NoOp function continues to
mean that +2 votes are not necessary.
Also extend the Council-Vote to +2, but similarly give no permissions
to use it so that council votes will appear as +1, -1, or red-x in
the case of a -2 veto (the -1/green-check dichotomy would also
seem weird for similar reasons).
Change-Id: I042e6168334fb11291edbaed97b3ae2424218166
Infra is adopting a new specs governance process; implement the
voting rules necessary for that.
Change-Id: I46071d52736cd0e1fc4b78df421601f1f357345c
Depends-On: Iea53a45bb533968a13a85c0e7fbb0787325e4095
This repo was created from filter branching the openstack-infra/
config repo. This process brought a lot of cruft with it in the
form of directories that we no longer need. This patch removes
that cruft so we begin with a tidier repo.
Change-Id: Ibffad1b11c0c5f84eedfb0365369f60c4961a0f3