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

Add clintonb to django-commons #119

Merged
merged 6 commits into from
Nov 22, 2024
Merged

Add clintonb to django-commons #119

merged 6 commits into from
Nov 22, 2024

Conversation

cunla
Copy link
Member

@cunla cunla commented Nov 22, 2024

Fix #118

Copy link

github-actions bot commented Nov 22, 2024

Terraform plan in terraform
With var files: terraform/production/org.tfvars terraform/production/repositories.tfvars
With variables: github_token = (sensitive value)

Plan: 2 to add, 0 to change, 0 to destroy.
Terraform used the selected providers to generate the following execution
plan. Resource actions are indicated with the following symbols:
+   create

Terraform will perform the following actions:

  # github_membership.this["clintonb"] will be created
+   resource "github_membership" "this" {
+       downgrade_on_destroy = false
+       etag                 = (known after apply)
+       id                   = (known after apply)
+       role                 = "member"
+       username             = "clintonb"
    }

  # github_membership.this["jburns6789"] will be created
+   resource "github_membership" "this" {
+       downgrade_on_destroy = false
+       etag                 = (known after apply)
+       id                   = (known after apply)
+       role                 = "member"
+       username             = "jburns6789"
    }

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

📝 Plan generated in Plan org changes and list them in a PR #50

@tim-schilling
Copy link
Member

Is this plan valid?

@cunla
Copy link
Member Author

cunla commented Nov 22, 2024

Is this plan valid?

This is strange, not sure exactly what caused this.

@cunla
Copy link
Member Author

cunla commented Nov 22, 2024

What is even stranger is that running locally the plan shows different results. I think I have a different terraform version

@cunla
Copy link
Member Author

cunla commented Nov 22, 2024

Ok, looks better now!

@cunla cunla merged commit ca11ad1 into main Nov 22, 2024
2 checks passed
@cunla cunla deleted the add-user/118 branch November 22, 2024 17:44
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.

✋ [MEMBER] - clintonb
3 participants