-
Notifications
You must be signed in to change notification settings - Fork 62
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
CI: add nerdctl #327
CI: add nerdctl #327
Conversation
|
|
Mmmm... did this get autoclosed because of the fix in nerdctl? @AkihiroSuda did you mean this to be closed, or shall we revive it as soon as we have a new nerdctl RC with the fix? |
f46b880
to
a8301ef
Compare
|
I am more baffled by the other failure, which seems to be that (docker) runc is not allowed to pivot root (because... apparmor?). Are we missing a step in the installation flow? Do you have a clue what's going on with this? |
a8301ef
to
b440fcf
Compare
b440fcf
to
0e425ab
Compare
dcd8563
to
b654443
Compare
The CI is now almost green, except the last "Test data persistency" step |
So, Kube API is not responding after lima restart? Maybe we can duplicate https://github.com/rootless-containers/usernetes/blob/master/.github/workflows/main.yaml#L109C1-L122C80 and add it after |
8a5bb45
to
2e8f366
Compare
Grumble grumble that was not super helpful. Looks like the kube API indeed is not being restarted on reboot (or it fails starting for some reason). Spitballing hypothesis:
We need more debug in there: for eg some If you do not get to this before me ^ I'll debug on a separate PR later on. |
bypass4netnsd seems not working |
90dd738
to
738cfdf
Compare
bypass4netns refuses to work because the PID file in |
Signed-off-by: Akihiro Suda <akihiro.suda.cz@hco.ntt.co.jp>
Signed-off-by: Akihiro Suda <akihiro.suda.cz@hco.ntt.co.jp>
738cfdf
to
bd052af
Compare
So... because the containing directory does not exist, right?
|
We must have that problem outside of usernetes. I think we can fix in nerdctl. |
Tentatively: containerd/nerdctl#3724 If you get a chance to try it with this patch, curious if that would fix it. Presumably, it works in certain cases because |
Thanks @apostasie , will try your patch later 👍 |
No description provided.