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

fix: Rename acsengineVersion tag to aksEngineVersion #264

Merged
merged 1 commit into from Jan 10, 2019

Conversation

CecileRobertMichon
Copy link
Contributor

Reason for Change:

This is a leftover "acs-engine" to "aks-engine" transition item. The tag that is added to every VM deployed with aks-engine is now "aksEngineVersion" instead of "acsengineVersion". Also changed variables in the code that referred to acsengineVersion.

Issue Fixed:

Requirements:

Notes:

@tariq1890
Copy link
Contributor

lgtm

@codecov
Copy link

codecov bot commented Jan 10, 2019

Codecov Report

Merging #264 into master will not change coverage.
The diff coverage is 61.53%.

@@           Coverage Diff           @@
##           master     #264   +/-   ##
=======================================
  Coverage   53.25%   53.25%           
=======================================
  Files          95       95           
  Lines       14246    14246           
=======================================
  Hits         7586     7586           
  Misses       5995     5995           
  Partials      665      665

@mboersma
Copy link
Member

/lgtm

@acs-bot acs-bot added the lgtm label Jan 10, 2019
@acs-bot acs-bot merged commit 982adab into Azure:master Jan 10, 2019
@acs-bot
Copy link

acs-bot commented Jan 10, 2019

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: CecileRobertMichon, mboersma

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:
  • OWNERS [CecileRobertMichon,mboersma]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@jackfrancis jackfrancis added this to Done in backlog Jan 11, 2019
@CecileRobertMichon CecileRobertMichon deleted the fix-version-tag branch February 7, 2019 00:24
juhacket pushed a commit to juhacket/aks-engine that referenced this pull request Mar 14, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
No open projects
backlog
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

4 participants