Skip to content
This repository has been archived by the owner on Oct 24, 2023. It is now read-only.

fix: primary scale set name #321

Closed
wants to merge 1 commit into from

Conversation

CJJ69
Copy link
Contributor

@CJJ69 CJJ69 commented Jan 15, 2019

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"

@welcome
Copy link

welcome bot commented Jan 15, 2019

💖 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: - fix: change azure disk cachingMode to ReadOnly - feat: make maximumLoadBalancerRuleCount configurable - docs: add note on AKS-Engine and AKS relationship
Make sure to check out the developer guide for guidance on testing your change.

@acs-bot
Copy link

acs-bot commented Jan 15, 2019

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: CJJ69
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: jackfrancis

If they are not already assigned, you can assign the PR to them by writing /assign @jackfrancis in a comment when ready.

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@CJJ69
Copy link
Contributor Author

CJJ69 commented Jan 15, 2019

/assign @jackfrancis

@jackfrancis
Copy link
Member

@PatrickLang can you take a look at this?

@stale
Copy link

stale bot commented Feb 16, 2019

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.

@stale stale bot added the stale label Feb 16, 2019
@mboersma
Copy link
Member

@CJJ69 sorry this has been sitting so long--it needs a rebase now.

@PatrickLang could you take a look at this Windows fix?

@stale stale bot removed the stale label Feb 21, 2019
@mboersma mboersma requested a review from PatrickLang February 21, 2019 18:24
@mboersma mboersma added the needs-rebase Changes in the target branch require a `git rebase` and `git push -f` label Feb 21, 2019
@CJJ69
Copy link
Contributor Author

CJJ69 commented Mar 12, 2019

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.

@stale
Copy link

stale bot commented Apr 11, 2019

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.

@stale stale bot added the stale label Apr 11, 2019
@stale stale bot closed this Apr 18, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
needs-rebase Changes in the target branch require a `git rebase` and `git push -f` size/XS stale
Projects
None yet
Development

Successfully merging this pull request may close these issues.

cluster-autoscaler (addon) repeatedly crashes.
4 participants