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

fix: silence k script #571

Merged
merged 3 commits into from
Feb 23, 2019
Merged

fix: silence k script #571

merged 3 commits into from
Feb 23, 2019

Conversation

mboersma
Copy link
Member

Reason for Change:

See https://github.com/deis/docker-go-dev/releases/tag/v1.18.5

Issue Fixed:

Closes #567

Should fix issues we are seeing in Jenkins E2E upgrade tests. I've already set ${GO_DEV_IMAGE} to v1.18.5 there and it looks like we are 💚 again.

Requirements:

Notes:

@codecov
Copy link

codecov bot commented Feb 23, 2019

Codecov Report

Merging #571 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #571   +/-   ##
=======================================
  Coverage   56.72%   56.72%           
=======================================
  Files          91       91           
  Lines       13859    13859           
=======================================
  Hits         7861     7861           
  Misses       5333     5333           
  Partials      665      665

@jackfrancis
Copy link
Member

/lgtm

@acs-bot acs-bot added the lgtm label Feb 23, 2019
@acs-bot acs-bot merged commit 814d3be into Azure:master Feb 23, 2019
@acs-bot
Copy link

acs-bot commented Feb 23, 2019

[APPROVALNOTIFIER] This PR is APPROVED

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

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

juhacket pushed a commit to juhacket/aks-engine that referenced this pull request Mar 14, 2019
* chore: update go-dev tools image to v1.18.5

See https://github.com/deis/docker-go-dev/releases/tag/v1.18.5

* feat: add local copy of k script for convenience

* docs: refer to local `k` script

Until our changes are merged upstream.
@mboersma mboersma deleted the fix-upgrade-ci branch June 21, 2019 19:59
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.

3 participants