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

DRE-8938 Add per workload TxThrottler metrics #110

Conversation

ejortegau
Copy link
Collaborator

Description

This is a backport of upstreamed vitessio#13526

Related Issue(s)

DRE-8938

Checklist

  • "Backport to:" labels have been added if this change should be back-ported
  • Tests were added or are not required
  • Did the new or modified tests pass consistently locally and on the CI
  • Documentation was added or is not required

Deployment Notes

N/A

This is a backport of upstreamed vitessio#13526

Signed-off-by: Eduardo J. Ortega U <5791035+ejortegau@users.noreply.github.com>
@ejortegau ejortegau marked this pull request as ready for review July 31, 2023 11:25
@ejortegau ejortegau requested a review from a team as a code owner July 31, 2023 11:25
@ejortegau
Copy link
Collaborator Author

Merging despite the failed tests as the problem is filing up disk in runners and has been happening since before this PR. See also https://slack-pde.slack.com/archives/C04NYAU82SC/p1690802801891229?thread_ts=1690798247.448699&cid=C04NYAU82SC.

@ejortegau
Copy link
Collaborator Author

OK, now all 3 failing checks are failing due to disk full, so mergin as is.

@ejortegau ejortegau merged commit 7203c35 into slack-vitess-r14.0.5-dsdefense Aug 1, 2023
235 of 238 checks passed
@ejortegau ejortegau deleted the eduardo.ortega/slack-vitess-r14.0.5-dsdefense-per-workload-txthrottler-metrics branch August 1, 2023 09:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants