Skip to content

[processor/interval]Fix incorrect exponential histogram test#36316

Merged
mx-psi merged 4 commits into
open-telemetry:mainfrom
lahsivjar:intervalprocessor-fix-tests
Nov 12, 2024
Merged

[processor/interval]Fix incorrect exponential histogram test#36316
mx-psi merged 4 commits into
open-telemetry:mainfrom
lahsivjar:intervalprocessor-fix-tests

Conversation

@lahsivjar
Copy link
Copy Markdown
Member

Description

Fixes the exponential histogram test to use exponential histogram as input rather than histogram.

Link to tracking issue

N/A

Testing

Run the unit tests using make test

Documentation

N/A

Comment thread processor/intervalprocessor/testdata/exp_histograms_are_aggregated/output.yaml Outdated
Copy link
Copy Markdown
Contributor

@RichieSams RichieSams left a comment

Choose a reason for hiding this comment

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

Nice catch! Thanks :)

@mx-psi mx-psi added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Nov 12, 2024
@mx-psi mx-psi merged commit f5c91d5 into open-telemetry:main Nov 12, 2024
@github-actions github-actions Bot added this to the next release milestone Nov 12, 2024
@lahsivjar lahsivjar deleted the intervalprocessor-fix-tests branch November 13, 2024 11:52
sbylica-splunk pushed a commit to sbylica-splunk/opentelemetry-collector-contrib that referenced this pull request Dec 17, 2024
…lemetry#36316)

<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description

Fixes the exponential histogram test to use exponential histogram as
input rather than histogram.

<!-- Issue number (e.g. open-telemetry#1234) or full URL to issue, if applicable. -->
#### Link to tracking issue
N/A

<!--Describe what testing was performed and which tests were added.-->
#### Testing
Run the unit tests using `make test`
<!--Describe the documentation added.-->
#### Documentation
N/A
<!--Please delete paragraphs that you did not use before submitting.-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

processor/interval Skip Changelog PRs that do not require a CHANGELOG.md entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants