-
Notifications
You must be signed in to change notification settings - Fork 426
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
AzureCluster failureDomains do not always match specific control plane AzureMachine SKU #5033
Comments
@nojnhuh should this be in the next milestone? |
I don't think this is something we need to do soon, at least since we've ironed out the regions we use in CI to avoid this. |
/priority backlog |
The Kubernetes project currently lacks enough contributors to adequately respond to all issues. This bot triages un-triaged issues according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle stale |
/remove-lifecycle stale |
/kind bug
What steps did you take and what happened:
CAPZ's AzureCluster controller contains logic to set the
status.failureDomains
to the available availability zones for the region of the cluster resources:cluster-api-provider-azure/azure/services/resourceskus/cache.go
Line 170 in 89efbf3
This function loops through all VM SKUs, returning any availability zone without restrictions for at least one SKU to be set in the AzureCluster's failureDomains. But when a specific VM SKU in a region has a
Zone
restriction preventing VMs from being created in specific zones, CAPZ may still list those zones in itsstatus.failureDomains
if some other SKU in the same region does not have the same restrictions, causing VMs to be created in zones with restrictions and fail.What did you expect to happen:
The AzureCluster failureDomains match the availability of the specific SKU of control plane VM being used.
Anything else you would like to add:
[Miscellaneous information that will assist in solving the issue.]
Environment:
kubectl version
):/etc/os-release
):The text was updated successfully, but these errors were encountered: