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

fix(kernel): files in overlayfs of container are missing #74

Merged
merged 1 commit into from
Oct 17, 2024

Conversation

BlackHole1
Copy link
Member

We found that when executing an overlay mount inside a Podman container, there is an issue: the middle layer contains a node_modules directory, but the upper layer cannot display this directory.

When we disable all overlay options, it returns to normal. Currently, there are no other side effects found from disabling the overlay options.

Description of Change

Required to run CI

  • applehv-rootfs-amd64
  • applehv-rootfs-arm64
  • wsl-rootfs-amd64
  • wsl-rootfs-arm64
  • initrd-amd64
  • initrd-arm64
  • kernel-amd64
  • kernel-arm64

We found that when executing an overlay mount inside a Podman container, there is an issue: the middle layer contains a node_modules directory, but the upper layer cannot display this directory.

When we disable all overlay options, it returns to normal. Currently, there are no other side effects found from disabling the overlay options.

Signed-off-by: Kevin Cui <bh@bugs.cc>
@BlackHole1 BlackHole1 requested a review from ihexon October 17, 2024 09:10
Copy link

Build Output Summary

Target Size
kernel-amd64 14.3791MB (15077536 bytes)
kernel-arm64 24.4946MB (25684480 bytes)

@BlackHole1 BlackHole1 merged commit 26e9d49 into main Oct 17, 2024
4 checks passed
@BlackHole1 BlackHole1 deleted the disable-overlay-options branch October 17, 2024 09:35
BlackHole1 added a commit to oomol-lab/ovm-js that referenced this pull request Oct 17, 2024
BlackHole1 added a commit to oomol-lab/ovm-js that referenced this pull request Oct 17, 2024
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

Successfully merging this pull request may close these issues.

2 participants