Deepthi Sigireddi

While v16 is no longer supported, it is fine for us to merge PRs on to the release branch to ease the process of cherry-picking for the affected user. This...

@GuptaManan100 wdyt? Should we be fixing CheckMySQL instead?

Changing local examples affects all contributors. I'd want to hear from a majority of the maintainer team before we decide whether to accept or reject this change.

> I don't have anything against this, but can you list out what you see as the practical advantages this offers @morgo? Moving the binary flags to config files is...

@arthurschreiber you might need to fix this up because of https://github.com/vitessio/vitess/pull/17278

Is there a dynamic flag test for vtorc? Worth adding this to that?

> we do a single GET for each tablet from the topo, even though we are getting tablets that are not part of the shard the current tablet is in....

The maintainer team does not plan to provide helm charts. You might get more responses in terms of interest in a community built helm chart by posting in slack (#kubernetes...

@joekelley I'll get this re-reviewed, but in the meantime, can you fix the failing DCO check? We won't be able to merge without that. Once that is fixed, we can...

Looks like there isn't a linked issue. That is something we require for anything other than trivial doc changes or code cleanup type of PRs. It will be good if...