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

filesystems: include more remote fs types, pass them to fsck #10

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

CameronNemo
Copy link
Contributor

No description provided.

@Vaelatern
Copy link
Member

Can't many of these also be local filesystems?

@Vaelatern
Copy link
Member

And I don't like the change to fsck -- why?

@CameronNemo
Copy link
Contributor Author

Which ones could be local filesystems? I'm not sure how GlusterFS, or Ceph, could be available that early in the boot sequence. The list was gathered from systemd source code, mountall (Upstart mount helper) source code, and Debian init scripts.

As for the fsck change: I thought it would be best if the fsck and mount commands operate on the same mountpoints and with the same information. Anything less might produce inconsistent results. Is a fsck even useful for remote filesystems?

@Vaelatern
Copy link
Member

If gluster is started in the ramdisk for some reason, it could very well be running at that point.

@CameronNemo
Copy link
Contributor Author

From what I have seen while digging through dracut, they fsck themselves before mounting anything (typically / or maybe /usr).

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