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

fix: replace some leftover acsengine by aksengine #268

Merged
merged 1 commit into from
Jan 11, 2019

Conversation

CecileRobertMichon
Copy link
Contributor

@CecileRobertMichon CecileRobertMichon commented Jan 10, 2019

Reason for Change:

Fix some acs-engine references that were left in the code. Most are variables. One is the generatorCode that is used in the CreationSource tag for VMs.

Issue Fixed:

Requirements:

Notes:

@codecov
Copy link

codecov bot commented Jan 10, 2019

Codecov Report

Merging #268 into master will not change coverage.
The diff coverage is 8.33%.

@@           Coverage Diff           @@
##           master     #268   +/-   ##
=======================================
  Coverage   53.16%   53.16%           
=======================================
  Files          95       95           
  Lines       14244    14244           
=======================================
  Hits         7573     7573           
  Misses       6006     6006           
  Partials      665      665

@tariq1890
Copy link
Contributor

/lgtm

@jackfrancis
Copy link
Member

/hold

Let's get #253 merged first, as this one will probably be easier to resolve merge conflicts than that one

@CecileRobertMichon
Copy link
Contributor Author

@bacongobbler I rebased can you please do a quick check to make sure I didn't unintentionally undo some of your doc changes?

@CecileRobertMichon
Copy link
Contributor Author

/hold cancel

Copy link
Contributor

@bacongobbler bacongobbler left a comment

Choose a reason for hiding this comment

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

Nope! This looks good to me.

@jackfrancis
Copy link
Member

/lgtm

@acs-bot acs-bot added the lgtm label Jan 11, 2019
@jackfrancis jackfrancis merged commit 6429de9 into Azure:master Jan 11, 2019
@acs-bot
Copy link

acs-bot commented Jan 11, 2019

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: CecileRobertMichon, jackfrancis, tariq1890

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,jackfrancis,tariq1890]

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

@CecileRobertMichon CecileRobertMichon deleted the fix-generator-src 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
None yet
Development

Successfully merging this pull request may close these issues.

5 participants