Skip to content

Conversation

@AkihiroSuda
Copy link
Member

@AkihiroSuda AkihiroSuda commented Jun 17, 2020

Previously shim v2 (io.containerd.runc.{v1,v2}) always used /run/containerd/runc as the runc root.

Fix #4326
Fix #2767

Previously shim v2 (`io.containerd.runc.{v1,v2}`) always used `/run/containerd/runc` as the runc root.

Fix containerd#4326

Signed-off-by: Akihiro Suda <akihiro.suda.cz@hco.ntt.co.jp>
@AkihiroSuda
Copy link
Member Author

Can we have v1.3.5 after merging and cherrypicking this?

Copy link
Member

@fuweid fuweid left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@fuweid
Copy link
Member

fuweid commented Jun 17, 2020

Can we have v1.3.5 after merging and cherrypicking this?

ping @containerd/containerd-release

Copy link
Member

@estesp estesp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

shim v2 doesn't pass Moby test TestContainerStartOnDaemonRestart (because shim v2 uses hard-coded RuncRoot) v2 runtime should support runtime_root

3 participants