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
Support "bundle" in addition to "bundlePath" #33
Support "bundle" in addition to "bundlePath" #33
Conversation
|
If we change this, will it break on Docker which uses older versions of runc? |
|
Without this PR I'm getting error: With the PR, |
9b865c9
to
10a2d87
Compare
|
@rhatdan yes, true. I've pushed a new version that supports both variants. |
10a2d87
to
cf51e26
Compare
It was changed in the OCI runtime spec to 1.0.0.rc5, so support both versions. Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
cf51e26
to
8431edd
Compare
|
I can confirm that updated PR still works for me. |
until projectatomic/oci-register-machine#33 is resolved
|
LGTM |
|
I have pushed updated versions of oci-register-machine into fedora updates testing. Please test and give good karma. |
|
Awesome thanks. We also need updated rpm in RHEL Atomic Host - I've created bz here: https://bugzilla.redhat.com/show_bug.cgi?id=1491461 |
|
Yup, I asked @lsm5 to make this available in 7.4.2 release. |
It was changed in the OCI runtime spec to 1.0.0.rc5
Signed-off-by: Giuseppe Scrivano gscrivan@redhat.com