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

Update GCE VM Instance example #6415

Conversation

modular-magician
Copy link
Collaborator

This was changed in GoogleCloudPlatform/magic-modules#4113, but e2 vm instances can't use local ssds! This has been broken for years 🤯

service_account_id was also not a valid value, but changing the machine type broke it first. Added some slightly better prefilled values, although there are still numerous foobars kicking around.

As funny as updating a broken config with another broken config would be, works on my machine (tm):

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

Do you want to perform these actions?
  Terraform will perform the actions described above.
  Only 'yes' will be accepted to approve.

  Enter a value: yes

google_service_account.default: Creating...
google_service_account.default: Creation complete after 2s [id=projects/graphite-test-rileykarson/serviceAccounts/[email protected]]
google_compute_instance.default: Creating...
google_compute_instance.default: Still creating... [10s elapsed]
google_compute_instance.default: Creation complete after 13s [id=projects/graphite-test-rileykarson/zones/us-central1-a/instances/my-instance]

Apply complete! Resources: 2 added, 0 changed, 0 destroyed.

Release Note Template for Downstream PRs (will be copied)

Derived from GoogleCloudPlatform/magic-modules#9131

@modular-magician modular-magician merged commit 7e9f903 into hashicorp:main Sep 29, 2023
4 checks passed
@modular-magician modular-magician deleted the downstream-pr-c73536a11d6c7ab1ae6439967a2e10af183c9e34 branch November 16, 2024 04:59
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.

1 participant