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

[v23.3.x] Ext4 detection #15855

Conversation

vbotbuildovich
Copy link
Collaborator

Backport of PR #13496

Print the detected filsystem type at info always, since this is
critical one-time info.

Also print the detected fs type in the warning message when the type
is not xfs or ext4.

(cherry picked from commit b6a869f)
Due to scylladb/seastar#251 all of ext2, ext3 and ext4 will be
detected as ext2 (they shared the same superblock magic number).

This leads to an erroneous warning claiming that you are not running
on ext4 nor XFS. For now, we will simply that assume any ext detection
means ext4, as ext2 and 3 use is very limited as ext4 was rolled out
more than 15 years ago and almost entirely supplanted use of the
earlier variants.

In any case, the change only changes the warning message emitted, not
any other behavior.

Issue redpanda-data#13469.

(cherry picked from commit 04168f2)
To make it clearer why we treat ext2 as ext4.

(cherry picked from commit f99eb9e)
As pointed out in review.

(cherry picked from commit 328c245)
We semi-support ext4 but in the syscheck we will now check for both
ext2 and ext4, as currently ext4 is always detected as ext2, but
in the future if ext4 is returned we should also treat this as ext4.

Issue redpanda-data#13469.

(cherry picked from commit 051c63c)
@vbotbuildovich vbotbuildovich added this to the v23.3.x-next milestone Dec 21, 2023
@vbotbuildovich vbotbuildovich added the kind/backport PRs targeting a stable branch label Dec 21, 2023
@nvartolomei nvartolomei marked this pull request as ready for review December 21, 2023 20:28
@piyushredpanda piyushredpanda modified the milestones: v23.3.x-next, v23.3.2 Jan 4, 2024
@piyushredpanda
Copy link
Contributor

/ci-repeat

@StephanDollberg StephanDollberg merged commit 57a6f65 into redpanda-data:v23.3.x Jan 5, 2024
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/redpanda kind/backport PRs targeting a stable branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants