Skip to content

Create ff-design team, give them push access to hyperspace - #198

Merged
ianconsolata merged 4 commits into
masterfrom
ianconsolata-patch-7
Mar 5, 2026
Merged

Create ff-design team, give them push access to hyperspace#198
ianconsolata merged 4 commits into
masterfrom
ianconsolata-patch-7

Conversation

@ianconsolata

Copy link
Copy Markdown
Contributor

Summary

Create ff-design team, give them push access to hyperspace

Why do you need this?

Design wants to work directly on frontend components in the repo, and we have branch protections setup to prevent them from merging directly to production without code review so there is no harm in giving them push access to create their own branches and PRs.

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

@ianconsolata
ianconsolata requested a review from a team as a code owner March 4, 2026 23:57
@github-actions

github-actions Bot commented Mar 4, 2026

Copy link
Copy Markdown
Contributor

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

Access Changes
User pwrepo:
  - will lose admin permission to filhyperspace

Comment thread github/filecoin-project.yml Outdated
Comment on lines +1894 to +1907
- ff-eng
push:
- ff-design
- pwrepo

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- pwrepo
- lanzafame
- pwrepo
has_discussions: false
merge_commit_message: PR_TITLE
merge_commit_title: MERGE_MESSAGE
secret_scanning_push_protection: false
secret_scanning: false
squash_merge_commit_message: COMMIT_MESSAGES
squash_merge_commit_title: COMMIT_OR_PR_TITLE
teams:
admin:
- ff-eng
push:
- ff-design

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pwreop is a collaborator, not a team

@BigLep

BigLep commented Mar 5, 2026

Copy link
Copy Markdown
Member

Hi Ian - I know you're adjusting permissions through github-mgmt. Feel free to do it directly in your repo too if that's easier for you. github mgmt will sync. I find github-mgmt great for resources that are more shared and mission critical for the ecosystem and want to give visibility or get checks on the changes.
Do as you wish, just wanted to make sure you feel freedom to not use github-mgmt for your team's repos.

@galargh

galargh commented Mar 5, 2026

Copy link
Copy Markdown
Contributor

FYI, unfortunately, team creation + assignment have to be split across 2 PRs. That's because we don't know the ID of a team until it actually exists and the current implementation needs it to handle team-repo assignments.

@ianconsolata

Copy link
Copy Markdown
Contributor Author

Sounds great! I can edit this PR to only do team creation (which I don't think I can do on my own), and then I will manually make the other changes in our repo directly. Good to know I don't need to go through this repo for internal repo changes

@github-actions

github-actions Bot commented Mar 5, 2026

Copy link
Copy Markdown
Contributor

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

Terraform plans

filecoin-project

Terraform used the selected providers to generate the following execution
plan. Resource actions are indicated with the following symbols:
  + create
  - destroy

Terraform will perform the following actions:

  # github_team.this["ff-design"] will be created
  + resource "github_team" "this" {
      + create_default_maintainer = false
      + etag                      = (known after apply)
      + id                        = (known after apply)
      + members_count             = (known after apply)
      + name                      = "ff-design"
      + node_id                   = (known after apply)
      + parent_team_read_id       = (known after apply)
      + parent_team_read_slug     = (known after apply)
      + privacy                   = "secret"
      + slug                      = (known after apply)
    }

  # github_team_membership.this["ff-design:filecoinfoundation-inf"] will be created
  + resource "github_team_membership" "this" {
      + etag     = (known after apply)
      + id       = (known after apply)
      + role     = "maintainer"
      + team_id  = (known after apply)
      + username = "filecoinfoundation-inf"
    }

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

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

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

  # github_team_membership.this["ff-eng:pwrepo"] will be destroyed
  # (because key ["ff-eng:pwrepo"] is not in for_each map)
  - resource "github_team_membership" "this" {
      - etag     = "W/\"0733cea500be0fa5763d728e1a68d19d28b06d0cc96a2c527cfdee27e788fa0f\"" -> null
      - id       = "16228647:pwrepo" -> null
      - role     = "member" -> null
      - team_id  = "16228647" -> null
      - username = "pwrepo" -> null
    }

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

@ianconsolata
ianconsolata merged commit 62167d8 into master Mar 5, 2026
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.

3 participants