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.

Sign up for GitHub

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

Jump to bottom

Scaling down the cluster brakes the master endpoint #160

Closed
AMecea opened this issue Nov 22, 2018 · 1 comment
Closed

Scaling down the cluster brakes the master endpoint #160

AMecea opened this issue Nov 22, 2018 · 1 comment
Labels
bug
Milestone
0.2.x

Comments

@AMecea
Copy link
Contributor

AMecea commented Nov 22, 2018

If the cluster is scaled down from 3 nodes to 1, when the syncers are created it starts with the nodes that are not master, with the nodes that do not exist anymore. So the first syncer fails and will never run the master syncer. Check the code here.
@AMecea AMecea added the bug label Nov 22, 2018
@AMecea
Copy link
Contributor Author

AMecea commented Nov 22, 2018

Related with #158

AMecea added a commit that referenced this issue Nov 23, 2018
@AMecea
Pod syncers no found error should be skiped, add some tests, fixes #160
fa7444f
@AMecea AMecea added this to the 0.2.x milestone Nov 23, 2018
AMecea added a commit that referenced this issue Nov 26, 2018
@AMecea
Pod syncers no found error should be skiped, add some tests, fixes #160
a4d70f8
AMecea added a commit that referenced this issue Nov 27, 2018
@AMecea
Pod syncers no found error should be skiped, add some tests, fixes #160
2043fc7
@AMecea AMecea closed this as completed in 7e3c23d Nov 27, 2018
chapsuk pushed a commit to chapsuk/mysql-operator that referenced this issue Oct 16, 2023
@enisoc
Support longer entity names with more character choices. (bitpoke#160)
fd8d705 
* Support longer entity names with more character choices.

Cluster, keyspace, cell, and backup location names can now be longer and
can contain more characters.

- VitessCluster names can be up to 63 chars and must match
  the regex `^[a-z0-9]([a-z0-9-.]*[a-z0-9])?$`.
- Keyspace and backup location names can be anything allowed
  as a k8s label value: up to 63 chars, matching the regex
  `^[A-Za-z0-9]([A-Za-z0-9-_.]*[A-Za-z0-9])?$`.
- Cell names are similar, but cannot contain `-` because Vitess doesn't
  support `-` in cell names due to the tablet alias format.

Signed-off-by: Anthony Yeh <[email protected]>

* Explicitly check validity of MaxLength.

Signed-off-by: Anthony Yeh <[email protected]>

* Deprecate old Join functions without constraints.

Signed-off-by: Anthony Yeh <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
0.2.x
Development

No branches or pull requests
1 participant
@AMecea