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

[apm_retention_filters] Add resources and datasource #2137

Merged
merged 24 commits into from
Nov 20, 2023

Conversation

SalahEddineBC
Copy link
Contributor

@SalahEddineBC SalahEddineBC commented Oct 6, 2023

This PR adds terraform support to control APM retention filters.
The newly added resources are:

  • apm_retention_filter to maintain individual retention filters.
  • apm_retention_filter_order to maintain the execution order for retention filters.

The newly added datasource is:

  • apm_retention_filters_order to get the current order.

depends on DataDog/datadog-api-client-go#2172 to be released

@SalahEddineBC SalahEddineBC force-pushed the salah.bachircherif/add-apm-retention-filters-resources branch from 9de1a5c to 8b8e191 Compare November 9, 2023 09:03
@SalahEddineBC SalahEddineBC marked this pull request as ready for review November 10, 2023 12:47
@SalahEddineBC SalahEddineBC requested review from a team as code owners November 10, 2023 12:47
@SalahEddineBC SalahEddineBC changed the title Add APM retention filters resources and datasource [APM retention filters] Add resources and datasource Nov 10, 2023
@SalahEddineBC SalahEddineBC changed the title [APM retention filters] Add resources and datasource [apm_retention_filters] Add resources and datasource Nov 10, 2023
aliciascott
aliciascott previously approved these changes Nov 10, 2023
Copy link

@aliciascott aliciascott left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a couple of small punctuation feedback, otherwise approved.

docs/data-sources/apm_retention_filters_order.md Outdated Show resolved Hide resolved
docs/resources/apm_retention_filter_order.md Outdated Show resolved Hide resolved
aliciascott
aliciascott previously approved these changes Nov 13, 2023
Copy link

@aliciascott aliciascott left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good

@skarimo skarimo merged commit 6d872b6 into master Nov 20, 2023
8 checks passed
@skarimo skarimo deleted the salah.bachircherif/add-apm-retention-filters-resources branch November 20, 2023 18:49
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.

3 participants