Gentoo Archives: gentoo-project

From: "Paweł Hajdan
To: gentoo-project@l.g.o
Subject: Re: [gentoo-project] Fwd: [gentoo-dev] rfc: formally allow qa to suspend commit rights
Date: Sun, 19 Jan 2014 20:19:01
Message-Id: 52DC3329.1070903@gentoo.org
In Reply to: Re: [gentoo-project] Fwd: [gentoo-dev] rfc: formally allow qa to suspend commit rights by William Hubbs
1 On 1/19/14, 10:29 AM, William Hubbs wrote:
2 > @@ -30,7 +30,8 @@ The QA team should be given certain abilities to look out for the best interests
3 > * In the case of disagreement among QA members the majority of
4 > established QA members must agree with the action. Some examples of
5 > disagreements are whether the perceived problem violates the policy or
6 > whether the solution makes the situation worse.
7 > * In the event that a developer still insists that a package does
8 > not break QA standards, an appeal can be made at the next council meeting.
9 > The package should be dealt with per QA's request until such a time that
10 > a decision is made by the council.
11 > * Just because a particular QA violation has yet to cause an issue
12 > does not change the fact that it is still a QA violation.
13 > -* If a particular developer persistently causes breakage, the QA
14 > team may request that Comrel re-evaluates that developer's commit rights.
15 > Evidence of past breakages will be presented with this request to Comrel.
16 > +* In case a particular developer persistently causes breakage, the
17 > QA lead may request commit rights of that developer to be suspended by the
18 > Infra team. Comrel should then proceed to evaluate the situation, by
19 > finding a compromise or pernamently revoking those rights.
20
21 +1 (please change pernamently to permanently).
22
23 > +* Should a situation arise where a developer causes breakage to the
24 > point that it cannot be ascribed to an honest mistake, either the QA
25 > lead or two members of the QA team can require the Infra team to
26 > temporarily suspend commit access for the developer, pending analysis of
27 > the causes and resolution to be provided by the QA team within 14 days
28 > of said suspension. Resolution for these kinds of issues is completely
29 > in the hands of the QA team, and only the Gentoo Council can revisit the
30 > case.
31
32 14 days sounds like pretty long. I recommend replacing that with
33 requiring that decision to be validated within 7 days by the QA team
34 lead. If he concurs, the next steps are outlined above, just as if he
35 himself started the process. If he decides to reverse their decision, he
36 can just ask infra and handle the rest of the matter within QA team.
37
38 It's also suspicious to me why not just go to Comrel also in the second
39 case, but the draft directly mentions the Council.
40
41 Paweł

Attachments

File name MIME type
signature.asc application/pgp-signature

Replies