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.

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

Bug: Changing provider of a nodepool leads to corrupt cluster #1344

Closed
Despire opened this issue Apr 23, 2024 · 1 comment · Fixed by #1371
Closed

Bug: Changing provider of a nodepool leads to corrupt cluster #1344

Despire opened this issue Apr 23, 2024 · 1 comment · Fixed by #1371
Assignees
Labels
bug Something isn't working good first issue Good for newcomers groomed Task that everybody agrees to pass the gatekeeper

Comments

@Despire
Copy link
Contributor

Despire commented Apr 23, 2024

Current Behaviour

    - name: aws-control-1
      providerSpec:
        name: aws-1
        region: eu-north-1
        zone: eu-north-1a
      count: 2
      serverType: t3.small
      #ubuntu
      image: ami-03df6dea56f8aa618

Changing the above provider aws-1 to aws-2 results in a corrupt cluster especially if the nodpool is used as a control plane.

Expected Behaviour

This edge-case should be handled correctly in claudie.

Steps To Reproduce

  1. Use a nodepool as a control-plane in a cluster
  2. change the provider of the nodepool
  3. watch the workflow fail in kube-eleven
@Despire Despire added the bug Something isn't working label Apr 23, 2024
@Despire
Copy link
Contributor Author

Despire commented Apr 26, 2024

add validation for guarding againts chaning the provider of a nodepool.

@Despire Despire added good first issue Good for newcomers groomed Task that everybody agrees to pass the gatekeeper labels Apr 26, 2024
@JKBGIT1 JKBGIT1 self-assigned this Apr 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers groomed Task that everybody agrees to pass the gatekeeper
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants