Skip to content

Convert Windows CI to use Microsoft MCR image urls#3230

Merged
estesp merged 1 commit intocontainerd:masterfrom
jterry75:fix_windows_image
Apr 19, 2019
Merged

Convert Windows CI to use Microsoft MCR image urls#3230
estesp merged 1 commit intocontainerd:masterfrom
jterry75:fix_windows_image

Conversation

@jterry75
Copy link
Copy Markdown
Contributor

Signed-off-by: Justin Terry (VM) [email protected]

@jterry75
Copy link
Copy Markdown
Contributor Author

@jhowardmsft - FYI

@estesp
Copy link
Copy Markdown
Member

estesp commented Apr 18, 2019

Image reference not quite right maybe?

time="2019-04-18T23:03:12Z" level=info msg="start to pull seed image"
failed to resolve reference "mcr.microsoft.com/microsoft/nanoserver:sac2016": mcr.microsoft.com/microsoft/nanoserver:sac2016 not found

@jterry75 jterry75 force-pushed the fix_windows_image branch from 67f39a0 to e6fc0ed Compare April 18, 2019 23:07
@jterry75
Copy link
Copy Markdown
Contributor Author

Lol. windows/nanoserver not microsoft/nanoserver

@fuweid
Copy link
Copy Markdown
Member

fuweid commented Apr 19, 2019

Great!

The fail case is flaky one. repush it will be fine.

Copy link
Copy Markdown

@lowenna lowenna left a comment

Choose a reason for hiding this comment

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

LGTM. (The appveyor run failed for other reasons - not sure how to restart it, but you might have to commit -amend and push --force to get another run)

@codecov-io
Copy link
Copy Markdown

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #3230   +/-   ##
=======================================
  Coverage   44.63%   44.63%           
=======================================
  Files         113      113           
  Lines       12161    12161           
=======================================
  Hits         5428     5428           
  Misses       5898     5898           
  Partials      835      835
Flag Coverage Δ
#linux 48.65% <ø> (ø) ⬆️
#windows 39.86% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a91e043...e6fc0ed. Read the comment docs.

Copy link
Copy Markdown
Member

@estesp estesp left a comment

Choose a reason for hiding this comment

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

LGTM

@estesp estesp merged commit c2fb5d7 into containerd:master Apr 19, 2019
@jterry75 jterry75 deleted the fix_windows_image branch January 15, 2020 19:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants