Skip to content

obs-pipelines: remove rate config field in sample processors#3384

Merged
clementd-dd merged 1 commit intoop-gafrom
clementd-dd/obs-pipelines-remove-rate-config-field-in-sample-processors
Dec 23, 2025
Merged

obs-pipelines: remove rate config field in sample processors#3384
clementd-dd merged 1 commit intoop-gafrom
clementd-dd/obs-pipelines-remove-rate-config-field-in-sample-processors

Conversation

@clementd-dd
Copy link
Copy Markdown
Contributor

@clementd-dd clementd-dd commented Dec 22, 2025

companion PR to DataDog/datadog-api-client-go#3546

note: the go-client update commit is not included in this PR to avoid merges. Since the initial datadog-api-spec PR has been merged in the op-ga branch, syncing the op-ga datadog branch to the corresponding go client auto-generated PR should work, even if individual commits fail in the meantime

@clementd-dd clementd-dd force-pushed the clementd-dd/obs-pipelines-remove-rate-config-field-in-sample-processors branch from c4bb4c9 to db873ff Compare December 22, 2025 15:55
@clementd-dd clementd-dd changed the title Clementd dd/obs pipelines remove rate config field in sample processors obs-pipelines: remove rate config field in sample processors Dec 22, 2025
remove the deprecated field, in favor of alway using `percentage`, and make `percentage` required.
@clementd-dd clementd-dd force-pushed the clementd-dd/obs-pipelines-remove-rate-config-field-in-sample-processors branch from db873ff to b476504 Compare December 23, 2025 10:30
@clementd-dd clementd-dd changed the base branch from master to op-ga December 23, 2025 10:31
@clementd-dd clementd-dd marked this pull request as ready for review December 23, 2025 10:38
@clementd-dd clementd-dd requested review from a team as code owners December 23, 2025 10:38
@clementd-dd
Copy link
Copy Markdown
Contributor Author

tested locally with a go-client update

@clementd-dd clementd-dd merged commit d546290 into op-ga Dec 23, 2025
11 of 18 checks passed
@clementd-dd clementd-dd deleted the clementd-dd/obs-pipelines-remove-rate-config-field-in-sample-processors branch December 23, 2025 10:47
dd-mergequeue Bot pushed a commit that referenced this pull request Jan 15, 2026
…metrics pipelines (#3386)

[datadog_observability_pipeline] Add OpenTelemetry Source

open PR

obs-pipelines: remove `rate` config field in `sample` processors (#3384)

remove the deprecated field, in favor of alway using `percentage`, and make `percentage` required.

obs-pipelines: support for optional descriptions in SDS rule patterns (#3387)

Add add_hostname, parse_xml and split_array processors (#3395)

* Replace security monitoring rules with an updated detection method (#3392)

Replace security monitoring rules with an updated detection method

Co-authored-by: paul.howardflanders <[email protected]>

* Add add_hostname, parse_xml and split_array processors

---------

Co-authored-by: Paul Howard-Flanders <[email protected]>

Add metric pipelines (#3385)

* Add metric pipelines

add CloudPrem destination

add kafka destination

remove env var

Merge branch 'vladimir-dd/backfill' into op-ga

add too_many_buckets_option (#3405)

bump api client

[datadog_observability_pipeline] Add referencetable and datastream (#3404)

* Add support for reference_table in enrichment processor and datastream option in elastic search destination

* Update to ReferenceTable reference

* Clean up last of additional properties now that I can properly reference the types from the client

* Remove trailing new line to fix formatting issue

add include for rules + rename processors to processor_groups (#3410)

* Add include field for rules

* Rename processors to processor_groups

bump client

remove unrelated failing test

bump Go client

bump Go client

record cassettes

Merge branch 'master' into op-ga

update docs

make docs

fix typo

add OP team to codeowners

record cassettes

exclude cassettes from codeowners

Co-authored-by: 20agbekodo <[email protected]>
Co-authored-by: taylorchandleryoung <[email protected]>
Co-authored-by: clementd-dd <[email protected]>
Co-authored-by: vladimir.zhuk <[email protected]>
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.

1 participant