Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Look at who has ADMIN #1000

Open
garethahealy opened this issue Jan 27, 2025 · 3 comments
Open

Look at who has ADMIN #1000

garethahealy opened this issue Jan 27, 2025 · 3 comments

Comments

@garethahealy
Copy link
Contributor

Off the back of the below, let's look at who has ADMIN and whether they need it. Certain teams should have ADMIN but I feel that should be called out in the team name. Those who do not need ADMIN should be MAINTAINER or similar.

@garethahealy
Copy link
Contributor Author

@redhat-cop/redhat-cop-sre

@garethahealy
Copy link
Contributor Author

garethahealy commented Jan 27, 2025

before i go ahead and make changes to the config.yaml - wanted some feedback.

so my current idea is to follow these rules/style:

  • top level team is only membership, doesn't include repos or maintainers
  • child team for admins/merges who are maintainers

so the below which is now:

agnosticd-core:
  description: Core team members for the agnosticd project.
  maintainers:
    - agonzalezrh
    - fridim
    - jkupferer
    - newgoliath
    - prakhar1985
    - rut31337
    - tonykay
    - wkulhanek
  members: []
  privacy: closed
  repos:
    agnosticd: admin

would become:

agnosticd-core:
  description: Core team members for the agnosticd project.
  members:
    - agonzalezrh
    - fridim
    - jkupferer
    - newgoliath
    - prakhar1985
    - rut31337
    - tonykay
    - wkulhanek
  privacy: closed
  teams:
    agnosticd-core-admins:
      description: Core admin members for the agnosticd project.
      maintainers:
        - agonzalezrh
        - fridim
        - jkupferer
        - newgoliath
        - prakhar1985
        - rut31337
        - tonykay
        - wkulhanek
      privacy: closed
      repos:
        agnosticd: admin

and containers-cop as-is:

container-cop:
  description: Contributors to the Containers & PaaS CoP
  maintainers:
    - etsauer
    - oybed
    - sabre1041
  members:
    - abaird-rh
    - bbeaudoin
    - beelandc
    - bszeti
    - ckavili
    - davgordo
    - drichtarik
    - garethahealy
    - huddlesj
    - infosec812
    - jaredburck
    - jimdillon
    - jooho
    - juliaaano
    - juozasa
    - kkoller
    - linuxstudio
    - lpsantil
    - luckylittle
    - makentenza
    - malacourse
    - mathianasj
    - mike4263
    - mophahr
    - mwitzenm
    - noelo
    - pabrahamsson
    - raffaelespazzoli
    - rpelisse
    - samueltauil
    - shah-zobair
    - springdo
    - stocky37
    - themoosman
    - vvaldez
  privacy: closed
  repos: {}
  teams:
    bats:
      description: Bash Automation Testing System
      maintainers:
        - ckavili
        - garethahealy
        - pabrahamsson
        - springdo
      privacy: closed
      repos: {}
      teams:
        bats-mergers:
          description: BATS Mergers & Admins
          maintainers:
            - ckavili
            - garethahealy
            - pabrahamsson
            - springdo
          privacy: closed
          repos:
            bats-library: admin

would become:

container-cop:
  description: Contributors to the Containers & PaaS CoP
  members:
    - etsauer
    - oybed
    - sabre1041
    - abaird-rh
    - bbeaudoin
    - beelandc
    - bszeti
    - ckavili
    - davgordo
    - drichtarik
    - garethahealy
    - huddlesj
    - infosec812
    - jaredburck
    - jimdillon
    - jooho
    - juliaaano
    - juozasa
    - kkoller
    - linuxstudio
    - lpsantil
    - luckylittle
    - makentenza
    - malacourse
    - mathianasj
    - mike4263
    - mophahr
    - mwitzenm
    - noelo
    - pabrahamsson
    - raffaelespazzoli
    - rpelisse
    - samueltauil
    - shah-zobair
    - springdo
    - stocky37
    - themoosman
    - vvaldez
  privacy: closed
  repos: {}
  teams:
    bats:
      description: Bash Automation Testing System
      members:
        - ckavili
        - garethahealy
        - pabrahamsson
        - springdo
      privacy: closed
      repos: {}
      teams:
        bats-admins:
          description: BATS Mergers & Admins
          maintainers:
            - ckavili
            - garethahealy
            - pabrahamsson
            - springdo
          privacy: closed
          repos:
            bats-library: admin

@garethahealy
Copy link
Contributor Author

thoughts @djdanielsson @sabre1041 ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant