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

Treat _Default & _Required Sinks same as buckets #16513

Conversation

modular-magician
Copy link
Collaborator

Fixes terraform-provider-google#7811

The google_logging_project_bucket_config resource graciously handles the “_Default” bucket by acquiring and updating the resource that already exists at the desired location. This PR updates google_logging_project_sink to behave the same way.

Release Note Template for Downstream PRs (will be copied)

logging: `google_logging_project_sink` now will aqcuire and update the resource that already exists at the desired location. These buckets cannot be removed so deleting this resource will remove the bucket config from your terraform state but will leave the logging bucket unchanged.   

Derived from GoogleCloudPlatform/magic-modules#9384

* Treat _Default & _Required Sinks same as buckets

* Update documentation.

* Update exclusions every time.

* Do not check if _Default bucket has been deleted.

* Clean-up log statements.

* Fixup documentation.

* Ensure SA IAM set before associating with bucket.
[upstream:407631823fe91540101b6747b98f75eafdc3ada4]

Signed-off-by: Modular Magician <magic-modules@google.com>
@modular-magician modular-magician merged commit 7cc63df into hashicorp:main Nov 13, 2023
Copy link

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 14, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

google_logging_project_sink does not work for automatically created sinks
1 participant