Skip to content

fix: make /lib/modules directory overlayfs over the hostpath#1997

Closed
rugggger wants to merge 1 commit intomainfrom
12-23-fix_make__lib_modules_directory_overlayfs_over_the_hostpath
Closed

fix: make /lib/modules directory overlayfs over the hostpath#1997
rugggger wants to merge 1 commit intomainfrom
12-23-fix_make__lib_modules_directory_overlayfs_over_the_hostpath

Conversation

@rugggger
Copy link
Contributor

@rugggger rugggger commented Dec 23, 2025

TL;DR

Implemented overlayfs for /lib/modules to support driver loading on read-only host mounts.

What changed?

  • Added a new setup_overlayfs_for_lib_modules() function in weka_runtime.py that creates a writable overlay filesystem for /lib/modules using tmpfs
  • The overlay allows writing to /lib/modules even when the host mount is read-only
  • Integrated this function into the driver loading workflow
  • Refactored driver dependencies code into a separate file (drivers.go) for better organization
  • Made /lib/modules mount read-only in non-COS environments

<release_notes>

Driver Loading on Read-Only Systems

Type: fix
Enabled driver loading even when the host's /lib/modules directory is mounted as read-only, by utilizing an overlay filesystem. This resolves issues with driver loading in environments where the host /lib/modules is read-only.
</release_notes>

Copy link
Contributor Author

rugggger commented Dec 23, 2025


How to use the Graphite Merge Queue

Add the label main-merge-queue to this PR to add it to the merge queue.

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has required the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

@rugggger rugggger marked this pull request as ready for review December 23, 2025 09:56
@graphite-app
Copy link

graphite-app bot commented Dec 23, 2025

Graphite Automations

"Add anton/matt/sergey/kristina as reviwers on operator PRs" took an action on this PR • (12/23/25)

3 reviewers were added to this PR based on Anton Bykov's automation.

@graphite-app
Copy link

graphite-app bot commented Dec 23, 2025

Merge activity

  • Dec 23, 1:25 PM UTC: rugggger added this pull request to the Graphite merge queue.
  • Dec 23, 1:25 PM UTC: CI is running for this pull request on a draft pull request (#2004) due to your merge queue CI optimization settings.
  • Dec 23, 2:05 PM UTC: Merged by the Graphite merge queue via draft PR: #2004.

@graphite-app graphite-app bot closed this Dec 23, 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.

2 participants