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

Defaulting to the GCP project specified in the ProviderConfig unless the optional spec.forProvider.project is given #275

Closed
ulucinar opened this issue Mar 30, 2023 · 0 comments · Fixed by #352
Labels
bug Something isn't working is:triaged

Comments

@ulucinar
Copy link
Collaborator

What happened?

How can we reproduce it?

We hit a bug, whose root cause was that we just ignore the spec.forProvider.provider optional field's value when generating the Terraform ID for a managed resource. For resources with an optional spec.forProvider.project field, the expected behavior is to use the project name if one is specified with the MR, and if not, use the ProviderConfig's project.

There are other managed resources with an optional spec.forProvider.project field. Those configured with the config.IdentifierFromProvider external-name configuration should be fine but we need to make sure others implement the defaulting logic correctly. An example could be this one and we need to check this and the others.

What environment did it happen in?

  • Provider Version: v0.29.0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working is:triaged
Projects
None yet
3 participants