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

[datadog_integration_gcp] Support enabling the CSPM integration #1748

Merged
merged 3 commits into from
Feb 6, 2023

Conversation

christophetd
Copy link
Contributor

@christophetd christophetd commented Feb 1, 2023

Context: While the API does support it, it's currently not possible to enable the GCP CSPM through Terraform. The AWS integration resource does have support for it: https://registry.terraform.io/providers/DataDog/datadog/latest/docs/resources/integration_aws#cspm_resource_collection_enabled

@christophetd christophetd requested review from a team as code owners February 1, 2023 10:43
@christophetd christophetd force-pushed the gcp-integration-support-enabling-cspm branch from ae6fa51 to 9741adc Compare February 1, 2023 10:48
@christophetd christophetd requested a review from a team as a code owner February 1, 2023 10:48
@christophetd christophetd changed the title Add support for enabling GCP CSPM through the 'cspm_resource_collection_enabled' attribute (closes #1743) [datadog_integration_gcp] Support enabling the CSPM integration Feb 1, 2023
@christophetd
Copy link
Contributor Author

(closes #1743)

@christophetd
Copy link
Contributor Author

👋🏼 Eager to get this PR reviewed, and will get it merged once the next release of the Go SDK lands next week

skarimo
skarimo previously approved these changes Feb 1, 2023
therve
therve previously approved these changes Feb 1, 2023
@christophetd
Copy link
Contributor Author

Thanks for the review! Will wait for the Go SDK release next week, update the go.mod and then merge it

rtrieu
rtrieu previously approved these changes Feb 1, 2023
@christophetd christophetd dismissed stale reviews from rtrieu and therve via 56d2819 February 6, 2023 13:07
@christophetd christophetd force-pushed the gcp-integration-support-enabling-cspm branch from d5b8851 to 56d2819 Compare February 6, 2023 13:07

Verified

This commit was signed with the committer’s verified signature.
christophetd Christophe Tafani-Dereeper
…on_enabled' attribute (closes #1743)
@christophetd christophetd force-pushed the gcp-integration-support-enabling-cspm branch from 56d2819 to bcea13c Compare February 6, 2023 13:08
@christophetd
Copy link
Contributor Author

Now rebased on latest master, let's wait for CI and should be good to merge

skarimo and others added 2 commits February 6, 2023 14:20

Verified

This commit was signed with the committer’s verified signature.
christophetd Christophe Tafani-Dereeper

Verified

This commit was signed with the committer’s verified signature.
christophetd Christophe Tafani-Dereeper
@christophetd christophetd force-pushed the gcp-integration-support-enabling-cspm branch from 6b52370 to d73ccef Compare February 6, 2023 13:21
@christophetd christophetd merged commit 005b744 into master Feb 6, 2023
@christophetd christophetd deleted the gcp-integration-support-enabling-cspm branch February 6, 2023 13:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants