-
Notifications
You must be signed in to change notification settings - Fork 73
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
Fixes #18534: Skip non-standard moint FS for free-space check #3351
Fixes #18534: Skip non-standard moint FS for free-space check #3351
Conversation
No, you need to first filter to keep only lines starting with "/" and then keep the original logic of looking for "/var" or use "/" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, you need to first filter to keep only lines starting with "/" and then keep the original logic of looking for "/var" or use "/"
dab3ecc
to
39cf8fb
Compare
Commit modified |
@@ -96,7 +96,7 @@ final object CheckFreeSpace extends Check { | |||
|
|||
def run: IOResult[HealthcheckResult] = { | |||
val file = root / "proc" / "mounts" | |||
val mountsContent = file.lines.map(x => x.split(" ")(1)).toList | |||
val mountsContent = file.lines.filter(_(0) == '/').map(x => x.split(" ")(1)).toList |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you had a comment to explain? With a link to the bug for future reference ?
39cf8fb
to
f8a64ac
Compare
Commit modified |
OK, merging this PR |
https://issues.rudder.io/issues/18534