-
Notifications
You must be signed in to change notification settings - Fork 4
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
Terraform + Ansible Github Actions #1
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Terraform Run Output 🤖Format and Style 🖌
|
Pusher | @hellais |
Action | pull_request |
Working Directory | ./tf/environments/production |
Workflow | .github/workflows/check_deploy.yml |
Last updated | Tue, 13 Feb 2024 11:00:52 GMT |
hellais
changed the title
Terraform + Ansible github actions
Terraform + Ansible Github Actions
Feb 4, 2024
Ansible Run Output 🤖Ansible Playbook Recap 🔍
Ansible playbook output 📖
|
Pusher | @hellais |
Action | pull_request |
Working Directory | ./tf/environments/production |
Workflow | .github/workflows/check_deploy.yml |
Last updated | Tue, 13 Feb 2024 11:03:38 GMT |
hellais
commented
Feb 6, 2024
hellais
commented
Feb 6, 2024
hellais
commented
Feb 6, 2024
Disable on trigger Re-enable Add runs-on s3 tf backend region should not be a variable Fix naming of variable
When you try to do it out of the box, you get this error: ``` Initializing the backend... Migrating from Terraform Cloud to s3 state. ╷ │ Error: Migrating state from Terraform Cloud to another backend is not yet implemented. │ │ Please use the API to do this: https://www.terraform.io/docs/cloud/api/state-versions.html ``` The instructions on that page are not every helpful. Luckily this fine fellow on the internet tells us how to do it here: https://nedinthecloud.com/2022/03/03/migrating-state-data-off-terraform-cloud/ tl;dr is to do this: 1. `terraform init` with the cloud provider 2. `mkdir -p terraform.tfstate.d/tfc-migration-test` 3. `terraform state pull > terraform.tfstate.d/production-tier1/terraform.tfstate` 4. `mv .terraform/terraform.tfstate .terraform/terraform.tfstate.old` 5. Drop the cloud section of the terraform provider and replace it with s3 6. `terraform init`
Use terraform_state_backend module to create backend Add docs on how to use this Comment out event_name filter Fix commenting action hotifx Update to new github output pattern Run terraform fmt Update the last comment instead of creating a new one Fix GITHUB multiline output Also, tables are nice Fix editing of comment? Fix bugs
Fix push command ghetto hax
Fix path to ansible inventory file Fix working directory fix typo
* Rename deploy workflow
* 'tf-actions' of github.com:ooni/devops: auto: update oonidataapi package version to v0.2.0.dev1
bassosimone
reviewed
Feb 8, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM if provided suggestions are applied
Co-authored-by: Simone Basso <[email protected]>
Co-authored-by: Simone Basso <[email protected]>
Co-authored-by: Simone Basso <[email protected]>
Co-authored-by: Simone Basso <[email protected]>
Co-authored-by: Simone Basso <[email protected]>
* 'tf-actions' of github.com:ooni/devops: Update scripts/restore-dumps.sh
bassosimone
approved these changes
Feb 13, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🐳
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Specifically we have added support for the following: