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

Cleanup GitHub-mgmt and Infra teams #83

Merged
merged 1 commit into from
Apr 3, 2024

Conversation

aschmahmann
Copy link
Contributor

Summary

Cleaning up the GitHub-mgmt and Infra teams.

Why do you need this?

Was noticing that there may be some extraneous permissions on repos and it seemed reasonable to clean them up:

  • GitHub Mgmt
    • @BigLep @guseggert don't seem likely to be merging PRs here in the near future and they're sensitive. Although always welcome back 😄
  • Infra
    • The Protocol Labs Infra doesn't manage anything in this group anymore (that's like the Interplanetary Shipyard Infra if anything). I've added @cewood as a placeholder/contact here, although my guess is that if it becomes necessary @ns4plabs will get added to the org and end up helping here).
    • @gmasgras @mcamou @thattommyhall are always welcome, and if there's some project you need permissions to help with just comment or raise a PR

What else do we need to know?

Nothing at the moment, I'm noticing there's a bunch of cleanup to be done but just taking it as I notice and have the time to file the PR.

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 review from a team as code owners April 3, 2024 19:29
Copy link
Contributor

github-actions bot commented Apr 3, 2024

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

Access Changes
User biglep:
  - will lose pull permission to ansible-ipfs
  - will lose pull permission to apt-on-ipfs
  - will lose pull permission to awesome-ipfs
  - will lose pull permission to clojars-mirror-test
  - will lose pull permission to cohosting
  - will lose pull permission to container-demos
  - will lose pull permission to contributors
  - will lose pull permission to contributors-hex-grid
  - will lose pull permission to cube
  - will lose pull permission to dapp-identity-api
  - will lose pull permission to dataviz
  - will lose pull permission to demo-ipfs-id-qr-codes
  - will lose pull permission to demo-ipfs-todo
  - will lose pull permission to discussify-browser-extension
  - will lose pull permission to discussify-styleguide
  - will lose pull permission to dnslink-cloudflare
  - will lose pull permission to dnslink-deploy
  - will lose pull permission to dnslink-dnsimple
  - will lose pull permission to durin
  - will lose pull permission to ecosystem-dashboard
  - will lose pull permission to ecosystem-directory
  - will lose pull permission to electron-mocha-main
  - will lose pull permission to empty-repository
  - will lose pull permission to equinix-billing-tools
  - will lose pull permission to experiments.peerpad.net
  - will lose pull permission to farcaster-funding-frame
  - will lose pull permission to get-gh-contributors
  - will lose pull permission to git-remote-ipld
  - will lose push permission to github-mgmt
  - will lose pull permission to go-ds-gcs
  - will lose pull permission to go-ds-remote-ipfs
  - will lose pull permission to go-dumpotron
  - will lose pull permission to go-ipfs-desktop
  - will lose pull permission to go-ipfs-docker-examples
  - will lose pull permission to go-libp2p-kitsune
  - will lose pull permission to gomobile-ipfs
  - will lose pull permission to gutenberg-to-ipfs
  - will lose pull permission to helia-contributors
  - will lose pull permission to helia-css
  - will lose pull permission to hubot-ipfs
  - will lose pull permission to i18n
  - will lose pull permission to igis
  - will lose pull permission to ignite-metrics
  - will lose pull permission to integration-mini-projects
  - will lose pull permission to ipfs-blob-store
  - will lose pull permission to ipfs-block-party
  - will lose pull permission to ipfs-camp-2022
  - will lose pull permission to ipfs-check
  - will lose pull permission to ipfs-cohost
  - will lose pull permission to ipfs-counter
  - will lose pull permission to ipfs-css
  - will lose pull permission to ipfs-dag-builder-vis
  - will lose pull permission to ipfs-deploy
  - will lose pull permission to ipfs-dns-deploy
  - will lose pull permission to ipfs-geoip
  - will lose pull permission to ipfs-github-action
  - will lose pull permission to ipfs-glam-community
  - will lose pull permission to ipfs-hubot
  - will lose pull permission to ipfs-iiif-db
  - will lose pull permission to ipfs-live-db
  - will lose pull permission to ipfs-npm-old
  - will lose pull permission to ipfs-npm-registry-mirror
  - will lose pull permission to ipfs-npm-republish
  - will lose pull permission to ipfs-or-gateway
  - will lose pull permission to ipfs-primer
  - will lose pull permission to ipfs-protocol-compliance-suite
  - will lose pull permission to ipfs-provider
  - will lose pull permission to ipfs-pubsub-1on1
  - will lose pull permission to ipfs-pubsub-peer-monitor
  - will lose pull permission to ipfs-pubsub-room
  - will lose pull permission to ipfs-pubsub-room-demo
  - will lose pull permission to ipfs-rsync
  - will lose pull permission to ipfs-senc
  - will lose pull permission to ipfs-service-worker-demos
  - will lose pull permission to ipfs-share-files
  - will lose pull permission to ipfs-snap
  - will lose pull permission to ipfs-thing-2022
  - will lose pull permission to ipfs-thing-2023
  - will lose pull permission to ipfs_dht_explorer
  - will lose pull permission to ipfsio-about
  - will lose pull permission to ipfsx
  - will lose pull permission to ipscend
  - will lose pull permission to is-ipfs
  - will lose pull permission to java-ipfs-http-client
  - will lose pull permission to js-crypto-key-composer
  - will lose pull permission to js-did-ipid
  - will lose pull permission to js-dnslink-dnsimple
  - will lose pull permission to js-human-crypto-keys
  - will lose pull permission to js-idm-bridge-postmsg
  - will lose pull permission to js-idm-client
  - will lose pull permission to js-idm-signatures
  - will lose pull permission to js-idm-wallet
  - will lose pull permission to js-ipfs-contributors
  - will lose pull permission to js-ipfs-level
  - will lose pull permission to js-ipfs-versidag
  - will lose pull permission to js-ipid-did-resolver
  - will lose pull permission to js-ipld-fixtures
  - will lose pull permission to js-libp2p-react-native
  - will lose pull permission to js-libp2p-transfer-performance
  - will lose pull permission to js-mock-ipfs-pinning-service
  - will lose pull permission to js-pinning-service-http-client
  - will lose pull permission to js-streams-benchmark
  - will lose pull permission to js-versidag
  - will lose pull permission to kipster
  - will lose pull permission to net-ipfs-core
  - will lose pull permission to net-ipfs-http-client
  - will lose pull permission to nft-school-examples
  - will lose pull permission to nomios-web
  - will lose pull permission to nomios-web-uikit
  - will lose pull permission to nomios.io
  - will lose pull permission to nopfs
  - will lose pull permission to npm-ipfs-registry-fetch
  - will lose pull permission to npm-on-ipfs
  - will lose pull permission to nuxt-module-ecosystem-directory
  - will lose pull permission to package-table
  - will lose pull permission to paperhub
  - will lose pull permission to peer-blog
  - will lose pull permission to peer-cacl
  - will lose pull permission to peer-crdt
  - will lose pull permission to peer-crdt-bind-codemirror
  - will lose pull permission to peer-crdt-example
  - will lose pull permission to peer-crdt-ipfs
  - will lose pull permission to peer-crdt-platform
  - will lose pull permission to peer-crdt-textarea-binding
  - will lose pull permission to peer-identity
  - will lose pull permission to peer-identity-demo
  - will lose pull permission to peer-pad-core
  - will lose pull permission to peer-star-infra
  - will lose pull permission to peer-star-peer-color
  - will lose pull permission to peer-star-react
  - will lose pull permission to peerpad-peer-crdt
  - will lose pull permission to pinning-service-compliance
  - will lose pull permission to pm-discussify
  - will lose pull permission to pm-idm
  - will lose pull permission to pm-peer-pad
  - will lose pull permission to protons-gossipsub-benchmark
  - will lose pull permission to py-datastore
  - will lose pull permission to py-datastore.leveldb
  - will lose pull permission to py-ipfs
  - will lose pull permission to py-ipfs-http-client
  - will lose pull permission to rb-pinning-service-api
  - will lose pull permission to react-idm-wallet
  - will lose pull permission to react-ipfs-url
  - will lose pull permission to react-native-ipfs-demo
  - will lose pull permission to release-please-ipfs-plugin
  - will lose pull permission to scala-ipfs-api
  - will lose pull permission to service-worker-gateway
  - will lose pull permission to shared-editing-demo
  - will lose pull permission to shipyard
  - will lose pull permission to simple-peer
  - will lose pull permission to space
  - will lose pull permission to swift-ipfs-http-client
  - will lose pull permission to tevere
  - will lose pull permission to varint-decoder
  - will lose pull permission to vole
  - will lose pull permission to w3rc
  - will lose pull permission to workshop-idm-chat-dapp
  - will lose pull permission to workshop-todo-dapp
  - will lose pull permission to www-helia-identify
  - will lose pull permission to www-helia-io
  - will lose pull permission to www-libp2p-webtransport-sessions
  - will lose pull permission to www-webtransport-pending-sessions
  - will lose pull permission to y-ipfs-connector
  - will lose pull permission to zarr-examples
User cewood:
  - will have the permission to go-dumpotron change from pull to admin
  - will have the permission to go-libp2p-kitsune change from pull to admin
  - will have the permission to ipfs-primer change from pull to push
  - will have the permission to ipfs-snap change from pull to admin
  - will have the permission to peer-star-infra change from pull to admin
User gmasgras:
  - will lose admin permission to go-dumpotron
  - will lose admin permission to go-libp2p-kitsune
  - will lose pull permission to ipfs-check
  - will lose push permission to ipfs-primer
  - will lose admin permission to ipfs-snap
  - will lose pull permission to ipfsio-about
  - will lose pull permission to nopfs
  - will lose admin permission to peer-star-infra
User guseggert:
  - will lose pull permission to ansible-ipfs
  - will lose pull permission to apt-on-ipfs
  - will lose pull permission to awesome-ipfs
  - will lose pull permission to clojars-mirror-test
  - will lose pull permission to cohosting
  - will lose pull permission to container-demos
  - will lose pull permission to contributors
  - will lose pull permission to contributors-hex-grid
  - will lose pull permission to cube
  - will lose pull permission to dapp-identity-api
  - will lose pull permission to dataviz
  - will lose pull permission to demo-ipfs-id-qr-codes
  - will lose pull permission to demo-ipfs-todo
  - will lose pull permission to discussify-browser-extension
  - will lose pull permission to discussify-styleguide
  - will lose pull permission to dnslink-cloudflare
  - will lose pull permission to dnslink-deploy
  - will lose pull permission to dnslink-dnsimple
  - will lose pull permission to durin
  - will lose pull permission to ecosystem-dashboard
  - will lose pull permission to ecosystem-directory
  - will lose pull permission to electron-mocha-main
  - will lose pull permission to empty-repository
  - will lose pull permission to equinix-billing-tools
  - will lose pull permission to experiments.peerpad.net
  - will lose pull permission to farcaster-funding-frame
  - will lose pull permission to get-gh-contributors
  - will lose pull permission to git-remote-ipld
  - will lose push permission to github-mgmt
  - will lose pull permission to go-ds-gcs
  - will lose pull permission to go-ds-remote-ipfs
  - will lose pull permission to go-dumpotron
  - will lose pull permission to go-ipfs-desktop
  - will lose pull permission to go-ipfs-docker-examples
  - will lose pull permission to go-libp2p-kitsune
  - will lose pull permission to gomobile-ipfs
  - will lose pull permission to gutenberg-to-ipfs
  - will lose pull permission to helia-contributors
  - will lose pull permission to helia-css
  - will lose pull permission to hubot-ipfs
  - will lose pull permission to i18n
  - will lose pull permission to igis
  - will lose pull permission to ignite-metrics
  - will lose pull permission to integration-mini-projects
  - will lose pull permission to ipfs-blob-store
  - will lose pull permission to ipfs-block-party
  - will lose pull permission to ipfs-camp-2022
  - will lose pull permission to ipfs-check
  - will lose pull permission to ipfs-cohost
  - will lose pull permission to ipfs-counter
  - will lose pull permission to ipfs-css
  - will lose pull permission to ipfs-dag-builder-vis
  - will lose pull permission to ipfs-deploy
  - will lose pull permission to ipfs-dns-deploy
  - will lose pull permission to ipfs-geoip
  - will lose pull permission to ipfs-github-action
  - will lose pull permission to ipfs-glam-community
  - will lose pull permission to ipfs-hubot
  - will lose pull permission to ipfs-iiif-db
  - will lose pull permission to ipfs-live-db
  - will lose pull permission to ipfs-npm-old
  - will lose pull permission to ipfs-npm-registry-mirror
  - will lose pull permission to ipfs-npm-republish
  - will lose pull permission to ipfs-or-gateway
  - will lose pull permission to ipfs-primer
  - will lose pull permission to ipfs-protocol-compliance-suite
  - will lose pull permission to ipfs-provider
  - will lose pull permission to ipfs-pubsub-1on1
  - will lose pull permission to ipfs-pubsub-peer-monitor
  - will lose pull permission to ipfs-pubsub-room
  - will lose pull permission to ipfs-pubsub-room-demo
  - will lose pull permission to ipfs-rsync
  - will lose pull permission to ipfs-senc
  - will lose pull permission to ipfs-service-worker-demos
  - will lose pull permission to ipfs-share-files
  - will lose pull permission to ipfs-snap
  - will lose pull permission to ipfs-thing-2022
  - will lose pull permission to ipfs-thing-2023
  - will lose pull permission to ipfs_dht_explorer
  - will lose pull permission to ipfsio-about
  - will lose pull permission to ipfsx
  - will lose pull permission to ipscend
  - will lose pull permission to is-ipfs
  - will lose pull permission to java-ipfs-http-client
  - will lose pull permission to js-crypto-key-composer
  - will lose pull permission to js-did-ipid
  - will lose pull permission to js-dnslink-dnsimple
  - will lose pull permission to js-human-crypto-keys
  - will lose pull permission to js-idm-bridge-postmsg
  - will lose pull permission to js-idm-client
  - will lose pull permission to js-idm-signatures
  - will lose pull permission to js-idm-wallet
  - will lose pull permission to js-ipfs-contributors
  - will lose pull permission to js-ipfs-level
  - will lose pull permission to js-ipfs-versidag
  - will lose pull permission to js-ipid-did-resolver
  - will lose pull permission to js-ipld-fixtures
  - will lose pull permission to js-libp2p-react-native
  - will lose pull permission to js-libp2p-transfer-performance
  - will lose pull permission to js-mock-ipfs-pinning-service
  - will lose pull permission to js-pinning-service-http-client
  - will lose pull permission to js-streams-benchmark
  - will lose pull permission to js-versidag
  - will lose pull permission to kipster
  - will lose pull permission to net-ipfs-core
  - will lose pull permission to net-ipfs-http-client
  - will lose pull permission to nft-school-examples
  - will lose pull permission to nomios-web
  - will lose pull permission to nomios-web-uikit
  - will lose pull permission to nomios.io
  - will lose pull permission to nopfs
  - will lose pull permission to npm-ipfs-registry-fetch
  - will lose pull permission to npm-on-ipfs
  - will lose pull permission to nuxt-module-ecosystem-directory
  - will lose pull permission to package-table
  - will lose pull permission to paperhub
  - will lose pull permission to peer-blog
  - will lose pull permission to peer-cacl
  - will lose pull permission to peer-crdt
  - will lose pull permission to peer-crdt-bind-codemirror
  - will lose pull permission to peer-crdt-example
  - will lose pull permission to peer-crdt-ipfs
  - will lose pull permission to peer-crdt-platform
  - will lose pull permission to peer-crdt-textarea-binding
  - will lose pull permission to peer-identity
  - will lose pull permission to peer-identity-demo
  - will lose pull permission to peer-pad-core
  - will lose pull permission to peer-star-infra
  - will lose pull permission to peer-star-peer-color
  - will lose pull permission to peer-star-react
  - will lose pull permission to peerpad-peer-crdt
  - will lose pull permission to pinning-service-compliance
  - will lose pull permission to pm-discussify
  - will lose pull permission to pm-idm
  - will lose pull permission to pm-peer-pad
  - will lose pull permission to protons-gossipsub-benchmark
  - will lose pull permission to py-datastore
  - will lose pull permission to py-datastore.leveldb
  - will lose pull permission to py-ipfs
  - will lose pull permission to py-ipfs-http-client
  - will lose pull permission to rb-pinning-service-api
  - will lose pull permission to react-idm-wallet
  - will lose pull permission to react-ipfs-url
  - will lose pull permission to react-native-ipfs-demo
  - will lose pull permission to release-please-ipfs-plugin
  - will lose pull permission to scala-ipfs-api
  - will lose pull permission to service-worker-gateway
  - will lose pull permission to shared-editing-demo
  - will lose pull permission to shipyard
  - will lose pull permission to simple-peer
  - will lose pull permission to space
  - will lose pull permission to swift-ipfs-http-client
  - will lose pull permission to tevere
  - will lose pull permission to varint-decoder
  - will lose pull permission to vole
  - will lose pull permission to w3rc
  - will lose pull permission to workshop-idm-chat-dapp
  - will lose pull permission to workshop-todo-dapp
  - will lose pull permission to www-helia-identify
  - will lose pull permission to www-helia-io
  - will lose pull permission to www-libp2p-webtransport-sessions
  - will lose pull permission to www-webtransport-pending-sessions
  - will lose pull permission to y-ipfs-connector
  - will lose pull permission to zarr-examples
User mcamou:
  - will lose admin permission to go-dumpotron
  - will lose push permission to ipfs-primer
  - will lose admin permission to ipfs-snap
  - will lose pull permission to ipfsio-about
  - will lose pull permission to nopfs
  - will lose admin permission to peer-star-infra
User thattommyhall:
  - will lose admin permission to go-dumpotron
  - will lose admin permission to go-libp2p-kitsune
  - will lose pull permission to ipfs-check
  - will lose push permission to ipfs-primer
  - will lose admin permission to ipfs-snap
  - will lose pull permission to ipfsio-about
  - will lose pull permission to nopfs
  - will lose admin permission to peer-star-infra

Copy link
Contributor

github-actions bot commented Apr 3, 2024

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

Terraform plans

ipfs-shipyard

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_team.this["infra"] will be updated in-place
  ~ resource "github_team" "this" {
      + create_default_maintainer = false
      ~ description               = "Protocol Labs Infrastructure Team" -> "Org Infrastructure"
        id                        = "3031417"
        name                      = "Infra"
        # (5 unchanged attributes hidden)
    }

  # github_team_membership.this["github-mgmt stewards:biglep"] will be destroyed
  # (because key ["github-mgmt stewards:biglep"] is not in for_each map)
  - resource "github_team_membership" "this" {
      - etag     = "W/\"d17320b4fc6b90a2c9666e2da8d4a6f2b6fd818ab0093dcbf3ad7f39ec6f47ce\"" -> null
      - id       = "6621987:BigLep" -> null
      - role     = "member" -> null
      - team_id  = "6621987" -> null
      - username = "BigLep" -> null
    }

  # github_team_membership.this["github-mgmt stewards:guseggert"] will be destroyed
  # (because key ["github-mgmt stewards:guseggert"] is not in for_each map)
  - resource "github_team_membership" "this" {
      - etag     = "W/\"59cc4e465fa50faebc567aa80277ddf7bed06ec5a554c6cfe638bd77d8d95d1b\"" -> null
      - id       = "6621987:guseggert" -> null
      - role     = "member" -> null
      - team_id  = "6621987" -> null
      - username = "guseggert" -> null
    }

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

  # github_team_membership.this["infra:gmasgras"] will be destroyed
  # (because key ["infra:gmasgras"] is not in for_each map)
  - resource "github_team_membership" "this" {
      - etag     = "W/\"3f55c2d61234221820089bfd21648bc525e05cd4743ad7818ac0dfe3436972a8\"" -> null
      - id       = "3031417:gmasgras" -> null
      - role     = "member" -> null
      - team_id  = "3031417" -> null
      - username = "gmasgras" -> null
    }

  # github_team_membership.this["infra:mcamou"] will be destroyed
  # (because key ["infra:mcamou"] is not in for_each map)
  - resource "github_team_membership" "this" {
      - etag     = "W/\"fd02926da8365bb1778ccfb992d79519bcc41092b58ed2139b29f30bddfeae2f\"" -> null
      - id       = "3031417:mcamou" -> null
      - role     = "member" -> null
      - team_id  = "3031417" -> null
      - username = "mcamou" -> null
    }

  # github_team_membership.this["infra:thattommyhall"] will be destroyed
  # (because key ["infra:thattommyhall"] is not in for_each map)
  - resource "github_team_membership" "this" {
      - etag     = "W/\"01972b9570b2c9c3026d64ca309f3f26d2e9ca52131152820fc54fbfa002fb12\"" -> null
      - id       = "3031417:thattommyhall" -> null
      - role     = "member" -> null
      - team_id  = "3031417" -> null
      - username = "thattommyhall" -> null
    }

Plan: 1 to add, 1 to change, 5 to destroy.

@aschmahmann aschmahmann merged commit 0fefc7a into master Apr 3, 2024
6 checks passed
@aschmahmann aschmahmann deleted the chore/cleanup-mgmtadmin-and-infra branch April 3, 2024 19:48
@thattommyhall
Copy link
Member

Thanks for the @ @aschmahmann, take care mate

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.

3 participants