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

decreasing nats cluster size to 1 destroys the cluster #210

Closed
k15r opened this issue Nov 2, 2023 · 1 comment · Fixed by #230
Closed

decreasing nats cluster size to 1 destroys the cluster #210

k15r opened this issue Nov 2, 2023 · 1 comment · Fixed by #230
Assignees
@grischperl
Labels
kind/bug Categorizes issue or PR as related to a bug.
Milestone
1.0.3

Comments

@k15r
Copy link
Contributor

k15r commented Nov 2, 2023

Description

  • enable nats module
  • edit the eventing-nats-CR and change cluster size to 1
  • result: 1 not-ready eventing-nats-node (the configuration stored in the pvc still holds information that the node was started in clustered mode)

Mitigation

  • prevent changing the nats clustersize for an existing cluster to 1 (once it was !=1)
  • for new clusters a size of 1 is fine
@k15r k15r added the kind/bug Categorizes issue or PR as related to a bug. label Nov 2, 2023
@friedrichwilken
Copy link
Contributor

hint: check validation rules

@grischperl grischperl self-assigned this Nov 13, 2023
@grischperl grischperl mentioned this issue Nov 21, 2023
Merged
@grischperl grischperl linked a pull request Nov 21, 2023 that will close this issue
Add cluster size validation rule #230
Merged
@friedrichwilken friedrichwilken added this to the 1.0.3 milestone Jan 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@grischperl grischperl

Labels
kind/bug Categorizes issue or PR as related to a bug.
Projects
None yet
Milestone
1.0.3
Development

Successfully merging a pull request may close this issue.

Add cluster size validation rule grischperl/nats-manager
3 participants
@k15r @friedrichwilken @grischperl