-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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 google/image.go
mappings
#10401
Comments
What'd be backwards-incompatible to change, sorry? If there are just new product mappings, we could add them additively right? |
It would be great to do |
https://registry.terraform.io/modules/matti/compute-instance/google/latest
|
…ashicorp#10401) [upstream:c1e30ed89595203b0b2bf597b7f1cd597eed17e5] Signed-off-by: Modular Magician <magic-modules@google.com>
Affected Resource(s)
Community Note
modular-magician
user, it is either in the process of being autogenerated, or is planned to be autogenerated soon. If an issue is assigned to a user, that user is claiming responsibility for the issue. If an issue is assigned tohashibot
, a community member has claimed the issue already.While looking at #10368, I noticed that the image mappings at https://github.com/hashicorp/terraform-provider-google/blob/master/google/image.go#L35 need to be updated.
coreos
mapping is currently broken:https://cloud.redhat.com/learn/topics/coreos
https://cloud.google.com/compute/docs/images/os-details
@rileykarson Can we squeeze this change in to v4? I can submit it.
Thanks
The text was updated successfully, but these errors were encountered: