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

Runtime-rs + clh does not work with nerdctl and docker #8579

Closed
amshinde opened this issue Dec 5, 2023 · 0 comments · Fixed by #8580
Closed

Runtime-rs + clh does not work with nerdctl and docker #8579

amshinde opened this issue Dec 5, 2023 · 0 comments · Fixed by #8580
Labels
bug Incorrect behaviour needs-review Needs to be assessed by the team.

Comments

@amshinde
Copy link
Member

amshinde commented Dec 5, 2023

Nerdctl and docker run hooks for network setup after the sandbox is started. This requires runtime to implement network hotlug to support network attach for a sandbox. As network hotplug is not currently impemented with clh, runtime-rs + clh does not work with nerdctl/docker.

@amshinde amshinde added bug Incorrect behaviour needs-review Needs to be assessed by the team. labels Dec 5, 2023
amshinde added a commit to amshinde/kata-containers that referenced this issue Dec 5, 2023
This is required for clh to work with nerdtcl and docker.
This fixes the issues seen with nerdctl while starting a container.
Hoewever, container exit with docker is still broken due to an unrelated
issue.

Fixes: kata-containers#8579

Signed-off-by: Archana Shinde <archana.m.shinde@intel.com>
@katacontainersbot katacontainersbot moved this from To do to In progress in Issue backlog Dec 5, 2023
amshinde added a commit to amshinde/kata-containers that referenced this issue Dec 5, 2023
This is required for clh to work with nerdtcl and docker.
This fixes the issues seen with nerdctl while starting a container.
Hoewever, container exit with docker is still broken due to an unrelated
issue.

Fixes: kata-containers#8579

Signed-off-by: Archana Shinde <archana.m.shinde@intel.com>
sprt pushed a commit to microsoft/kata-containers that referenced this issue Dec 20, 2023
This is required for clh to work with nerdtcl and docker.
This fixes the issues seen with nerdctl while starting a container.
Hoewever, container exit with docker is still broken due to an unrelated
issue.

Fixes: kata-containers#8579

Signed-off-by: Archana Shinde <archana.m.shinde@intel.com>
sprt pushed a commit to microsoft/kata-containers that referenced this issue Dec 20, 2023
This is required for clh to work with nerdtcl and docker.
This fixes the issues seen with nerdctl while starting a container.
Hoewever, container exit with docker is still broken due to an unrelated
issue.

Fixes: kata-containers#8579

Signed-off-by: Archana Shinde <archana.m.shinde@intel.com>
sprt pushed a commit to microsoft/kata-containers that referenced this issue Dec 20, 2023
This is required for clh to work with nerdtcl and docker.
This fixes the issues seen with nerdctl while starting a container.
Hoewever, container exit with docker is still broken due to an unrelated
issue.

Fixes: kata-containers#8579

Signed-off-by: Archana Shinde <archana.m.shinde@intel.com>
sprt pushed a commit to microsoft/kata-containers that referenced this issue Dec 20, 2023
This is required for clh to work with nerdtcl and docker.
This fixes the issues seen with nerdctl while starting a container.
Hoewever, container exit with docker is still broken due to an unrelated
issue.

Fixes: kata-containers#8579

Signed-off-by: Archana Shinde <archana.m.shinde@intel.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Incorrect behaviour needs-review Needs to be assessed by the team.
Projects
Issue backlog
  
In progress
Development

Successfully merging a pull request may close this issue.

1 participant