Cleanup permissions for shipyard maintained repos #211

aschmahmann · 2024-08-20T18:43:20Z

Summary

This is a general cleanup of permissions across a number of repos that Shipyard maintains, also adds a shipyard team. The general forms of the changes look like:

Remove individuals in exchange for teams
Replace w3dt-stewards access with shipyard access (generally with shipyard having maintain rather than admin unless the repo already had lots of individually added admins, although even there we could probably reduce permissions more)
Removed the github-mgmt stewards groups pull permissions since those don't seem necessary (pull generally seems not that useful in public repos, and there's nothing special for the github-mgmt stewards to need there that they won't have elsewhere via org permissions)
Reduced some users' permissions who haven't made commits in a while (but happy to restore if they're interested)

Why do you need this?

Overall the number of admins in these repos seems much higher than necessary, especially given the ability to escalate via github-mgmt if needed. For repos shipyard maintains I'd like permissions scoped down where possible. This seems like the easiest cleanup although I wouldn't be surprised if we want to shrink permissions more and/or look at more repos than just the ones I've covered here.

What else do we need to know?

TODO: tag people with permission reductions once CI generates the list

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

…ssary teams

…o to push access

…rs, remove github-mgmt pull permissions

…ermissions

…move github-mgmt stewards pull permissions

…ewards pull permissions, add shipyard as maintainers

…ams, reduce admins, remove github-mgmt stewards pull permissions

…e github-mgmt stewards pull permissions

…stewards pull permissions

github-actions · 2024-08-20T18:44:17Z

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

Access Changes

User 2color:
  - will have the permission to boxo change from admin to maintain
  - will have the permission to kubo change from admin to maintain
  - will have the permission to rainbow change from admin to maintain
  - will have the permission to someguy change from admin to maintain
User aarshkshah1992:
  - will have the permission to boxo change from maintain to push
User achingbrain:
  - will have the permission to boxo change from admin to maintain
  - will have the permission to kubo change from admin to maintain
  - will have the permission to rainbow change from admin to maintain
  - will have the permission to service-worker-gateway change from pull to admin
  - will have the permission to someguy change from admin to maintain
User adlrocha:
  - will have the permission to boxo change from maintain to push
User alanshaw:
  - will have the permission to boxo change from admin to push
  - will have the permission to kubo change from admin to push
  - will lose admin permission to rainbow
  - will lose admin permission to someguy
User anorth:
  - will have the permission to boxo change from maintain to push
User arajasek:
  - will lose admin permission to boxo
  - will lose admin permission to kubo
  - will lose admin permission to rainbow
  - will lose admin permission to someguy
User aschmahmann:
  - will have the permission to helia change from admin to maintain
  - will have the permission to helia-cli change from pull to maintain
  - will have the permission to helia-delegated-routing-v1-http-api change from admin to maintain
  - will have the permission to helia-http-gateway change from pull to maintain
  - will have the permission to helia-remote-pinning change from pull to maintain
  - will have the permission to helia-verified-fetch change from admin to maintain
User daviddias:
  - will have the permission to boxo change from admin to push
  - will have the permission to kubo change from admin to push
  - will lose admin permission to rainbow
  - will lose admin permission to someguy
User dennis-tra:
  - will lose push permission to kubo
User dignifiedquire:
  - will have the permission to boxo change from maintain to push
User dirkmc:
  - will have the permission to boxo change from maintain to push
User frrist:
  - will have the permission to boxo change from maintain to push
User galargh:
  - will lose admin permission to kubo
User gammazero:
  - will have the permission to helia change from admin to maintain
  - will have the permission to helia-delegated-routing-v1-http-api change from admin to maintain
  - will have the permission to helia-verified-fetch change from admin to maintain
  - will gain maintain permission to helia-cli
  - will gain maintain permission to helia-http-gateway
  - will gain maintain permission to helia-remote-pinning
  - will gain admin permission to service-worker-gateway
User gmasgras:
  - will have the permission to boxo change from maintain to push
User guillaumemichel:
  - will have the permission to boxo change from admin to maintain
  - will have the permission to helia change from admin to maintain
  - will have the permission to helia-delegated-routing-v1-http-api change from admin to maintain
  - will have the permission to helia-verified-fetch change from admin to maintain
  - will have the permission to kubo change from admin to maintain
  - will have the permission to rainbow change from admin to maintain
  - will have the permission to someguy change from admin to maintain
  - will gain maintain permission to helia-cli
  - will gain maintain permission to helia-http-gateway
  - will gain maintain permission to helia-remote-pinning
  - will gain admin permission to service-worker-gateway
User guseggert:
  - will have the permission to boxo change from maintain to push
User hacdias:
  - will have the permission to boxo change from maintain to push
User hannahhoward:
  - will have the permission to boxo change from maintain to push
User hsanjuan:
  - will have the permission to boxo change from admin to push
  - will have the permission to kubo change from admin to push
  - will lose admin permission to rainbow
  - will lose admin permission to someguy
User iand:
  - will have the permission to boxo change from maintain to push
User ischasny:
  - will have the permission to boxo change from maintain to push
User jacobheun:
  - will have the permission to boxo change from maintain to push
User jbenet:
  - will have the permission to boxo change from admin to push
  - will have the permission to kubo change from admin to push
  - will lose admin permission to rainbow
  - will lose admin permission to someguy
User jorropo:
  - will have the permission to boxo change from admin to push
  - will lose admin permission to helia
  - will lose admin permission to helia-delegated-routing-v1-http-api
  - will lose admin permission to helia-verified-fetch
  - will lose admin permission to ipfs-check
  - will have the permission to kubo change from admin to push
  - will lose admin permission to rainbow
  - will lose admin permission to someguy
User kubuxu:
  - will have the permission to boxo change from admin to push
  - will have the permission to kubo change from admin to push
  - will lose admin permission to rainbow
  - will lose admin permission to someguy
User laurentsenta:
  - will lose admin permission to kubo
User lidel:
  - will have the permission to helia change from admin to maintain
  - will have the permission to helia-cli change from push to maintain
  - will have the permission to helia-delegated-routing-v1-http-api change from admin to maintain
  - will have the permission to helia-http-gateway change from pull to maintain
  - will have the permission to helia-remote-pinning change from pull to maintain
  - will have the permission to helia-verified-fetch change from admin to maintain
User magik6k:
  - will have the permission to boxo change from maintain to push
User marcopolo:
  - will have the permission to boxo change from admin to maintain
  - will have the permission to helia change from admin to maintain
  - will have the permission to helia-delegated-routing-v1-http-api change from admin to maintain
  - will have the permission to helia-verified-fetch change from admin to maintain
  - will have the permission to kubo change from admin to maintain
  - will have the permission to rainbow change from admin to maintain
  - will have the permission to someguy change from admin to maintain
  - will gain maintain permission to helia-cli
  - will gain maintain permission to helia-http-gateway
  - will gain maintain permission to helia-remote-pinning
User marten-seemann:
  - will have the permission to boxo change from admin to push
  - will have the permission to kubo change from admin to push
  - will lose admin permission to rainbow
  - will lose admin permission to someguy
User masih:
  - will have the permission to boxo change from maintain to push
User mishmosh:
  - will lose pull permission to boxo
  - will lose pull permission to helia
  - will lose pull permission to helia-cli
  - will lose pull permission to helia-delegated-routing-v1-http-api
  - will lose pull permission to helia-http-gateway
  - will lose pull permission to helia-remote-pinning
  - will lose pull permission to helia-verified-fetch
  - will lose pull permission to rainbow
  - will lose pull permission to service-worker-gateway
  - will lose pull permission to someguy
User momack2:
  - will lose admin permission to boxo
  - will have the permission to kubo change from admin to pull
  - will lose admin permission to rainbow
  - will lose admin permission to someguy
User olizilla:
  - will lose admin permission to boxo
  - will have the permission to kubo change from admin to pull
  - will lose admin permission to rainbow
  - will lose admin permission to someguy
User petar:
  - will have the permission to boxo change from maintain to push
User raulk:
  - will have the permission to boxo change from maintain to push
User ribasushi:
  - will have the permission to boxo change from maintain to push
User rvagg:
  - will have the permission to boxo change from maintain to push
User sgtpooki:
  - will have the permission to boxo change from admin to maintain
  - will have the permission to kubo change from admin to maintain
  - will have the permission to rainbow change from admin to maintain
  - will have the permission to someguy change from admin to maintain
User stebalien:
  - will have the permission to boxo change from admin to push
  - will lose pull permission to helia
  - will lose pull permission to helia-cli
  - will lose pull permission to helia-delegated-routing-v1-http-api
  - will lose pull permission to helia-http-gateway
  - will lose pull permission to helia-remote-pinning
  - will lose pull permission to helia-verified-fetch
  - will have the permission to kubo change from admin to push
  - will lose admin permission to rainbow
  - will lose pull permission to service-worker-gateway
  - will lose admin permission to someguy
User travisperson:
  - will have the permission to boxo change from maintain to push
User vyzo:
  - will have the permission to boxo change from maintain to push
User warpfork:
  - will have the permission to boxo change from maintain to push
User whizzzkid:
  - will lose admin permission to helia
  - will lose admin permission to helia-cli
  - will lose admin permission to helia-delegated-routing-v1-http-api
  - will lose admin permission to helia-http-gateway
  - will lose admin permission to helia-remote-pinning
  - will lose admin permission to helia-verified-fetch
  - will lose pull permission to service-worker-gateway
User whyrusleeping:
  - will have the permission to boxo change from admin to push
  - will have the permission to kubo change from admin to push
  - will lose admin permission to rainbow
  - will lose admin permission to someguy
User willscott:
  - will have the permission to boxo change from maintain to push
  - will lose pull permission to helia
  - will lose pull permission to helia-cli
  - will lose pull permission to helia-delegated-routing-v1-http-api
  - will lose pull permission to helia-http-gateway
  - will lose pull permission to helia-remote-pinning
  - will lose pull permission to helia-verified-fetch
  - will lose pull permission to rainbow
  - will lose pull permission to service-worker-gateway
  - will lose pull permission to someguy

github-actions · 2024-08-20T18:44:39Z

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:
  + create
  ~ update in-place
  - destroy

Terraform will perform the following actions:

  # github_branch_protection.this["kubo:bifrost-*"] will be updated in-place
  ~ resource "github_branch_protection" "this" {
        id                              = "MDIwOkJyYW5jaFByb3RlY3Rpb25SdWxlMjEwNDk3MDM="
      ~ push_restrictions               = [
          - "/aschmahmann",
          - "/gmasgras",
          - "/thattommyhall",
          + "ipfs/kubo-maintainers",
        ]
        # (10 unchanged attributes hidden)
    }

  # github_branch_protection.this["kubo:feat/stabilize-dht"] will be updated in-place
  ~ resource "github_branch_protection" "this" {
        id                              = "MDIwOkJyYW5jaFByb3RlY3Rpb25SdWxlMTUwNDUxNjk="
      ~ push_restrictions               = [
          - "/aschmahmann",
          - "/gmasgras",
          + "ipfs/kubo-maintainers",
        ]
        # (10 unchanged attributes hidden)
    }

  # github_branch_protection.this["kubo:master"] will be updated in-place
  ~ resource "github_branch_protection" "this" {
        id                              = "MDIwOkJyYW5jaFByb3RlY3Rpb25SdWxlMjI5MDgxMA=="
      ~ push_restrictions               = [
          + "ipfs/kubo-maintainers",
        ]
        # (10 unchanged attributes hidden)

        # (2 unchanged blocks hidden)
    }

  # github_branch_protection.this["kubo:release"] will be updated in-place
  ~ resource "github_branch_protection" "this" {
        id                              = "MDIwOkJyYW5jaFByb3RlY3Rpb25SdWxlNjU3MzY4"
      ~ push_restrictions               = [
          - "/Jorropo",
          - "/Stebalien",
          - "/aschmahmann",
          - "/hacdias",
          - "/whyrusleeping",
        ]
        # (10 unchanged attributes hidden)

        # (2 unchanged blocks hidden)
    }

  # github_branch_protection.this["kubo:release-*"] will be updated in-place
  ~ resource "github_branch_protection" "this" {
        id                              = "MDIwOkJyYW5jaFByb3RlY3Rpb25SdWxlMTU2NzQyNTc="
      ~ push_restrictions               = [
          - "/Stebalien",
          - "/aschmahmann",
          - "/hsanjuan",
          - "/whyrusleeping",
        ]
        # (10 unchanged attributes hidden)
    }

  # github_repository_collaborator.this["helia-cli:achingbrain"] will be destroyed
  # (because key ["helia-cli:achingbrain"] is not in for_each map)
  - resource "github_repository_collaborator" "this" {
      - id         = "helia-cli:achingbrain" -> null
      - permission = "admin" -> null
      - repository = "helia-cli" -> null
      - username   = "achingbrain" -> null
    }

  # github_repository_collaborator.this["helia-delegated-routing-v1-http-api:achingbrain"] will be destroyed
  # (because key ["helia-delegated-routing-v1-http-api:achingbrain"] is not in for_each map)
  - resource "github_repository_collaborator" "this" {
      - id         = "helia-delegated-routing-v1-http-api:achingbrain" -> null
      - permission = "admin" -> null
      - repository = "helia-delegated-routing-v1-http-api" -> null
      - username   = "achingbrain" -> null
    }

  # github_repository_collaborator.this["helia-http-gateway:sgtpooki"] will be destroyed
  # (because key ["helia-http-gateway:sgtpooki"] is not in for_each map)
  - resource "github_repository_collaborator" "this" {
      - id         = "helia-http-gateway:SgtPooki" -> null
      - permission = "admin" -> null
      - repository = "helia-http-gateway" -> null
      - username   = "SgtPooki" -> null
    }

  # github_repository_collaborator.this["helia-http-gateway:whizzzkid"] will be destroyed
  # (because key ["helia-http-gateway:whizzzkid"] is not in for_each map)
  - resource "github_repository_collaborator" "this" {
      - id         = "helia-http-gateway:whizzzkid" -> null
      - permission = "admin" -> null
      - repository = "helia-http-gateway" -> null
      - username   = "whizzzkid" -> null
    }

  # github_repository_collaborator.this["helia-remote-pinning:sgtpooki"] will be destroyed
  # (because key ["helia-remote-pinning:sgtpooki"] is not in for_each map)
  - resource "github_repository_collaborator" "this" {
      - id         = "helia-remote-pinning:SgtPooki" -> null
      - permission = "admin" -> null
      - repository = "helia-remote-pinning" -> null
      - username   = "SgtPooki" -> null
    }

  # github_repository_collaborator.this["kubo:dennis-tra"] will be destroyed
  # (because key ["kubo:dennis-tra"] is not in for_each map)
  - resource "github_repository_collaborator" "this" {
      - id         = "kubo:dennis-tra" -> null
      - permission = "push" -> null
      - repository = "kubo" -> null
      - username   = "dennis-tra" -> null
    }

  # github_repository_collaborator.this["kubo:lidel"] will be destroyed
  # (because key ["kubo:lidel"] is not in for_each map)
  - resource "github_repository_collaborator" "this" {
      - id         = "kubo:lidel" -> null
      - permission = "admin" -> null
      - repository = "kubo" -> null
      - username   = "lidel" -> null
    }

  # github_repository_collaborator.this["service-worker-gateway:2color"] will be destroyed
  # (because key ["service-worker-gateway:2color"] is not in for_each map)
  - resource "github_repository_collaborator" "this" {
      - id         = "service-worker-gateway:2color" -> null
      - permission = "admin" -> null
      - repository = "service-worker-gateway" -> null
      - username   = "2color" -> null
    }

  # github_repository_collaborator.this["service-worker-gateway:aschmahmann"] will be destroyed
  # (because key ["service-worker-gateway:aschmahmann"] is not in for_each map)
  - resource "github_repository_collaborator" "this" {
      - id         = "service-worker-gateway:aschmahmann" -> null
      - permission = "admin" -> null
      - repository = "service-worker-gateway" -> null
      - username   = "aschmahmann" -> null
    }

  # github_repository_collaborator.this["service-worker-gateway:lidel"] will be destroyed
  # (because key ["service-worker-gateway:lidel"] is not in for_each map)
  - resource "github_repository_collaborator" "this" {
      - id         = "service-worker-gateway:lidel" -> null
      - permission = "admin" -> null
      - repository = "service-worker-gateway" -> null
      - username   = "lidel" -> null
    }

  # github_repository_collaborator.this["service-worker-gateway:marcopolo"] will be destroyed
  # (because key ["service-worker-gateway:marcopolo"] is not in for_each map)
  - resource "github_repository_collaborator" "this" {
      - id         = "service-worker-gateway:MarcoPolo" -> null
      - permission = "admin" -> null
      - repository = "service-worker-gateway" -> null
      - username   = "MarcoPolo" -> null
    }

  # github_repository_collaborator.this["service-worker-gateway:sgtpooki"] will be destroyed
  # (because key ["service-worker-gateway:sgtpooki"] is not in for_each map)
  - resource "github_repository_collaborator" "this" {
      - id         = "service-worker-gateway:SgtPooki" -> null
      - permission = "admin" -> null
      - repository = "service-worker-gateway" -> null
      - username   = "SgtPooki" -> null
    }

  # github_team.this["shipyard"] will be created
  + resource "github_team" "this" {
      + create_default_maintainer = false
      + description               = "Members of Interplanetary Shipyard who work with or on IPFS"
      + etag                      = (known after apply)
      + id                        = (known after apply)
      + members_count             = (known after apply)
      + name                      = "shipyard"
      + node_id                   = (known after apply)
      + privacy                   = "secret"
      + slug                      = (known after apply)
    }

  # github_team_membership.this["helia-dev:whizzzkid"] will be destroyed
  # (because key ["helia-dev:whizzzkid"] is not in for_each map)
  - resource "github_team_membership" "this" {
      - etag     = "W/\"004122db9e59a99753541c983eafb6c7ba8f9587d75179b2e6afca9f07bc27ce\"" -> null
      - id       = "7676419:whizzzkid" -> null
      - role     = "member" -> null
      - team_id  = "7676419" -> null
      - username = "whizzzkid" -> null
    }

  # github_team_membership.this["kubo maintainers:jorropo"] will be destroyed
  # (because key ["kubo maintainers:jorropo"] is not in for_each map)
  - resource "github_team_membership" "this" {
      - etag     = "W/\"31ae88602d846916c2b08e19c6fe23e5d4148631bad07576e7de05950b445e48\"" -> null
      - id       = "6744049:Jorropo" -> null
      - role     = "member" -> null
      - team_id  = "6744049" -> null
      - username = "Jorropo" -> null
    }

  # github_team_membership.this["shipyard:2color"] will be created
  + resource "github_team_membership" "this" {
      + etag     = (known after apply)
      + id       = (known after apply)
      + role     = "member"
      + team_id  = (known after apply)
      + username = "2color"
    }

  # github_team_membership.this["shipyard:achingbrain"] will be created
  + resource "github_team_membership" "this" {
      + etag     = (known after apply)
      + id       = (known after apply)
      + role     = "member"
      + team_id  = (known after apply)
      + username = "achingbrain"
    }

  # github_team_membership.this["shipyard:aschmahmann"] will be created
  + resource "github_team_membership" "this" {
      + etag     = (known after apply)
      + id       = (known after apply)
      + role     = "maintainer"
      + team_id  = (known after apply)
      + username = "aschmahmann"
    }

  # github_team_membership.this["shipyard:gammazero"] will be created
  + resource "github_team_membership" "this" {
      + etag     = (known after apply)
      + id       = (known after apply)
      + role     = "member"
      + team_id  = (known after apply)
      + username = "gammazero"
    }

  # github_team_membership.this["shipyard:guillaumemichel"] will be created
  + resource "github_team_membership" "this" {
      + etag     = (known after apply)
      + id       = (known after apply)
      + role     = "member"
      + team_id  = (known after apply)
      + username = "guillaumemichel"
    }

  # github_team_membership.this["shipyard:lidel"] will be created
  + resource "github_team_membership" "this" {
      + etag     = (known after apply)
      + id       = (known after apply)
      + role     = "maintainer"
      + team_id  = (known after apply)
      + username = "lidel"
    }

  # github_team_membership.this["shipyard:marcopolo"] will be created
  + resource "github_team_membership" "this" {
      + etag     = (known after apply)
      + id       = (known after apply)
      + role     = "member"
      + team_id  = (known after apply)
      + username = "MarcoPolo"
    }

  # github_team_membership.this["shipyard:sgtpooki"] will be created
  + resource "github_team_membership" "this" {
      + etag     = (known after apply)
      + id       = (known after apply)
      + role     = "member"
      + team_id  = (known after apply)
      + username = "SgtPooki"
    }

  # github_team_repository.this["admin:boxo"] will be destroyed
  # (because key ["admin:boxo"] is not in for_each map)
  - resource "github_team_repository" "this" {
      - etag       = "W/\"a18ec7fd3157202760a2eacfeeb481299b6541f3ad168266fa32468d6fb26728\"" -> null
      - id         = "1516991:boxo" -> null
      - permission = "admin" -> null
      - repository = "boxo" -> null
      - team_id    = "1516991" -> null
    }

  # github_team_repository.this["admin:kubo"] will be destroyed
  # (because key ["admin:kubo"] is not in for_each map)
  - resource "github_team_repository" "this" {
      - etag       = "W/\"49a7a25e4019d90314c1cd82d6cd9ce733735732d7f967c64cc597f4c9714e91\"" -> null
      - id         = "1516991:kubo" -> null
      - permission = "admin" -> null
      - repository = "kubo" -> null
      - team_id    = "1516991" -> null
    }

  # github_team_repository.this["admin:rainbow"] will be destroyed
  # (because key ["admin:rainbow"] is not in for_each map)
  - resource "github_team_repository" "this" {
      - etag       = "W/\"f8430965d7e7b9dc7d50ee009345a6159de9bd25e9cbcb0436f05477190aa314\"" -> null
      - id         = "1516991:rainbow" -> null
      - permission = "admin" -> null
      - repository = "rainbow" -> null
      - team_id    = "1516991" -> null
    }

  # github_team_repository.this["admin:someguy"] will be destroyed
  # (because key ["admin:someguy"] is not in for_each map)
  - resource "github_team_repository" "this" {
      - etag       = "W/\"1bf3d64fc07f1071099ea4bce87abaedd6d321bb09a8ba1808fcb351ddcb7cf3\"" -> null
      - id         = "1516991:someguy" -> null
      - permission = "admin" -> null
      - repository = "someguy" -> null
      - team_id    = "1516991" -> null
    }

  # github_team_repository.this["github-mgmt stewards:boxo"] will be destroyed
  # (because key ["github-mgmt stewards:boxo"] is not in for_each map)
  - resource "github_team_repository" "this" {
      - etag       = "W/\"c39b22a824f1b5d17e33ee59aa42281a831b86f5582c91adc160aa27ee18886b\"" -> null
      - id         = "6421993:boxo" -> null
      - permission = "pull" -> null
      - repository = "boxo" -> null
      - team_id    = "6421993" -> null
    }

  # github_team_repository.this["github-mgmt stewards:helia"] will be destroyed
  # (because key ["github-mgmt stewards:helia"] is not in for_each map)
  - resource "github_team_repository" "this" {
      - etag       = "W/\"4be61384ac8310d56c38189d6fd5749f587b211745bb2865419c5f70bc2c4611\"" -> null
      - id         = "6421993:helia" -> null
      - permission = "pull" -> null
      - repository = "helia" -> null
      - team_id    = "6421993" -> null
    }

  # github_team_repository.this["github-mgmt stewards:helia-cli"] will be destroyed
  # (because key ["github-mgmt stewards:helia-cli"] is not in for_each map)
  - resource "github_team_repository" "this" {
      - etag       = "W/\"a6f3456cbb6fe765e0ba5f313f0f777e7ad71832966cbd8231a723f76840e546\"" -> null
      - id         = "6421993:helia-cli" -> null
      - permission = "pull" -> null
      - repository = "helia-cli" -> null
      - team_id    = "6421993" -> null
    }

  # github_team_repository.this["github-mgmt stewards:helia-delegated-routing-v1-http-api"] will be destroyed
  # (because key ["github-mgmt stewards:helia-delegated-routing-v1-http-api"] is not in for_each map)
  - resource "github_team_repository" "this" {
      - etag       = "W/\"1edabc0105149b60e1705c83875a1a5148ac59e1228f9d2204c628a5b9fef00e\"" -> null
      - id         = "6421993:helia-delegated-routing-v1-http-api" -> null
      - permission = "pull" -> null
      - repository = "helia-delegated-routing-v1-http-api" -> null
      - team_id    = "6421993" -> null
    }

  # github_team_repository.this["github-mgmt stewards:helia-http-gateway"] will be destroyed
  # (because key ["github-mgmt stewards:helia-http-gateway"] is not in for_each map)
  - resource "github_team_repository" "this" {
      - etag       = "W/\"bcbc6dfc45198118ef91bf94908bc7339fea5e2d9431b0b6356432520c960bdb\"" -> null
      - id         = "6421993:helia-http-gateway" -> null
      - permission = "pull" -> null
      - repository = "helia-http-gateway" -> null
      - team_id    = "6421993" -> null
    }

  # github_team_repository.this["github-mgmt stewards:helia-remote-pinning"] will be destroyed
  # (because key ["github-mgmt stewards:helia-remote-pinning"] is not in for_each map)
  - resource "github_team_repository" "this" {
      - etag       = "W/\"61edc647084946c4a3ab45282ed3ecb6ad360a4592aab81baaed7b541b3e372b\"" -> null
      - id         = "6421993:helia-remote-pinning" -> null
      - permission = "pull" -> null
      - repository = "helia-remote-pinning" -> null
      - team_id    = "6421993" -> null
    }

  # github_team_repository.this["github-mgmt stewards:helia-verified-fetch"] will be destroyed
  # (because key ["github-mgmt stewards:helia-verified-fetch"] is not in for_each map)
  - resource "github_team_repository" "this" {
      - etag       = "W/\"08c37e3237d21cbe90d6ebf5771090e135f6f70f803bf6b8c809fe44b623fa7e\"" -> null
      - id         = "6421993:helia-verified-fetch" -> null
      - permission = "pull" -> null
      - repository = "helia-verified-fetch" -> null
      - team_id    = "6421993" -> null
    }

  # github_team_repository.this["github-mgmt stewards:kubo"] will be destroyed
  # (because key ["github-mgmt stewards:kubo"] is not in for_each map)
  - resource "github_team_repository" "this" {
      - etag       = "W/\"e3e11c6518d1e078d0a9d60049efa1d9e5d4c4b2e369146701885a6de1d640eb\"" -> null
      - id         = "6421993:kubo" -> null
      - permission = "pull" -> null
      - repository = "kubo" -> null
      - team_id    = "6421993" -> null
    }

  # github_team_repository.this["github-mgmt stewards:rainbow"] will be destroyed
  # (because key ["github-mgmt stewards:rainbow"] is not in for_each map)
  - resource "github_team_repository" "this" {
      - etag       = "W/\"122c410f1f0150a54bb9bf6abba8782c5c91893419f7cc42cc853f17d6887dba\"" -> null
      - id         = "6421993:rainbow" -> null
      - permission = "pull" -> null
      - repository = "rainbow" -> null
      - team_id    = "6421993" -> null
    }

  # github_team_repository.this["github-mgmt stewards:service-worker-gateway"] will be destroyed
  # (because key ["github-mgmt stewards:service-worker-gateway"] is not in for_each map)
  - resource "github_team_repository" "this" {
      - etag       = "W/\"09461b95a98c75fd1c133155c390a4de016220ba04d37479f4d7b1b800346c59\"" -> null
      - id         = "6421993:service-worker-gateway" -> null
      - permission = "pull" -> null
      - repository = "service-worker-gateway" -> null
      - team_id    = "6421993" -> null
    }

  # github_team_repository.this["github-mgmt stewards:someguy"] will be destroyed
  # (because key ["github-mgmt stewards:someguy"] is not in for_each map)
  - resource "github_team_repository" "this" {
      - etag       = "W/\"886aed077dfd7f3787f493fcb838e01d4e3a6288984a4aef46c6d28087d84029\"" -> null
      - id         = "6421993:someguy" -> null
      - permission = "pull" -> null
      - repository = "someguy" -> null
      - team_id    = "6421993" -> null
    }

  # github_team_repository.this["helia-dev:service-worker-gateway"] will be updated in-place
  ~ resource "github_team_repository" "this" {
        id         = "7676419:service-worker-gateway"
      ~ permission = "pull" -> "admin"
        # (3 unchanged attributes hidden)
    }

  # github_team_repository.this["ipdx:kubo"] will be destroyed
  # (because key ["ipdx:kubo"] is not in for_each map)
  - resource "github_team_repository" "this" {
      - etag       = "W/\"49a7a25e4019d90314c1cd82d6cd9ce733735732d7f967c64cc597f4c9714e91\"" -> null
      - id         = "6349983:kubo" -> null
      - permission = "admin" -> null
      - repository = "kubo" -> null
      - team_id    = "6349983" -> null
    }

  # github_team_repository.this["maintainers:kubo"] will be destroyed
  # (because key ["maintainers:kubo"] is not in for_each map)
  - resource "github_team_repository" "this" {
      - etag       = "W/\"da952a7e722d57c27f9d588496a24d43d3af8f939637097f78c502f6d447d924\"" -> null
      - id         = "3729031:kubo" -> null
      - permission = "push" -> null
      - repository = "kubo" -> null
      - team_id    = "3729031" -> null
    }

  # github_team_repository.this["merge - go:boxo"] will be updated in-place
  ~ resource "github_team_repository" "this" {
        id         = "3364102:boxo"
      ~ permission = "maintain" -> "push"
        # (3 unchanged attributes hidden)
    }

  # github_team_repository.this["repos - go:boxo"] will be updated in-place
  ~ resource "github_team_repository" "this" {
        id         = "3232508:boxo"
      ~ permission = "maintain" -> "push"
        # (3 unchanged attributes hidden)
    }

  # github_team_repository.this["shipyard:boxo"] will be created
  + resource "github_team_repository" "this" {
      + etag       = (known after apply)
      + id         = (known after apply)
      + permission = "maintain"
      + repository = "boxo"
      + team_id    = (known after apply)
    }

  # github_team_repository.this["shipyard:helia"] will be created
  + resource "github_team_repository" "this" {
      + etag       = (known after apply)
      + id         = (known after apply)
      + permission = "maintain"
      + repository = "helia"
      + team_id    = (known after apply)
    }

  # github_team_repository.this["shipyard:helia-cli"] will be created
  + resource "github_team_repository" "this" {
      + etag       = (known after apply)
      + id         = (known after apply)
      + permission = "maintain"
      + repository = "helia-cli"
      + team_id    = (known after apply)
    }

  # github_team_repository.this["shipyard:helia-delegated-routing-v1-http-api"] will be created
  + resource "github_team_repository" "this" {
      + etag       = (known after apply)
      + id         = (known after apply)
      + permission = "maintain"
      + repository = "helia-delegated-routing-v1-http-api"
      + team_id    = (known after apply)
    }

  # github_team_repository.this["shipyard:helia-http-gateway"] will be created
  + resource "github_team_repository" "this" {
      + etag       = (known after apply)
      + id         = (known after apply)
      + permission = "maintain"
      + repository = "helia-http-gateway"
      + team_id    = (known after apply)
    }

  # github_team_repository.this["shipyard:helia-remote-pinning"] will be created
  + resource "github_team_repository" "this" {
      + etag       = (known after apply)
      + id         = (known after apply)
      + permission = "maintain"
      + repository = "helia-remote-pinning"
      + team_id    = (known after apply)
    }

  # github_team_repository.this["shipyard:helia-verified-fetch"] will be created
  + resource "github_team_repository" "this" {
      + etag       = (known after apply)
      + id         = (known after apply)
      + permission = "maintain"
      + repository = "helia-verified-fetch"
      + team_id    = (known after apply)
    }

  # github_team_repository.this["shipyard:kubo"] will be created
  + resource "github_team_repository" "this" {
      + etag       = (known after apply)
      + id         = (known after apply)
      + permission = "maintain"
      + repository = "kubo"
      + team_id    = (known after apply)
    }

  # github_team_repository.this["shipyard:rainbow"] will be created
  + resource "github_team_repository" "this" {
      + etag       = (known after apply)
      + id         = (known after apply)
      + permission = "maintain"
      + repository = "rainbow"
      + team_id    = (known after apply)
    }

  # github_team_repository.this["shipyard:service-worker-gateway"] will be created
  + resource "github_team_repository" "this" {
      + etag       = (known after apply)
      + id         = (known after apply)
      + permission = "admin"
      + repository = "service-worker-gateway"
      + team_id    = (known after apply)
    }

  # github_team_repository.this["shipyard:someguy"] will be created
  + resource "github_team_repository" "this" {
      + etag       = (known after apply)
      + id         = (known after apply)
      + permission = "maintain"
      + repository = "someguy"
      + team_id    = (known after apply)
    }

  # github_team_repository.this["w3dt-stewards:boxo"] will be destroyed
  # (because key ["w3dt-stewards:boxo"] is not in for_each map)
  - resource "github_team_repository" "this" {
      - etag       = "W/\"a18ec7fd3157202760a2eacfeeb481299b6541f3ad168266fa32468d6fb26728\"" -> null
      - id         = "4656983:boxo" -> null
      - permission = "admin" -> null
      - repository = "boxo" -> null
      - team_id    = "4656983" -> null
    }

  # github_team_repository.this["w3dt-stewards:helia"] will be destroyed
  # (because key ["w3dt-stewards:helia"] is not in for_each map)
  - resource "github_team_repository" "this" {
      - etag       = "W/\"b5a744a2916811afb57a448b089eaf0e4a1997ff56a84154094619ad3a5f3106\"" -> null
      - id         = "4656983:helia" -> null
      - permission = "admin" -> null
      - repository = "helia" -> null
      - team_id    = "4656983" -> null
    }

  # github_team_repository.this["w3dt-stewards:helia-delegated-routing-v1-http-api"] will be destroyed
  # (because key ["w3dt-stewards:helia-delegated-routing-v1-http-api"] is not in for_each map)
  - resource "github_team_repository" "this" {
      - etag       = "W/\"875182c667eeee3a49cd6c9fbfadf516730380015e3d4632f013a73de9ea90bd\"" -> null
      - id         = "4656983:helia-delegated-routing-v1-http-api" -> null
      - permission = "admin" -> null
      - repository = "helia-delegated-routing-v1-http-api" -> null
      - team_id    = "4656983" -> null
    }

  # github_team_repository.this["w3dt-stewards:helia-verified-fetch"] will be destroyed
  # (because key ["w3dt-stewards:helia-verified-fetch"] is not in for_each map)
  - resource "github_team_repository" "this" {
      - etag       = "W/\"d3ae40e40545231575ef601180ee4b514dbd32430eb72d0dc7d6421a78785289\"" -> null
      - id         = "4656983:helia-verified-fetch" -> null
      - permission = "admin" -> null
      - repository = "helia-verified-fetch" -> null
      - team_id    = "4656983" -> null
    }

  # github_team_repository.this["w3dt-stewards:kubo"] will be destroyed
  # (because key ["w3dt-stewards:kubo"] is not in for_each map)
  - resource "github_team_repository" "this" {
      - etag       = "W/\"49a7a25e4019d90314c1cd82d6cd9ce733735732d7f967c64cc597f4c9714e91\"" -> null
      - id         = "4656983:kubo" -> null
      - permission = "admin" -> null
      - repository = "kubo" -> null
      - team_id    = "4656983" -> null
    }

  # github_team_repository.this["w3dt-stewards:rainbow"] will be destroyed
  # (because key ["w3dt-stewards:rainbow"] is not in for_each map)
  - resource "github_team_repository" "this" {
      - etag       = "W/\"f8430965d7e7b9dc7d50ee009345a6159de9bd25e9cbcb0436f05477190aa314\"" -> null
      - id         = "4656983:rainbow" -> null
      - permission = "admin" -> null
      - repository = "rainbow" -> null
      - team_id    = "4656983" -> null
    }

  # github_team_repository.this["w3dt-stewards:someguy"] will be destroyed
  # (because key ["w3dt-stewards:someguy"] is not in for_each map)
  - resource "github_team_repository" "this" {
      - etag       = "W/\"1bf3d64fc07f1071099ea4bce87abaedd6d321bb09a8ba1808fcb351ddcb7cf3\"" -> null
      - id         = "4656983:someguy" -> null
      - permission = "admin" -> null
      - repository = "someguy" -> null
      - team_id    = "4656983" -> null
    }

Plan: 20 to add, 8 to change, 38 to destroy.

aschmahmann · 2024-08-20T18:59:01Z

github/ipfs.yml

    teams:
      admin:
        - helia-dev
-      pull:
-        - github-mgmt stewards
+      maintain:
+        - shipyard


@achingbrain @SgtPooki I added Shipyard as maintainers for some repos where there were no ambient admin permissions for the "admin" or w3dt-stewards teams. It might be that these are unnecessary or should just be push permissions. Happy to downgrade if you think that makes more sense.

this should be fine, and we haven't touched helia-cli in a while, and likely won't

aschmahmann · 2024-08-20T19:00:47Z

github/ipfs.yml

        push_restrictions:
-          - /aschmahmann
-          - /gmasgras
-          - /thattommyhall
+          - ipfs/kubo-maintainers


@ipfs/ipdx how can I set this to push restrictions on, but with no associated group? IIUC there's nothing you can do anyway to stop admins from pushing (or maintainers from pushing with approval) so adding groups here seems unnecessary provided you can keep the restrictions enabled.

aschmahmann · 2024-08-20T19:01:40Z

github/ipfs.yml

-      admin:
-        - lidel
-      push:
-        - dennis-tra


@dennis-tra I'm removing your permissions here and there will be a follow up PR to add you to the IPFS org and from there you can get added to any teams you need to be.

Sounds good, thanks for the heads-up! 👍

aschmahmann · 2024-08-20T19:04:18Z

github/ipfs.yml

-      pull:
-        - github-mgmt stewards


@ipfs/ipdx any idea why these pull permissions got added everywhere in fe64a02?

Yes! This is because github-mgmt stewards group is designated as moderator and security manager as per #189

So, unfortunately, they're going to come back, but you don't necessarily have to restore them yourself. The apply should go through anyway, and the config will be updated during the weekly sync.

aschmahmann · 2024-08-20T19:07:34Z

github/ipfs.yml

+      maintain:
+        - shipyard


@lidel WDYT about giving push to repos Go for rainbow and someguy?

Probably ok, that group already has "maintain" in boxo, so push will make it easier for existing community to submit PRs.

ps. there is a separate long-term meta-worry in that there is way too many people in https://github.com/orgs/ipfs/teams/repos-go, and if our intention is to limit security risks / access, we should plan to subset that group.

aschmahmann · 2024-08-20T19:08:53Z

github/ipfs.yml

@@ -5809,12 +5772,10 @@ repositories:
    squash_merge_commit_title: COMMIT_OR_PR_TITLE
    teams:
      admin:
-        - admin
        - ipdx


@ipfs/ipdx do you need admin here (recognizing the @galargh has org admin permissions anyway, but trying to understand what permissions are necessary for you to get your work done.

Nope, I think we can get rid of this one here.

Suggested change

- ipdx

SgtPooki

looks fine to me (only searched for my username and looked through removals or items where I was left). I really wish it was easier to see the specific repo the changes are for

SgtPooki · 2024-08-23T17:08:09Z

github/ipfs.yml

    teams:
      admin:
        - helia-dev
-      pull:
-        - github-mgmt stewards
+      maintain:
+        - shipyard


this should be fine, and we haven't touched helia-cli in a while, and likely won't

galargh

Great job!

galargh · 2024-09-02T07:39:42Z

github/ipfs.yml

-      pull:
-        - github-mgmt stewards


Yes! This is because github-mgmt stewards group is designated as moderator and security manager as per #189

So, unfortunately, they're going to come back, but you don't necessarily have to restore them yourself. The apply should go through anyway, and the config will be updated during the weekly sync.

galargh · 2024-09-02T07:41:35Z

github/ipfs.yml

@@ -5809,12 +5772,10 @@ repositories:
    squash_merge_commit_title: COMMIT_OR_PR_TITLE
    teams:
      admin:
-        - admin
        - ipdx


Nope, I think we can get rid of this one here.

Suggested change

- ipdx

SgtPooki · 2024-10-10T16:05:08Z

@aschmahmann should we merge this on monday?

aschmahmann added 14 commits August 20, 2024 14:34

permissions: add shipyard team

94bba16

permissions(kubo): switch to teams and remove more admins and unnecce…

49a84fc

…ssary teams

permissions(kubo): remove Jorropo from kubo maintainers team

592e521

permissions(boxo): remove most admins and downgrade merge and repos g…

bd1cecd

…o to push access

permissions(rainbow): reduce admins

fe7e606

permissions(someguy): reduce admins

265f480

permissions(helia-http-gateway): use teams, add shipyard as maintaine…

9f4cd1a

…rs, remove github-mgmt pull permissions

permissions(helia-dev): remove whizzzkid

277b1a1

permissions(helia): reduce admins, remove github-mgmt stewards pull p…

574faff

…ermissions

permissions(helia-service-worker-gateway): switch admins to teams, re…

faf4a43

…move github-mgmt stewards pull permissions

permissions(helia-cli): switch admins to teams, remove github-mgmt st…

25d5ddd

…ewards pull permissions, add shipyard as maintainers

permissions(helia-delegated-routing-v1-http-api): switch admins to te…

846063a

…ams, reduce admins, remove github-mgmt stewards pull permissions

permissions(helia-remote-pinning): add shipyard as maintainers, remov…

19b61c5

…e github-mgmt stewards pull permissions

permissions(helia-verified-fetch): reduce admins, remove github-mgmt …

d2bb753

…stewards pull permissions

aschmahmann requested review from a team as code owners August 20, 2024 18:43

aschmahmann requested a review from laurentsenta August 20, 2024 18:43

aschmahmann commented Aug 20, 2024

View reviewed changes

SgtPooki approved these changes Aug 23, 2024

View reviewed changes

ipfs-mgmt-read-write bot added 2 commits August 25, 2024 00:35

Merge branch 'master' into permissions/cleanup-shipyard-maintained-repos

1ee6821

Merge branch 'master' into permissions/cleanup-shipyard-maintained-repos

1e78c92

galargh approved these changes Sep 2, 2024

View reviewed changes

ipfs-mgmt-read-write bot added 4 commits September 8, 2024 00:29

Merge branch 'master' into permissions/cleanup-shipyard-maintained-repos

2107e7c

Merge branch 'master' into permissions/cleanup-shipyard-maintained-repos

aa41b23

Merge branch 'master' into permissions/cleanup-shipyard-maintained-repos

2f1308f

Merge branch 'master' into permissions/cleanup-shipyard-maintained-repos

2b2c2d7

Merge branch 'master' into permissions/cleanup-shipyard-maintained-repos

b9870df

ipfs-mgmt-read-write bot added 3 commits October 13, 2024 00:29

Merge branch 'master' into permissions/cleanup-shipyard-maintained-repos

9d09e4a

Merge branch 'master' into permissions/cleanup-shipyard-maintained-repos

0e65246

Merge branch 'master' into permissions/cleanup-shipyard-maintained-repos

5c1d96a

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Cleanup permissions for shipyard maintained repos #211

Cleanup permissions for shipyard maintained repos #211

aschmahmann commented Aug 20, 2024

github-actions bot commented Aug 20, 2024

github-actions bot commented Aug 20, 2024 •

edited

Loading

aschmahmann Aug 20, 2024

SgtPooki Aug 23, 2024

aschmahmann Aug 20, 2024

aschmahmann Aug 20, 2024

dennis-tra Aug 20, 2024

aschmahmann Aug 20, 2024

galargh Sep 2, 2024

aschmahmann Aug 20, 2024

lidel Aug 21, 2024 •

edited

Loading

aschmahmann Aug 20, 2024

galargh Sep 2, 2024

SgtPooki left a comment

SgtPooki Aug 23, 2024

galargh left a comment

galargh Sep 2, 2024

galargh Sep 2, 2024

SgtPooki commented Oct 10, 2024

Cleanup permissions for shipyard maintained repos #211

Are you sure you want to change the base?

Cleanup permissions for shipyard maintained repos #211

Conversation

aschmahmann commented Aug 20, 2024

Summary

Why do you need this?

What else do we need to know?

Reviewer's Checklist

github-actions bot commented Aug 20, 2024

github-actions bot commented Aug 20, 2024 • edited Loading

Terraform plans

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

lidel Aug 21, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

SgtPooki left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

galargh left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

SgtPooki commented Oct 10, 2024

github-actions bot commented Aug 20, 2024 •

edited

Loading

lidel Aug 21, 2024 •

edited

Loading