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

feat: Set the default Windows sku and version when upgrading a VMSS #2581

Merged
merged 1 commit into from Jan 15, 2020

Conversation

AbelHu
Copy link
Member

@AbelHu AbelHu commented Jan 15, 2020

Reason for Change:

This PR sets default WindowsSku and WindowsVersion when upgrading a VMSS.

Issue Fixed:

Requirements:

Notes:

@mboersma
Copy link
Member

/azp run pr-e2e

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@codecov
Copy link

codecov bot commented Jan 15, 2020

Codecov Report

Merging #2581 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #2581      +/-   ##
==========================================
+ Coverage   72.15%   72.16%   +0.01%     
==========================================
  Files         130      130              
  Lines       24486    24497      +11     
==========================================
+ Hits        17668    17679      +11     
  Misses       5797     5797              
  Partials     1021     1021

Copy link
Contributor

@marosset marosset left a comment

Choose a reason for hiding this comment

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

/lgtm

@marosset
Copy link
Contributor

/approve

@acs-bot acs-bot merged commit ac9c550 into Azure:master Jan 15, 2020
@welcome
Copy link

welcome bot commented Jan 15, 2020

Congrats on merging your first pull request! 🎉🎉🎉

@acs-bot
Copy link

acs-bot commented Jan 15, 2020

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: AbelHu, marosset

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

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants