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

remove outdated filesystem recommendation #224

Merged
merged 1 commit into from
Dec 12, 2023
Merged

Conversation

rptaylor
Copy link
Contributor

@rptaylor rptaylor commented Dec 8, 2023

This is more of an issue/discussion but this repo doesn't seem to have issues.
Anyway this line recommending ext3 or 4 over XFS has existed unchanged in the documentation for 8 years and recently made it into the "Best Practices for CernVM FS in HPC" tutorial, probably reaching many eyes.

If there is a specific reason for preferring ext4 over XFS it should be explained and justified in the documentation. But the fact that it was put in 8 years ago and never updated leads me to believe the information is likely stale. A great deal has changed in that time, including extensive improvements and engineering efforts going into XFS and likely ext4 as well. To remain valid as a recommendation, some current benchmarking (or other clarifying information) would be needed.
One can find endless debates about ext4 vs XFS but all in all they both seem fine these days. If one foresees a need to shrink a filesystem, use ext4. XFS seems better for more parallel IO streams (which I would think should favour CVMFS) whereas ext4 may be better for single streams.

But in particular nobody should be recommending ext3 anymore! Unless there is forthcoming information to justify this recommendation I would suggest removing it. @jblomer @DrDaveD ?

@rptaylor
Copy link
Contributor Author

rptaylor commented Dec 8, 2023

Some related discussion: https://unix.stackexchange.com/questions/525613/xfs-vs-ext4-performance (though it was still 4 years ago, probably recent enough to still have some relevance)

@jblomer
Copy link
Member

jblomer commented Dec 9, 2023

I'm fine with removing ext3. For xfs, I'd like to hear opinions from stratum 1 admins, perhaps we can discuss in the meeting on Monday @vvolkl ?

@rptaylor
Copy link
Contributor Author

Sorry I couldn't make it to the meeting, were there any comments?

We have been using XFS for /srv/cvmfs, basically for the reasons I described above.

@vvolkl
Copy link
Contributor

vvolkl commented Dec 12, 2023

Yes, we discussed this in the coordination meeting and concluded that this information is indeed stale. Thanks for bringing it up!

@vvolkl vvolkl merged commit c162105 into cvmfs:latest Dec 12, 2023
2 checks passed
@rptaylor rptaylor deleted the patch-17 branch January 29, 2024 18:36
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.

3 participants