Skip to content

Comments

219 no bindmount leak#252

Draft
JonathonReinhart wants to merge 2 commits intomainfrom
219-no-bindmount-leak
Draft

219 no bindmount leak#252
JonathonReinhart wants to merge 2 commits intomainfrom
219-no-bindmount-leak

Conversation

@JonathonReinhart
Copy link
Owner

No description provided.

This works if scubaroot = /home/$USER: is_bind_mount() detects the bind mount
because we're trying to mount the /jreinhart subdirectory of the /home/
mount:

  scubainit: ERROR - Error creating home dir: "/home/jreinhart" is on a bind-mount: /dev/mapper/magnum--vg-home[/jreinhart]

But it fails for scubaroot = /home: is_bind_mount() sees root="/" and
returns false.

I think there is simply no way to reliably detect a bind mount!

So now we might be to the point of either passing the list of
bind-mount volumes from scuba to scubainit, or by checking for this
situation in scuba instead.
@github-actions
Copy link

github-actions bot commented Apr 8, 2024

Test Results

    6 files      6 suites   2m 1s ⏱️
  177 tests   175 ✅  2 💤 0 ❌
1 062 runs  1 050 ✅ 12 💤 0 ❌

Results for commit 6f50a85.

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