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

xfs_scrub controller #9545

Open
Tracked by #9249 ...
frezbo opened this issue Oct 22, 2024 · 5 comments · May be fixed by #9848
Open
Tracked by #9249 ...

xfs_scrub controller #9545

frezbo opened this issue Oct 22, 2024 · 5 comments · May be fixed by #9848
Assignees

Comments

@frezbo
Copy link
Member

frezbo commented Oct 22, 2024

Feature Request

Implement a Talos controller for periodically running xfs_scrub. This also needs to be configurable by a machine configuration document.

References

@dsseng
Copy link
Member

dsseng commented Nov 28, 2024

Perhaps it should be not only for XFS but flexible with possibilities to include e.g. btrfs we plan on adding, as it is also often used with a maintenance schedule in place. So I'd do a config format listing scrub configs for each FS type.

@smira
Copy link
Member

smira commented Nov 28, 2024

If we want to make it simple, I would start with defaults and a feature flag which enables it (and we keep it disabled for now). Then, we look at all VolumeStatuses, find ones which are Ready, match on filesystem type, and schedule xfs_scrub accordingly from boot time

@frezbo
Copy link
Member Author

frezbo commented Nov 28, 2024

If we want to make it simple, I would start with defaults and a feature flag which enables it (and we keep it disabled for now). Then, we look at all VolumeStatuses, find ones which are Ready, match on filesystem type, and schedule xfs_scrub accordingly from boot time

I think we'd still need the interval to be configurable, since this could potentially lockup writes to filesystem

@dsseng
Copy link
Member

dsseng commented Nov 28, 2024

Yes, these operations might make system slow

@dsseng dsseng self-assigned this Nov 28, 2024
@dsseng
Copy link
Member

dsseng commented Nov 28, 2024

We should also note in the docs that xfs_scrub is considered an experimental program according to its docs

@dsseng dsseng linked a pull request Nov 29, 2024 that will close this issue
2 tasks
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 a pull request may close this issue.

3 participants