Skip to content

[processor/cumulativetodelta] Rename cumulativetodelta to cumulative_to_delta with deprecated alias cumulativetodelta#49487

Open
agarvin-nr wants to merge 7 commits into
open-telemetry:mainfrom
agarvin-nr:agarvin/cumulative-to-delta-rename
Open

[processor/cumulativetodelta] Rename cumulativetodelta to cumulative_to_delta with deprecated alias cumulativetodelta#49487
agarvin-nr wants to merge 7 commits into
open-telemetry:mainfrom
agarvin-nr:agarvin/cumulative-to-delta-rename

Conversation

@agarvin-nr

@agarvin-nr agarvin-nr commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Description

Renames the cumulativetodelta processor to cumulative_to_delta to standardize to lower_snake_case.

Link to tracking issue

#45339

Testing

  • Updates testdata/config.yaml and TestLoadConfig in config_test.go to load with new component name
  • Adds testdata/config-deprecated and TestLoadDeprecatedConfig inconfig_test.go to validate that the deprecated name still loads. Both test and config were copied from the original.
  • Ran make test and make lint

Documentation

README.md updated

Authorship

  • I, a human, wrote this pull request description myself.

@agarvin-nr agarvin-nr changed the title [processor/cumulativetodelta] Rename cumulativetodelta to cumulative_to_delta [processor/cumulativetodelta] Rename cumulativetodelta to cumulative_to_delta with deprecated alias cumulativetodelta Jul 6, 2026
change_type: bug_fix

component: processor/cumulativetodelta
component: processor/cumulative_to_delta

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I changed the component names in these preexisting changelog file entries because CI and make validate-chlog were failing, let me know if there's a different way I should have handled this!

@agarvin-nr agarvin-nr marked this pull request as ready for review July 6, 2026 23:16
@agarvin-nr agarvin-nr requested review from a team and TylerHelmuth as code owners July 6, 2026 23:16
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