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

Fix running executor_pid_test on ZFS #234

Merged
merged 1 commit into from
Dec 1, 2024

Conversation

ngie-eign
Copy link
Contributor

@ngie-eign ngie-eign commented Nov 30, 2024

The testcases prior to this change relied on setting UF_NOUNLINK, which is unfortunately bugged on ZFS: ZFS does not treat UF_NOUNLINK as a valid value for fflags, which causes the testcases to fail.

Switch to SF_NOUNLINK. This particular fflag works work ZFS, unlike UF_NOUNLINK, but requires root privileges, so add the needed metadata to the Kyuafile.

As part of this, I needed to convert the test from atf-c++(3) to atf-c(3). The atf-c++(3) APIs do not have the printf-like equivalent formatting, so it was very difficult figuring out why things weren't working in atf-c++(3).

While here, clean up the code to DRY some of the repeated logic that it contained.

The UF_NOUNLINK bug has been reported in openzfs/zfs#16809 .

Fixes: e26c804

@ngie-eign ngie-eign requested review from emaste and vangyzen November 30, 2024 16:44
The testcases prior to this change relied on setting UF_NOUNLINK, which
is unfortunately bugged on ZFS: ZFS does not treat UF_NOUNLINK as a valid
value for fflags, which causes the testcases to fail.

Switch to SF_NOUNLINK. This particular fflag works work ZFS, unlike
UF_NOUNLINK, but requires root privileges, so add the needed metadata
to the Kyuafile.

As part of this, I needed to convert the test from atf-c++(3) to
atf-c(3). The atf-c++(3) APIs do not have the printf-like equivalent
formatting, so it was very difficult figuring out why things weren't
working in atf-c++(3).

While here, clean up the code to DRY some of the repeated logic that it
contained.

The UF_NOUNLINK bug has been reported in openzfs/zfs#16809 .

Fixes:  e26c804
@ngie-eign ngie-eign force-pushed the fix-executor-pid-test-on-zfs branch from f0c14f1 to 16ab764 Compare December 1, 2024 01:27
@ngie-eign ngie-eign merged commit 6682751 into freebsd:master Dec 1, 2024
@ngie-eign ngie-eign deleted the fix-executor-pid-test-on-zfs branch December 1, 2024 01:29
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