Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Changing the mysql.innodb.lock_structs metric type to gauge #17452

Merged
merged 4 commits into from May 2, 2024

Conversation

aldrickdev
Copy link
Contributor

What does this PR do?

This PR will change the metric type of mysql.innodb.lock_structs to a gauge instead of rate.

Motivation

The motivation for this fix was a support case when the agent was generating the log below:

| CORE | INFO | (pkg/aggregator/check_sampler.go:141 in commitSeries) | No value returned for check metric 'mysql.innodb.lock_structs' on host '***' and tags '***': Rate value is negative, discarding it (the underlying counter may have been reset)

This caused the metric to never be reported.

Additional Notes

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Changelog entries must be created for modifications to shipped code
  • Add the qa/skip-qa label if the PR doesn't need to be tested during QA.
  • If you need to backport this PR to another branch, you can add the backport/<branch-name> label to the PR and it will automatically open a backport PR once this one is merged

Copy link

github-actions bot commented Apr 25, 2024

Test Results

   18 files     18 suites   17m 13s ⏱️
  146 tests   146 ✅  0 💤 0 ❌
1 296 runs  1 279 ✅ 17 💤 0 ❌

Results for commit 9e8f53d.

♻️ This comment has been updated with latest results.

@nenadnoveljic nenadnoveljic merged commit 0ee4a23 into master May 2, 2024
33 checks passed
@nenadnoveljic nenadnoveljic deleted the fixing-mysql.innodb.lock_structs-metric-type branch May 2, 2024 13:49
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.

None yet

6 participants