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

fix: add missing container images in VHD #284

Merged
merged 3 commits into from Jan 11, 2019

Conversation

CecileRobertMichon
Copy link
Contributor

Reason for Change:

Noticed that Tiller 2.11.0 and blobfuse-flexvolume addon container images were missing from the VHD install script. This PR adds them.

Issue Fixed:

Requirements:

Notes:

@mboersma
Copy link
Member

lgtm

@@ -87,7 +87,7 @@ for PAUSE_VERSION in ${PAUSE_VERSIONS}; do
pullContainerImage "docker" "k8s.gcr.io/pause-amd64:${PAUSE_VERSION}"
done

TILLER_VERSIONS="2.8.1"
TILLER_VERSIONS="2.8.1 2.11.0"
Copy link
Member

Choose a reason for hiding this comment

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

bummer

@jackfrancis
Copy link
Member

/lgtm

@codecov
Copy link

codecov bot commented Jan 11, 2019

Codecov Report

Merging #284 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #284      +/-   ##
==========================================
+ Coverage   53.21%   53.22%   +<.01%     
==========================================
  Files          95       95              
  Lines       14235    14233       -2     
==========================================
  Hits         7575     7575              
+ Misses       5995     5993       -2     
  Partials      665      665

@jackfrancis jackfrancis added this to In progress in backlog Jan 11, 2019
packer/install-dependencies.sh Show resolved Hide resolved
@mboersma
Copy link
Member

/lgtm

@acs-bot
Copy link

acs-bot commented Jan 11, 2019

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: CecileRobertMichon, jackfrancis, 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,jackfrancis,mboersma]

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

@acs-bot acs-bot removed the lgtm label Jan 11, 2019
@acs-bot
Copy link

acs-bot commented Jan 11, 2019

New changes are detected. LGTM label has been removed.

@CecileRobertMichon CecileRobertMichon merged commit 7976655 into Azure:master Jan 11, 2019
backlog automation moved this from In progress to Done Jan 11, 2019
jackfrancis pushed a commit that referenced this pull request Jan 11, 2019
tiller 2.11.0
blobfuse-flexvolume 1.0.7
@CecileRobertMichon CecileRobertMichon deleted the vhd-tiller-image branch February 7, 2019 00:24
juhacket pushed a commit to juhacket/aks-engine that referenced this pull request Mar 14, 2019
tiller 2.11.0
blobfuse-flexvolume 1.0.7
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