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

chore: SinglePlacementGroup=false if VMSS + SLB #3054

Conversation

jackfrancis
Copy link
Member

Reason for Change:

This PR sets the VMSS SinglePlacementGroup configuration to false if using Standard LB.

Issue Fixed:

Requirements:

Notes:

@codecov
Copy link

codecov bot commented Apr 9, 2020

Codecov Report

Merging #3054 into master will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #3054   +/-   ##
=======================================
  Coverage   70.62%   70.62%           
=======================================
  Files         145      145           
  Lines       25147    25148    +1     
=======================================
+ Hits        17761    17762    +1     
  Misses       6283     6283           
  Partials     1103     1103           
Impacted Files Coverage Δ
pkg/api/defaults.go 92.22% <100.00%> (+0.01%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8fc62ac...db7613a. Read the comment docs.

Copy link
Member

@mboersma mboersma left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@jackfrancis jackfrancis merged commit d61bb68 into Azure:master Apr 10, 2020
@jackfrancis jackfrancis deleted the vmss-standard-lb-single-placement-group-false branch April 10, 2020 00:22
AbelHu pushed a commit to AbelHu/aks-engine that referenced this pull request May 15, 2020
andyliuliming added a commit that referenced this pull request May 15, 2020
* build: fetch k8s Windows .zip from kubernetesartifacts storage (#2655)

* build: fetch k8s Windows .zip from kubernetesartifacts storage

* docs: remove build-windows-k8s

* build: remove unsupported v1.15.5 from Windows VHD

* fix: generate windows .zip URL correctly for download

* test: update UT expectations for windowszip

* fix: Expose error details in Windows CSE (#3159)

* feat: Updating Windows VHDs to include May patches (#3263)

* chore: update zip name for mooncake mirror (#3237)

* chore: SinglePlacementGroup=false if VMSS + SLB (#3054)

* fix: the os disk size restriction is changed in azure. (#3043)

* refactor: Create signable PS scripts (#3015)

* feat: Refactor PS scripts to be static

* fix: don't hardcode pause image for containerD on Windows (#3158)

* chore: update Windows pause image (#3210)

* fix: USER_ASSIGNED_IDENTITY_ID is empty in azure.json (#3254)

Co-authored-by: Matt Boersma <[email protected]>
Co-authored-by: Mark Rossetti <[email protected]>
Co-authored-by: tamilmani1989 <[email protected]>
Co-authored-by: Jack Francis <[email protected]>
Co-authored-by: Liming Liu <[email protected]>
Co-authored-by: Kalya Subramanian <[email protected]>
AbelHu pushed a commit to AbelHu/aks-engine that referenced this pull request May 16, 2020
AbelHu pushed a commit to AbelHu/aks-engine that referenced this pull request May 16, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants