Switch count method to monotonic count, that's correct for OM counts #13972

iliakur · 2023-02-15T20:36:23Z

This is a fix for a previous fix given that I learned the difference between count and monotonic_count methods 🙈

Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
PR title must be written as a CHANGELOG entry (see why)
Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
PR must have changelog/ and integration/ labels attached
If the PR doesn't need to be tested during QA, please add a qa/skip-qa label.

github-actions · 2023-02-15T20:38:43Z

The validations job has failed; please review the Files changed tab for possible suggestions to resolve.

codecov · 2023-02-15T20:47:27Z

Codecov Report

Merging #13972 (e310302) into master (6845bac) will decrease coverage by 0.03%.
The diff coverage is 100.00%.

Flag	Coverage Δ
amazon_msk	`89.00% <100.00%> (ø)`

Flags with carried forward coverage won't be shown. Click here to find out more.

github-actions · 2023-02-15T21:02:09Z

The validations job has failed; please review the Files changed tab for possible suggestions to resolve.

Switch count method to monotonic count, that's correct for OM counts

e310302

iliakur added the changelog/Fixed label Feb 15, 2023

iliakur requested a review from a team as a code owner February 15, 2023 20:36

ghost added the integration/amazon_msk label Feb 15, 2023

yzhan289 approved these changes Feb 15, 2023

View reviewed changes

yzhan289 merged commit b7a6f9c into master Feb 15, 2023

yzhan289 deleted the ik/amazon_msk/switch-count-to-monotonic-count branch February 15, 2023 22:04