New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Take care of manifest config blob #10805
Take care of manifest config blob #10805
Conversation
Make sure to match image having config name equal to the wanted blob digest. Remember the image and allow access to its layers. Signed-off-by: Michal Minář <miminar@redhat.com>
Manifest configs are fetched using Get() method from blob store. Pullthrough middleware needs to override it as well to allow for pulling manifest v2 schema 2 images from remote repositories. Signed-off-by: Michal Minář <miminar@redhat.com>
os::cmd::expect_success "oc login -u schema2-user -p pass" | ||
os::cmd::expect_success "oc project schema2" | ||
# tagging remote docker.io/busybox image | ||
os::cmd::expect_success "oc tag --source=docker busybox '${DOCKER_REGISTRY}/schema2/busybox'" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
actually, I was wrong... I think this was supposed to be docker tag
, but without the --source
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it's trying to tag and push using pure docker API calls
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
--source
removed
Signed-off-by: Michal Minář <miminar@redhat.com>
Signed-off-by: Michal Minář <miminar@redhat.com>
1c66f73
to
c442bf4
Compare
[merge] |
@mfojtik thanks! You were faster 😄 |
[test] |
Evaluated for origin test up to c442bf4 |
[merge] |
Evaluated for origin merge up to c442bf4 |
continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/8715/) |
continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/8716/) |
clone of: #10759
Resolves #10730
Manifest V2 schema 2 introduces a special blob called config which used to be emeedded in earlier schema. See upstream's spec for details. The config is stored as a regular blob on registry's storage. Thus the config needs to be treated similar to a regular layer:
The PR addresses the first 3 items. I'd like to cover the pruning case in a follow-up.