Skip to content

fix Prometheus test iterator increment#2006

Merged
esigo merged 1 commit intoopen-telemetry:mainfrom
esigo:fix-1956
Mar 1, 2023
Merged

fix Prometheus test iterator increment#2006
esigo merged 1 commit intoopen-telemetry:mainfrom
esigo:fix-1956

Conversation

@esigo
Copy link
Copy Markdown
Member

@esigo esigo commented Feb 26, 2023

Fixes #1956 (issue)

Changes

avoid iterator increment when end reached.

For significant contributions please make sure you have completed the following items:

  • CHANGELOG.md updated for non-trivial changes
  • Unit tests have been added
  • Changes in public API reviewed

@esigo esigo requested a review from a team February 26, 2023 13:40
@esigo esigo added the size/XS Denotes a PR that changes 0-9 lines. label Feb 26, 2023
@codecov
Copy link
Copy Markdown

codecov bot commented Feb 26, 2023

Codecov Report

Merging #2006 (52866d9) into main (5c121cf) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #2006   +/-   ##
=======================================
  Coverage   87.29%   87.29%           
=======================================
  Files         166      166           
  Lines        4662     4662           
=======================================
  Hits         4069     4069           
  Misses        593      593           

Copy link
Copy Markdown
Member

@lalitb lalitb left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks for the fix,

@lalitb lalitb added the ok-to-merge The PR is ok to merge (has two approves or raised by a maintainer/approver and has one approve) label Feb 28, 2023
@esigo esigo merged commit 075f45d into open-telemetry:main Mar 1, 2023
@esigo esigo deleted the fix-1956 branch March 1, 2023 19:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ok-to-merge The PR is ok to merge (has two approves or raised by a maintainer/approver and has one approve) size/XS Denotes a PR that changes 0-9 lines.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

TranslateToPrometheusHistogramNormal test fails with "Assertion failed: cannot increment end list iterator"

2 participants