[LesnyRumcajs]

Summary

@hanabi1224 -- (leaving today)
++ @EclesioMeloJunior (joined two weeks ago)

Why do you need this?

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

[github-actions]

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

Access Changes

User hanabi1224:
  - will lose push permission to bellperson
  - will lose push permission to builtin-actors
  - will lose push permission to fips
  - will lose push permission to fvm-test-vectors
  - will lose push permission to go-state-types
  - will lose maintain permission to ref-fvm
User eclesiomelojunior:
  - will join the organization as a [object Object] (remind them to accept the email invitation)
  - will gain push permission to bellperson
  - will gain push permission to builtin-actors
  - will gain push permission to fips
  - will gain push permission to fvm-test-vectors
  - will gain push permission to go-state-types
  - will gain maintain permission to ref-fvm

[galargh]

@rjan90 We're going to have to remove hanabi1224 from the org manually since GitHub as Code doesn't support it yet for security reasons. However, FYI, we should be rolling out support for safer-destructive actions very soon.

[rvagg]

@galargh what's the next steps here to make this happen?

[github-actions]

Before merge, verify that all the following plans are correct. After merge, Apply will regenerate the plans from the merged commit and continue only if they match.

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_membership.this["eclesiomelojunior"] will be created
  + resource "github_membership" "this" {
      + downgrade_on_destroy = false
      + etag                 = (known after apply)
      + id                   = (known after apply)
      + role                 = "member"
      + username             = "EclesioMeloJunior"
    }

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

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

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

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

[BigLep]

I kept hanabi1224 as a member for now. I put in the alumni team. @galargh will be doing cleanup soon to move folks as external collaborators (which should handle freeing up hanabi1224's membership seat. He doesn't have forest permissions). I'll merge this.