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

bump runc to at least runc-1.0.0-51 #284

Closed
sjenning opened this issue Sep 11, 2018 · 10 comments
Closed

bump runc to at least runc-1.0.0-51 #284

sjenning opened this issue Sep 11, 2018 · 10 comments
Assignees

Comments

@sjenning
Copy link

fixes containers/podman#1396

blocking switch from docker to podman for bootkube

@crawford

@crawford
Copy link
Contributor

Actually, it doesn't block the move to podman, but it would still be good to have.

@dustymabe
Copy link
Member

does podman vendor runc? should we be bumping podman instead ?

@ashcrow
Copy link
Member

ashcrow commented Sep 12, 2018

runc is a dep of podman

$ rpm -q --requires podman | grep runc
runc

@ashcrow
Copy link
Member

ashcrow commented Sep 12, 2018

This work is already being done by @yuqi-zhang in COREOS-477 as using the new repo will pull in the updated runc.

@ashcrow ashcrow assigned yuqi-zhang and unassigned cgwalters Sep 12, 2018
@sjenning
Copy link
Author

@crawford true as long as we removed --tmpfs

https://github.com/openshift/installer/pull/207/files#r216868889

slightly less secure than before but... in the grand scheme, not sure if it makes a difference.

@wking
Copy link
Member

wking commented Sep 13, 2018

Was this addressed by #286?

@ashcrow
Copy link
Member

ashcrow commented Sep 13, 2018

No. The extras repo in #286 still is under the requested version. The work @yuqi-zhang is doing should fix this issue.

@yuqi-zhang
Copy link
Contributor

Added in: #288

podman-0.8.5-3.gitdc5a711.el7.x86_64
cri-o-1.11.3-2.rhaos3.11.git4fbb022.el7_5.x86_64
cri-tools-1.0.0-6.rhaos3.11.git78ec590.el7.x86_64
runc-1.0.0-52.dev.git70ca035.el7_5.x86_64
containernetworking-plugins-0.7.1-1.el7.x86_64

@wking
Copy link
Member

wking commented Sep 14, 2018

$ curl -s http://aos-ostree.rhev-ci-vms.eng.rdu2.redhat.com/rhcos/images/cloud/latest/pkglist.txt | grep runc
 runc-1.0.0-52.dev.git70ca035.el7_5.x86_64

/close

@openshift-ci-robot
Copy link

@wking: Closing this issue.

In response to this:

$ curl -s http://aos-ostree.rhev-ci-vms.eng.rdu2.redhat.com/rhcos/images/cloud/latest/pkglist.txt | grep runc
runc-1.0.0-52.dev.git70ca035.el7_5.x86_64

/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

wking added a commit to wking/openshift-installer that referenced this issue Sep 14, 2018
We'd removed this in c234fc3 (*: use podman instead of docker,
2018-09-05, openshift#207) to work around [1,2].  Now that RHCOS is up to:

  $ curl -s http://aos-ostree.rhev-ci-vms.eng.rdu2.redhat.com/rhcos/images/cloud/latest/pkglist.txt | grep runc
   runc-1.0.0-52.dev.git70ca035.el7_5.x86_64

we can restore the option.

[1]: openshift/os#284
[2]: containers/podman#1396
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

No branches or pull requests

8 participants