1 |
I stand by what I said. |
2 |
|
3 |
Before revoking commit access, first make sure that their absence is |
4 |
not actively obstructing development, like if their maintainership is |
5 |
occupying packages in need of attention that they aren't looking after |
6 |
and which can't be tampered with by others without infringing on that |
7 |
maintainership. Check to see if there's outstanding bugs against the |
8 |
packages. |
9 |
|
10 |
Then simply email them as stipulated in undertaking procedures. |
11 |
|
12 |
Personally, I think that revocation of commit access and retirement |
13 |
should be considered two distinct processes. |
14 |
|
15 |
Commit access can be revoked and restored with a little bit of |
16 |
administration, and if a developer is inactive for a lengthy period of |
17 |
time, whether they make use of devaway or not, they should have their |
18 |
commit access temporarily revoked just on grounds of security |
19 |
principles. Dormant accounts are potentially vulnerable to being |
20 |
hijacked. This is where the developer's responses (or lack thereof) |
21 |
to activity probing emails should come in handy. |
22 |
|
23 |
Actual retirement though in my opinion should be reserved for cases |
24 |
where their absence is actively harming development, for example by |
25 |
having their absentee maintainership of a package or whatnot obstruct |
26 |
maintenance or bugfixing or development or what have you. |
27 |
|
28 |
In my opinion the last thing gentoo needs is to make it harder for |
29 |
people to contribute. |
30 |
|
31 |
My two cents. |
32 |
|
33 |
On Mon, Nov 12, 2018 at 3:22 PM Alec Warner <antarus@g.o> wrote: |
34 |
> |
35 |
> On Fri, Nov 2, 2018 at 11:05 AM Michał Górny <mgorny@g.o> wrote: |
36 |
>> |
37 |
>> Hello, |
38 |
>> |
39 |
>> The Undertakers team has frequently received various forms of |
40 |
>> 'criticism' of their effort in attempting to find and retire inactive |
41 |
>> developers. This is getting as far as to claim that we shouldn't retire |
42 |
>> anyone because there are no limits on commit slots. |
43 |
> |
44 |
> |
45 |
> So I think the problems are not about commit slots (I think that is a poor way to think about it.) I think the problems we have seen are around developers who lose interest in various areas of the tree. |
46 |
> |
47 |
> - Herds / Projects that list N people, but really only have 1-2 active developers. Sometimes the herd / project has no active developers. |
48 |
> - Metadata.xml that lists N people, but really only have 1-2 active maintainers. Sometimes the package has no active developers. |
49 |
> |
50 |
> The result of the above are essentially: |
51 |
> - Work on a given area of the tree has to wait some time while the existing (inactive) maintainer is pinged. |
52 |
> - A given area of the tree may look well covered (e.g. package has many maintainers listed) when in fact this is untrue and none of the maintainers are active. This leads to developers possibly ignoring that portion of the tree. |
53 |
> |
54 |
> To me, retiring 'inactive' developers is really done to address these issues. If inactive developers are removed from maintainer lists from time to time, we get a better signal on what packages are actively maintained, vs packages that need more support. |
55 |
> |
56 |
> I don't have any particular problem with people who maintain only a few packages; they may not commit often but as long as they care for the packages assigned to them I think they still bring value. The trick is differentiating between these people and inactive people. This is one reason why we always email people; there is an expectation that active developers respond to email and inactive developers do not. It seems to have served us well thus far. |
57 |
> |
58 |
>> |
59 |
>> |
60 |
>> Therefore, I would like to ask the wider community a general question: |
61 |
>> how do you feel about preserving commit access for people who no longer |
62 |
>> actively commit to Gentoo? I'm talking about extreme cases, say, |
63 |
>> no commits to any user-visible repository for over a year. |
64 |
>> |
65 |
>> |
66 |
>> -- |
67 |
>> Best regards, |
68 |
>> Michał Górny |