Skip to content
This repository has been archived by the owner on Jun 13, 2021. It is now read-only.

Update cnab-to-oci to v0.3.0-beta2 #766

Closed

Conversation

zappy-shu
Copy link
Contributor

- What I did

Updates vendor of cnab-to-oci to v0.3.0-beta2:

  • Push images using relocation map first, then from the bundle
  • Fix Docker CLI authentication to registries for the push
  • Simplify error message returned when pulling CNAB bundle that is not found

- How to verify it

Attempt to pull an app that does not exist. E.g. docker app pull carolinebriaud/vapp

The error message should now be simplified to:

carolinebriaud/vapp: docker.io/carolinebriaud/vapp:latest: not found

- Description for the changelog
Simplify error message returned when pulling CNAB bundle that is not found

- A picture of a cute animal (not mandatory)

hedgehog-2019-11-07

Updates vendor of cnab-to-oci to v0.3.0-beta2:

* Push images using relocation map first, then from the bundle
* Fix Docker CLI authentication to registries for the push
* Simplify error message returned when pulling CNAB bundle that is not found

Signed-off-by: Nick Adcock <nick.adcock@docker.com>
@codecov
Copy link

codecov bot commented Nov 26, 2019

Codecov Report

Merging #766 into master will decrease coverage by 0.11%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #766      +/-   ##
==========================================
- Coverage   69.66%   69.55%   -0.12%     
==========================================
  Files          64       64              
  Lines        3597     3567      -30     
==========================================
- Hits         2506     2481      -25     
+ Misses        760      758       -2     
+ Partials      331      328       -3
Impacted Files Coverage Δ
internal/commands/image/rm.go 58.62% <0%> (-3.88%) ⬇️
types/parameters/parameters.go 92.06% <0%> (-1.59%) ⬇️
internal/store/bundle.go 76.41% <0%> (-0.17%) ⬇️

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 9a49da4...fd7984e. Read the comment docs.

@eunomie
Copy link
Member

eunomie commented Nov 26, 2019

I think we can close it as the bump as been done in #748

@zappy-shu zappy-shu closed this Nov 26, 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.

4 participants