-
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
Fix vtctldclient command in vtop test script and the fix operator example #12458
Conversation
Signed-off-by: Manan Gupta <manan@planetscale.com>
Review ChecklistHello reviewers! 👋 Please follow this checklist when reviewing this Pull Request. General
If a new flag is being introduced:
If a workflow is added or modified:
Bug fixes
Non-trivial changes
New/Existing features
Backward compatibility
|
Signed-off-by: deepthi <deepthi@planetscale.com>
Signed-off-by: Manan Gupta <manan@planetscale.com>
Signed-off-by: Manan Gupta <manan@planetscale.com>
Just as a note, the test in the script |
@@ -74,8 +74,35 @@ spec: | |||
- kind | |||
- name | |||
type: object | |||
x-kubernetes-map-type: atomic |
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.
for my knowledge.. Is it some auto generated file ? and what are all these changes?
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.
It is auto generated, but on vitess-operator. Whenever it changes, we have to copy them here, but I forgot to do that this time. 😭
Anyways these changes came from upgrading the controller-gen version from controller-gen.kubebuilder.io/version: v0.6.2
to controller-gen.kubebuilder.io/version: v0.11.3
@@ -252,7 +252,7 @@ function verifyVtadminSetup() { | |||
function verifyVTOrcSetup() { | |||
# Set the primary tablet to readOnly using the vtctld and wait for VTOrc to repair | |||
primaryTablet=$(getPrimaryTablet) | |||
vtctldclient SetReadOnly "$primaryTablet" | |||
vtctldclient SetWritable "$primaryTablet" false |
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.
Do we still have SetReadWrite command as well?
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.
No, we don't. In vtctldclient we only have SetWritable
.
In vtctlclient we have SetReadOnly
and SetReadWrite
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
Description
This PR fixes a
vtctldclient
command which was incorrectly pushed in the PR #12440 while fixing the issue #12438This PR also updates the operator.yaml file and removes
enforceSemiSync
property since it is deleted.These changes were missed in #12440 though they were included in #12457
Related Issue(s)
Checklist
Deployment Notes