rancher / dashboard

The Rancher UI
https://rancher.com
Apache License 2.0
441 stars 243 forks source link

aks v2 UI - min and max count for "Enable Auto Scaling", should either have default values or should be made mandatory #3412

Open prabalsharma opened 2 years ago

prabalsharma commented 2 years ago

On enabling Auto Scaling in NPs, user might miss to set a min and max node count, which results in error

Error failed to create cluster: containerservice.ManagedClustersClient#CreateOrUpdate: Failure sending request: StatusCode=0 -- Original Error: Code="InvalidParameter" Message="Required parameter agentPoolProfile.minCount is missing (null)." Target="agentPoolProfile.minCount"

This can be avoided if there is a default value for min and max node counts or the field is made mandatory (when a user enables autoscaling) just like DNS Prefix and Cluster Resource Group .

Steps to repro:

  1. Docker install master-3e44415
  2. Create AKS cluster
  3. Select Enable Auto Scaling (no not fill min and max node count)
  4. Create
gaktive commented 2 years ago

@gaktive to check with backend if this can be populated; otherwise, we should default to Mandatory.

gaktive commented 2 years ago

For 2.6.0, this can be worked around. We should fix this in the next version, though this may require a change in UX since AKS has a conditional UI when autoscaling is enabled.

gaktive commented 1 year ago

We should look at this when we port the AKS page from Ember to Vue.