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

GCP filestore restore procedure to also update terraform config #4657

Merged
merged 3 commits into from
Aug 22, 2024

Conversation

consideRatio
Copy link
Contributor

@consideRatio consideRatio commented Aug 22, 2024

The issue was that after restore via a web console, terraform will end up trying to recreate the filestore instance because it is trying to force source_backup to be null when it is after the web console restore referencing the backup it was restored from.

The fix in this PR is to update the docs to update the terraform config as well after having made a change in the web console. We could reasonably also do a filestore restore using terraform by setting source_backup directly, but that felt like adding complexity to a very sensitive operation. Due to that, I've instead just patched to set source_backup retroactively in terraform.

@consideRatio consideRatio merged commit 98ffccd into 2i2c-org:main Aug 22, 2024
4 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.

Handle terraform apply in 2i2c GCP cluster aka. pilot-hubs cluster
1 participant