Skip to content
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

Re-vendor, notably opencontainers/image-spec to fix tests #468

Merged
merged 1 commit into from
Feb 9, 2018

Conversation

mtrmac
Copy link
Contributor

@mtrmac mtrmac commented Jan 10, 2018

Anyone running (vndr) currently ends up with failing tests in OCI schema validation because gojsonschema has fixed its "$ref" interpretation, exposing inconsistent URI usage inside image-spec/schema.

So, this runs (vndr), and uses mtrmac/image-spec:id-based-loader ( opencontainers/image-spec#739 ) to make the tests pass again. As soon as that PR is merged we should revert to using the upstream image-spec repo again.

@mtrmac
Copy link
Contributor Author

mtrmac commented Jan 10, 2018

(Alternatively, we could leave the repo as is, and just keep telling people to run vndr $your_repo instead of vndr, giving up on rebasing the other libraries as well. I don’t have a strong preference.)

@mtrmac mtrmac force-pushed the oci-schema-rebase branch 2 times, most recently from d358311 to f152660 Compare January 22, 2018 16:32
@mtrmac mtrmac force-pushed the oci-schema-rebase branch from f152660 to 09beda1 Compare February 9, 2018 17:17
@mtrmac mtrmac changed the title Re-vendor, and use mtrmac/image-spec:id-based-loader to fix tests Re-vendor, notably opencontainers/image-spec to fix tests Feb 9, 2018
Anyone running (vndr) currently ends up with failing tests in OCI schema
validation because gojsonschema has fixed its "$ref" interpretation, exposing
inconsistent URI usage inside image-spec/schema.

So, this runs (vndr), and uses mtrmac/image-spec:id-based-loader
( opencontainers/image-spec#739 ) to make the tests pass
again.  As soon as that PR is merged we should revert to using the upstream
image-spec repo again.
@mtrmac mtrmac force-pushed the oci-schema-rebase branch from 09beda1 to dc1567c Compare February 9, 2018 17:39
@mtrmac
Copy link
Contributor Author

mtrmac commented Feb 9, 2018

@runcom PTAL. The fix has been merged upstream, so this is primarily a rebase of opencontainers/image-spec.

@runcom
Copy link
Member

runcom commented Feb 9, 2018

LGTM

@runcom runcom merged commit 1307cac into containers:master Feb 9, 2018
@mtrmac mtrmac deleted the oci-schema-rebase branch February 12, 2018 16:51
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 5, 2023
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.

2 participants