Skip to content

Resolve project image to manifest images#40

Merged
ernoaapa merged 3 commits intomasterfrom
resolve-proper-image
Apr 25, 2018
Merged

Resolve project image to manifest images#40
ernoaapa merged 3 commits intomasterfrom
resolve-proper-image

Conversation

@ernoaapa
Copy link
Owner

Previously we were using the non-manifest images like
'docker.io/arm64v8/node:latest' when we were resolving project
images automatically.
Nowadays the 'docker.io/library/node:latest' is manifest image
which supports multiple different architectures so we don't
need to handle all the cases by ourself.

@ernoaapa ernoaapa force-pushed the resolve-proper-image branch from 222d168 to bb992a8 Compare April 18, 2018 03:43
@codecov
Copy link

codecov bot commented Apr 18, 2018

Codecov Report

Merging #40 into master will decrease coverage by 0.3%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #40      +/-   ##
==========================================
- Coverage   25.48%   25.17%   -0.31%     
==========================================
  Files          50       50              
  Lines        2433     2415      -18     
==========================================
- Hits          620      608      -12     
+ Misses       1746     1740       -6     
  Partials       67       67
Impacted Files Coverage Δ
pkg/resolve/image.go 93.75% <100%> (+9.75%) ⬆️

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 35a0dc3...46eb09e. Read the comment docs.

Previously we were using the non-manifest images like
'docker.io/arm64v8/node:latest' when we were resolving project
images automatically.
Nowadays the 'docker.io/library/node:latest' is manifest image
which supports multiple different architectures so we don't
need to handle all the cases by ourself.
@ernoaapa ernoaapa force-pushed the resolve-proper-image branch 8 times, most recently from 8cb3a0b to 79a82b1 Compare April 25, 2018 11:33
@ernoaapa ernoaapa force-pushed the resolve-proper-image branch from 79a82b1 to 46eb09e Compare April 25, 2018 11:41
@ernoaapa ernoaapa merged commit d4554e9 into master Apr 25, 2018
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.

1 participant