Skip to content

Conversation

@mmishra100
Copy link

This change enables EROFS filesystems to be specified as regular mounts in the OCI bundle config.json, not just as rootfs or debug mounts.

EROFS mounts are now tracked in goferMountConfs alongside lisafs mounts. The implementation adds:

  • IsErofsMount() helper in specutils to identify EROFS mounts
  • Updated mount index tracking across container.go, gofer.go, and vfs.go
  • Support for opening EROFS image files and passing FDs to the sandbox
  • EROFS case in getMountNameAndOptions() for proper mount setup

Key implementation details:

  • EROFS mounts are included in goferMountConfs but skip gofer-specific processing (e.g., lisafs serving, filestore creation)
  • Mount type determination happens before mount hint logic
  • Proper index tracking ensures EROFS mounts increment indices but skip lisafs-only operations

Fixes #12307

This change enables EROFS filesystems to be specified as regular mounts
in the OCI bundle config.json, not just as rootfs or debug mounts.

EROFS mounts are now tracked in goferMountConfs alongside lisafs mounts.
The implementation adds:
- IsErofsMount() helper in specutils to identify EROFS mounts
- Updated mount index tracking across container.go, gofer.go, and vfs.go
- Support for opening EROFS image files and passing FDs to the sandbox
- EROFS case in getMountNameAndOptions() for proper mount setup

Key implementation details:
- EROFS mounts are included in goferMountConfs but skip gofer-specific
  processing (e.g., lisafs serving, filestore creation)
- Mount type determination happens before mount hint logic
- Proper index tracking ensures EROFS mounts increment indices but skip
  lisafs-only operations

Fixes google#12307
@google-cla
Copy link

google-cla bot commented Nov 28, 2025

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@mmishra100 mmishra100 mentioned this pull request Nov 28, 2025
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.

Erofs as mount

1 participant