Skip to content

Conversation

@aschmahmann
Copy link
Contributor

Summary

Updating the org admins to handle availability and practicality issues.

  • Removed Andy Schwab (@andyschwab-admin, @andyschwab) since he's no longer involved in the project
  • Promoted @lidel from temporary to permanent owner given he's in practice been dealing with things that need org owner permissions since this list was shrunk in 2023
  • Added myself (back) as an admin since I'm also regularly around and involved, have done this before, and am operating in a different time zone from the others

Why do you need this?

The bus factor on the org owners was too low (today we almost ran into a major issue with CI being dead and this repo effectively stalled). Additionally we had an org owner who wasn't really around. This should resolve both issues without really increasing the risk surface.

What else do we need to know?

DRI: myself

Reviewer's Checklist

  • It is clear where the request is coming from (if unsure, ask)
  • All the automated checks passed
  • The YAML changes reflect the summary of the request
  • The Terraform plan posted as a comment reflects the summary of the request

@aschmahmann aschmahmann requested a review from a team as a code owner July 7, 2025 18:02
@aschmahmann aschmahmann requested review from a team and galargh July 7, 2025 18:02
@github-actions
Copy link
Contributor

github-actions bot commented Jul 7, 2025

The following access changes will be introduced as a result of applying the plan:

Access Changes
User andyschwab-admin:
  - will have the role in the organization change from admin to member
User aschmahmann:
  - will have the role in the organization change from member to admin

@aschmahmann aschmahmann force-pushed the feat/update-owners branch from 5fba196 to 6e0c03a Compare July 7, 2025 18:07
@github-actions
Copy link
Contributor

github-actions bot commented Jul 7, 2025

Before merge, verify that all the following plans are correct. They will be applied as-is after the merge.

Terraform plans

ipfs

Terraform used the selected providers to generate the following execution
plan. Resource actions are indicated with the following symbols:
  ~ update in-place

Terraform will perform the following actions:

  # github_membership.this["andyschwab-admin"] will be updated in-place
  ~ resource "github_membership" "this" {
        id       = "ipfs:andyschwab-admin"
      ~ role     = "admin" -> "member"
        # (2 unchanged attributes hidden)
    }

  # github_membership.this["aschmahmann"] will be updated in-place
  ~ resource "github_membership" "this" {
        id       = "ipfs:aschmahmann"
      ~ role     = "member" -> "admin"
        # (2 unchanged attributes hidden)
    }

Plan: 0 to add, 2 to change, 0 to destroy.

@galargh galargh merged commit 16a0209 into master Jul 15, 2025
6 checks passed
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

Successfully merging this pull request may close these issues.

5 participants