-
Notifications
You must be signed in to change notification settings - Fork 1k
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
pkg/init: Mount /sys/fs/bpf #3516
Conversation
Not sure why CI is failing, the error seems certainly unrelated:
|
I wonder if I should update any of the examples, and if so, the question is - which ones? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Let's get the image pushed out and CI renewed, and when it is green, we can merge it in.
Did you run the |
Nope, looks like you did not. Let's do that. |
aa3a0f0
to
fff04cf
Compare
Looking better now. Image is almost pushed out. |
Image is out. |
@deitch is |
NOTE: This will be a shared mount, due to root being turned into a shared with `MC_REC` set: `mount("", "/", "", rec|shared, "")`. For some reason setting `shared` when mounting `/sys/fs/bpf` doesn't work at all, perhaps that's just a kernel feature. Signed-off-by: Ilya Dmitrichenko <errordeveloper@gmail.com>
fff04cf
to
86fb6ba
Compare
@deitch CI is getting greener 🎉 |
Yup. That last one takes a long time to run, can be 60+ minutes (lots of building OSes and running them and tearing them down). When it clears green, we can merge this in. |
- Description for the changelog
pkg/init: mount /sys/fs/bpf (fix docker/for-mac#4454)