Cleaning infra groups on Pagure and GitHub

[Michal Konecny]

Hi everyone,

I did a cleaning in Fedora infra groups in both Github (fedora-infra 
organization https://github.com/fedora-infra) and Pagure (fedora-infra 
group https://pagure.io/group/fedora-infra). I removed the people that 
were inactive in the last year.

I also removed people from infra-sig FAS group 
(https://accounts.fedoraproject.org/group/infra-sig/) I only left the 
same members as in fedora-infra pagure group (I didn't add any). I 
thought that those groups should be synced, but it doesn't seem like 
they are. Is this correct?

If I accidentally removed somebody who should be in those groups, let me 
know and I will add you back.

Michal

[Pavel Raiskup]

On pátek 1. září 2023 14:59:12 CEST Michal Konecny wrote:
>  I did a cleaning in Fedora infra groups in both Github (fedora-infra
>
>  organization https://github.com/fedora-infra) and Pagure (fedora-infra 
>  group https://pagure.io/group/fedora-infra). I removed the people that 
>  were inactive in the last year. 
I received "Removed from the "Fedora Infrastructure" organization"
e-mail.  **I think** I was added there because we need to be able to set
status on https://github.com/fedora-infra/statusfpo  - not sure if I'll
be able to this?  Do I actually need to be in that group?  Otherwise, I
try to be active (ansible.git, Fedora Copr), not sure how "inactive in
the last year" happened. :-)

Anyways, thank you for doing this maintenance task, Michal!
Pavel

>  I also removed people from infra-sig FAS group 
>  (https://accounts.fedoraproject.org/group/infra-sig/) I only left the 
>  same members as in fedora-infra pagure group (I didn't add any). I 
>  thought that those groups should be synced, but it doesn't seem like 
>  they are. Is this correct?
>  
>  If I accidentally removed somebody who should be in those groups, let me 
>  know and I will add you back.
>  
>  Michal
>  _______________________________________________
>  infrastructure mailing list -- infrastructure(a)lists.fedoraproject.org
>  To unsubscribe send an email to infrastructure-leave(a)lists.fedoraproject.org
>  Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
>  List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
>  List Archives:
> https://lists.fedoraproject.org/archives/list/[e-mail]...
>  Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
>

[Michal Konecny]

Hi Pavel,

I sent you invite again.

I looked at the activity only on GitHub side for the fedora-infra 
repositories, that could be the reason of inactivity.

Michal

On 01. 09. 23 15:16, Pavel Raiskup wrote:
>  On pátek 1. září 2023 14:59:12 CEST Michal Konecny wrote:
> > I did a cleaning in Fedora infra groups in both Github (fedora-infra
> > organization https://github.com/fedora-infra) and Pagure (fedora-infra
> > group https://pagure.io/group/fedora-infra). I removed the people that
> > were inactive in the last year.
>  I received "Removed from the "Fedora Infrastructure" organization"
>  e-mail.  **I think** I was added there because we need to be able to set
>  status on https://github.com/fedora-infra/statusfpo  - not sure if I'll
>  be able to this?  Do I actually need to be in that group?  Otherwise, I
>  try to be active (ansible.git, Fedora Copr), not sure how "inactive in
>  the last year" happened. :-)
>
>  Anyways, thank you for doing this maintenance task, Michal!
>  Pavel
>
> > I also removed people from infra-sig FAS group
> > (https://accounts.fedoraproject.org/group/infra-sig/) I only left the
> > same members as in fedora-infra pagure group (I didn't add any). I
> > thought that those groups should be synced, but it doesn't seem like
> > they are. Is this correct?
> >
> > If I accidentally removed somebody who should be in those groups, let me
> > know and I will add you back.
> >
> > Michal
> > _______________________________________________
> > infrastructure mailing list -- infrastructure(a)lists.fedoraproject.org
> > To unsubscribe send an email to infrastructure-leave(a)lists.fedoraproject.org
> > Fedora Code of Conduct:
> https://docs.fedoraproject.org/en-US/project/code-of-conduct/
> > List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
> > List Archives:
> https://lists.fedoraproject.org/archives/list/[e-mail]...
> > Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
> >
>
>
>

[Pierre-Yves Chibon]

On Fri, Sep 01, 2023 at 02:59:12PM +0200, Michal Konecny wrote:
>  Hi everyone,
>  
>  I did a cleaning in Fedora infra groups in both Github (fedora-infra
>  organization https://github.com/fedora-infra) and Pagure (fedora-infra group
>  https://pagure.io/group/fedora-infra). I removed the people that were
>  inactive in the last year.
>  
>  I also removed people from infra-sig FAS group
>  (https://accounts.fedoraproject.org/group/infra-sig/) I only left the same
>  members as in fedora-infra pagure group (I didn't add any). I thought that
>  those groups should be synced, but it doesn't seem like they are. Is this
>  correct? 
Groups in FAS and pagure aren't automatically synced (unlike with src.fp.o).
I had a script back then that would allow to do it (I believe this may the
something that CentOS use or used):
https://pagure.io/pagure-utility/blob/master/f/sync_fas_group_membership.py
It definitely needs a refresh to become useful again though.


Pierre

[Michal Konecny]

Thanks for pointing on this, I will add it to my TODO list.

Michal

On 01. 09. 23 15:19, Pierre-Yves Chibon wrote:
>  On Fri, Sep 01, 2023 at 02:59:12PM +0200, Michal Konecny wrote:
> > Hi everyone,
> >
> > I did a cleaning in Fedora infra groups in both Github (fedora-infra
> > organization https://github.com/fedora-infra) and Pagure (fedora-infra group
> > https://pagure.io/group/fedora-infra). I removed the people that were
> > inactive in the last year.
> >
> > I also removed people from infra-sig FAS group
> > (https://accounts.fedoraproject.org/group/infra-sig/) I only left the same
> > members as in fedora-infra pagure group (I didn't add any). I thought that
> > those groups should be synced, but it doesn't seem like they are. Is this
> > correct?
>  Groups in FAS and pagure aren't automatically synced (unlike with src.fp.o).
>  I had a script back then that would allow to do it (I believe this may the
>  something that CentOS use or used):
>  https://pagure.io/pagure-utility/blob/master/f/sync_fas_group_membership.py
>  It definitely needs a refresh to become useful again though.
>
>
>  Pierre
>  _______________________________________________
>  infrastructure mailing list -- infrastructure(a)lists.fedoraproject.org
>  To unsubscribe send an email to infrastructure-leave(a)lists.fedoraproject.org
>  Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
>  List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
>  List Archives:
> https://lists.fedoraproject.org/archives/list/[e-mail]...
>  Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue

[Kevin Fenzi]

On Fri, Sep 01, 2023 at 02:59:12PM +0200, Michal Konecny wrote:
>  Hi everyone,
>  
>  I did a cleaning in Fedora infra groups in both Github (fedora-infra
>  organization https://github.com/fedora-infra) and Pagure (fedora-infra group
>  https://pagure.io/group/fedora-infra). I removed the people that were
>  inactive in the last year.
>  
>  I also removed people from infra-sig FAS group
>  (https://accounts.fedoraproject.org/group/infra-sig/) I only left the same
>  members as in fedora-infra pagure group (I didn't add any). I thought that
>  those groups should be synced, but it doesn't seem like they are. Is this
>  correct?
>  
>  If I accidentally removed somebody who should be in those groups, let me
>  know and I will add you back. 
Thanks for doing the cleanup. 

I think perhaps we should try and do this more regularly (once a year?
twice a year after releases?) and use a more open method. I like the
method nb used for clearing out groups a while back:

* Identify the potentially inactive people
* Open a ticket explaining that we are removing inactive people and
@mention all of the people on the list.
* If they see it and decide they want to be active again, they can
comment in the ticket about that
* After a week or whatever we remove everyone who hasn't responded.

That gives people a chance to notice and become active again and also
prevents any surprize when they are removed. 

Thoughts?

kevin

[Michal Konecny]

That sounds like a reasonable way to do it.

To determine the activity I was using audit log for organization on 
GitHub, but it doesn't seem to work very well. In case of Pagure I was 
just looking at user activity and if there weren't any on infra repos in 
last year.

Michal

On 05. 09. 23 17:16, Kevin Fenzi wrote:
>  On Fri, Sep 01, 2023 at 02:59:12PM +0200, Michal Konecny wrote:
> > Hi everyone,
> >
> > I did a cleaning in Fedora infra groups in both Github (fedora-infra
> > organization https://github.com/fedora-infra) and Pagure (fedora-infra group
> > https://pagure.io/group/fedora-infra). I removed the people that were
> > inactive in the last year.
> >
> > I also removed people from infra-sig FAS group
> > (https://accounts.fedoraproject.org/group/infra-sig/) I only left the same
> > members as in fedora-infra pagure group (I didn't add any). I thought that
> > those groups should be synced, but it doesn't seem like they are. Is this
> > correct?
> >
> > If I accidentally removed somebody who should be in those groups, let me
> > know and I will add you back.
>  Thanks for doing the cleanup.
>
>  I think perhaps we should try and do this more regularly (once a year?
>  twice a year after releases?) and use a more open method. I like the
>  method nb used for clearing out groups a while back:
>
>  * Identify the potentially inactive people
>  * Open a ticket explaining that we are removing inactive people and
>  @mention all of the people on the list.
>  * If they see it and decide they want to be active again, they can
>  comment in the ticket about that
>  * After a week or whatever we remove everyone who hasn't responded.
>
>  That gives people a chance to notice and become active again and also
>  prevents any surprize when they are removed.
>
>  Thoughts?
>
>  kevin
>
>  _______________________________________________
>  infrastructure mailing list -- infrastructure(a)lists.fedoraproject.org
>  To unsubscribe send an email to infrastructure-leave(a)lists.fedoraproject.org
>  Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
>  List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
>  List Archives:
> https://lists.fedoraproject.org/archives/list/[e-mail]...
>  Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue