Skip to content

fix(mcv): switch to vfs storage and remove fuse-overlayfs dependencies - #173

Open
cmagina wants to merge 1 commit into
redhat-et:mainfrom
cmagina:mcv-image-overlayfs-fix
Open

fix(mcv): switch to vfs storage and remove fuse-overlayfs dependencies#173
cmagina wants to merge 1 commit into
redhat-et:mainfrom
cmagina:mcv-image-overlayfs-fix

Conversation

@cmagina

@cmagina cmagina commented Aug 18, 2026

Copy link
Copy Markdown
Collaborator

VFS storage driver avoids nested overlay-on-overlay issues when running buildah inside containers. Remove fuse-overlayfs/fuse3 packages as they are only needed for overlay storage driver.

Resolves "mkdir /io.triton.manifest: operation not permitted" errors when running MCV in Docker/Podman environments (common on Fedora where docker is podman).

VFS storage driver avoids nested overlay-on-overlay issues when running
buildah inside containers. Remove fuse-overlayfs/fuse3 packages as they
are only needed for overlay storage driver.

Resolves "mkdir /io.triton.manifest: operation not permitted" errors
when running MCV in Docker/Podman environments (common on Fedora where
docker is podman).

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Signed-off-by: Craig Magina <cmagina@redhat.com>
@cmagina
cmagina requested a review from maryamtahhan August 18, 2026 19:31
@cmagina cmagina self-assigned this Aug 18, 2026
@coderabbitai

coderabbitai Bot commented Aug 18, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 46f03eaf-74df-4173-abf9-a21a532c5d85

📥 Commits

Reviewing files that changed from the base of the PR and between 135a404 and ecaa35d.

📒 Files selected for processing (1)
  • mcv/images/Containerfile

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Summary by CodeRabbit

  • Bug Fixes
    • Updated container storage configuration for improved compatibility and reliability.
    • Removed obsolete FUSE-related package requirements from container targets.

Walkthrough

The container targets remove FUSE storage dependencies and change container storage configuration from overlay with fuse-overlayfs to the vfs driver.

Changes

Container storage configuration

Layer / File(s) Summary
Replace FUSE-based storage configuration
mcv/images/Containerfile
The minimal, NVIDIA, and unified targets remove fuse-overlayfs and fuse3. Their storage configuration uses the vfs driver without a mount_program setting.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Merge Risk: ⚪ Minimal · up to ecaa3

The PR switches MCV image storage to VFS and removes unnecessary fuse packages; no actionable merge-blocking risk remains after normal checks and review.

Suggested reviewers: maryamtahhan, billy99

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the switch to VFS storage and removal of obsolete dependencies.
Description check ✅ Passed The description explains the VFS change, dependency removal, and the container runtime error it resolves.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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.

1 participant