-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
VerticalSplitClone no longer supports -min_rdonly_healthy_tablets #5433
Conversation
Can you fix the DCO? The Details link will tell you what to do. |
@chrisrgithub We can't approve this until the DCO is addressed. |
…lows -min_rdonly_tablets Signed-off-by: ChrisR <radsurfer@gmail.com>
6db4a1a
to
20be237
Compare
I believe this file has been added to the wrong place. It should be a change to the existing file of that name in examples/helm. |
I have modified this branch locally to correct the file placement. Will confer with Deepthi on how to re-apply and modify or restart this PR. |
Deepthi states that this PR will get updated correctly, and it appears so. |
…lows -min_rdonly_tablets (with corrected file location) Signed-off-by: ChrisR <radsurfer@gmail.com>
fe1e4bf
to
4a111dc
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
* Vitess tester workflow (#16127) Signed-off-by: Manan Gupta <manan@planetscale.com> * feat: fix test summary in workflow Signed-off-by: Manan Gupta <manan@planetscale.com> * feat: regenerate CI Signed-off-by: Manan Gupta <manan@planetscale.com> --------- Signed-off-by: Manan Gupta <manan@planetscale.com> Co-authored-by: Manan Gupta <35839558+GuptaManan100@users.noreply.github.com> Co-authored-by: Manan Gupta <manan@planetscale.com>
...but still allows -min_rdonly_tablets. Changed it to that.