vitessio / vitess

Vitess is a database clustering system for horizontal scaling of MySQL.
http://vitess.io
Apache License 2.0
18.7k stars 2.1k forks source link

e2e framework change to find empty throttler config #17228

Closed harshit-gangal closed 1 week ago

harshit-gangal commented 1 week ago

Description

This PR will stop emitted config not found logs from throttler for e2e test in vttablet.

Example:

throttler.go:592] Throttler.retryReadAndApplyThrottlerConfig(): error reading throttler config. Will retry in 10s. Err=node doesn't exist: keyspaces/vttest/SrvKeyspace

Related Issue(s)

Checklist

Deployment Notes

vitess-bot[bot] commented 1 week ago

Review Checklist

Hello reviewers! :wave: Please follow this checklist when reviewing this Pull Request.

General

Tests

Documentation

New flags

If a workflow is added or modified:

Backward compatibility

codecov[bot] commented 1 week ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 67.38%. Comparing base (f6067e0) to head (3c55db3). Report is 1 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #17228 +/- ## ========================================== - Coverage 67.40% 67.38% -0.02% ========================================== Files 1570 1570 Lines 252903 252903 ========================================== - Hits 170460 170410 -50 - Misses 82443 82493 +50 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.