-
Notifications
You must be signed in to change notification settings - Fork 156
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
Add enhancement proposal for controlling external seeds #1306
Add enhancement proposal for controlling external seeds #1306
Conversation
9522d50
to
6940d85
Compare
6940d85
to
137bd25
Compare
137bd25
to
7de178b
Compare
9a108d2
to
40203b8
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, thanks!
6739bde
to
b05fd7a
Compare
a176593
to
ef25174
Compare
ef25174
to
17574f8
Compare
/retest |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: rzetelskik, tnozicka The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@rzetelskik: The following test failed, say
Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
docs only, the e2e jobs are not triggered any more on these (but they have been triggered here before) |
@tnozicka: /override requires failed status contexts, check run or a prowjob name to operate on.
Only the following failed contexts/checkruns were expected:
If you are trying to override a checkrun that has a space in it, you must put a double quote on the context. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/override ci/prow/e2e-gke-serial |
@tnozicka: Overrode contexts on behalf of tnozicka: ci/prow/e2e-gke, ci/prow/e2e-gke-parallel, ci/prow/e2e-gke-serial In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
6ff7fa8
into
scylladb:master
Description of your changes:
Currently, ScyllaDB Operator does not provide any support, neither automated, nor manual, for controlling external seeds propagated to ScyllaDB cluster nodes.
Controlling the provision of external seeds is a prerequisite for supporting both manual and automated setup of multi datacenter ScyllaDB clusters, a feature long requested by many of its users, and as such is a vital step on our roadmap.
This PR proposes an extension to the existing ScyllaCluster API and corresponding changes to the ScyllaDB operator and sidecar.
Which issue is resolved by this Pull Request:
Resolves #1304