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

API does not throw error while adding nodepool with AZ #751

Open
cpinjani opened this issue Dec 12, 2024 · 0 comments
Open

API does not throw error while adding nodepool with AZ #751

cpinjani opened this issue Dec 12, 2024 · 0 comments
Assignees
Labels
kind/bug Something isn't working
Milestone

Comments

@cpinjani
Copy link
Contributor

Setup
Rancher version: v2.10-c4944b3ca0b39c474121df9e7ef31f7aff4e4f7a-head
AKS Operator version: v1.10.1-rc.2

Description
Via UI, it is not possible to add nodepool with Availability Zone for a cluster created in region with no support for AZ. Updating this via API does not raise any error. It simply reverts the changes to AKSConfig after a few minutes.
The API should raise an error for the same.

Steps to reproduce:

  • Create a cluster in region without AZ support Eg. WestUS
  • Add nodepool via API and mention availability zones

Expected results:
The API must throw an error while adding nodepool with AZ in region where AZ are not supported.

Logs

time="2024-12-12T08:17:17Z" level=info msg="Checking configuration for cluster [cpinjani-akswest (id: c-4zdpb)]"
time="2024-12-12T08:17:18Z" level=info msg="Adding node pool [pool11] for cluster [cpinjani-akswest (id: c-4zdpb)]"
time="2024-12-12T08:17:24Z" level=info msg="Waiting for cluster [cpinjani-akswest (id: c-4zdpb)] to update node pool [pool11]"
time="2024-12-12T08:17:55Z" level=info msg="Checking configuration for cluster [cpinjani-akswest (id: c-4zdpb)]"
time="2024-12-12T08:17:56Z" level=info msg="Cluster [cpinjani-akswest (id: c-4zdpb)] finished updating"
time="2024-12-12T08:17:57Z" level=info msg="Checking configuration for cluster [cpinjani-akswest (id: c-4zdpb)]"
time="2024-12-12T08:17:59Z" level=info msg="Configuration for cluster [cpinjani-akswest (id: c-4zdpb)] was verified"
@cpinjani cpinjani added the kind/bug Something isn't working label Dec 12, 2024
@cpinjani cpinjani added this to the v2.10.2 milestone Dec 12, 2024
@mjura mjura self-assigned this Dec 24, 2024
@mjura mjura moved this from Backlog to In Progress (8 max) in CAPI & Hosted Kubernetes providers (EKS/AKS/GKE) Jan 3, 2025
mjura added a commit to mjura/rancher that referenced this issue Jan 13, 2025
mjura added a commit to mjura/rancher that referenced this issue Jan 14, 2025
mjura added a commit to mjura/rancher that referenced this issue Jan 14, 2025
mjura added a commit to mjura/rancher that referenced this issue Jan 15, 2025
mjura added a commit to mjura/rancher that referenced this issue Jan 15, 2025
mjura added a commit to mjura/rancher that referenced this issue Jan 15, 2025
mjura added a commit to mjura/rancher that referenced this issue Jan 15, 2025
mjura added a commit to mjura/aks-operator that referenced this issue Jan 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Something isn't working
Projects
Development

No branches or pull requests

2 participants