-
Notifications
You must be signed in to change notification settings - Fork 519
Conversation
💖 Thanks for opening your first pull request! 💖 We use semantic commit messages to streamline the release process. Before your pull request can be merged, you should make sure your first commit and PR title start with a semantic prefix. Examples of commit messages with semantic prefixes: - |
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: CJJ69 If they are not already assigned, you can assign the PR to them by writing The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/assign @jackfrancis |
@PatrickLang can you take a look at this? |
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
@CJJ69 sorry this has been sitting so long--it needs a rebase now. @PatrickLang could you take a look at this Windows fix? |
This issue occurs if Windows vmss is specified as first pool in agentPoolProfiles. Then the variable for primaryScaleSetName does not match actual Windows ScaleSet ID. |
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
Reason for Change:
VM Scale Set name used is incorrect for Windows agent pool, cluster auto scaler fails.
Azure/acs-engine#4055
Issue Fixed:
Fixes #209
Requirements:
Notes:
Tested on Azure. Creating Windows agent pool with "availabilityProfile": "VirtualMachineScaleSets"