Skip to content

Commit

Permalink
Fix running executor_pid_test on ZFS
Browse files Browse the repository at this point in the history 
The testcases prior to this change relied on setting UF_UNLINK, which
is unfortunately bugged on ZFS: ZFS does not treat UF_UNLINK as a valid
value for fflags, which causes the testcases to fail.

Switch to SF_UNLINK. This particular fflag 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_UNLINK bug has been reported in openzfs/zfs#16809 .
  • Loading branch information
@ngie-eign
ngie-eign committed Nov 29, 2024
1 parent d0a1ffa commit 8be05bc
Show file tree
Hide file tree
Showing 3 changed files with 86 additions and 84 deletions.
2 changes: 1 addition & 1 deletion utils/process/Kyuafile
View file
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ atf_test_program{name="child_test"}
atf_test_program{name="deadline_killer_test"}
atf_test_program{name="exceptions_test"}
atf_test_program{name="executor_test"}
atf_test_program{name="executor_pid_test"}
atf_test_program{name="executor_pid_test", required_user="root"}
atf_test_program{name="fdstream_test"}
atf_test_program{name="isolation_test"}
atf_test_program{name="operations_test"}
Expand Down
4 changes: 2 additions & 2 deletions utils/process/Makefile.am.inc
View file
Original file line number Diff line number Diff line change
Expand Up @@ -84,8 +84,8 @@ utils_process_executor_test_CXXFLAGS = $(UTILS_CFLAGS) $(ATF_CXX_CFLAGS)
utils_process_executor_test_LDADD = $(UTILS_LIBS) $(ATF_CXX_LIBS)

tests_utils_process_PROGRAMS += utils/process/executor_pid_test
utils_process_executor_pid_test_SOURCES = utils/process/executor_pid_test.cpp
utils_process_executor_pid_test_CXXFLAGS = $(UTILS_CFLAGS) $(ATF_CXX_CFLAGS)
utils_process_executor_pid_test_SOURCES = utils/process/executor_pid_test.c
utils_process_executor_pid_test_CFLAGS = $(UTILS_CFLAGS) $(ATF_CXX_CFLAGS)
utils_process_executor_pid_test_LDADD = $(UTILS_LIBS) $(ATF_CXX_LIBS)

tests_utils_process_PROGRAMS += utils/process/fdstream_test
Expand Down
164 changes: 83 additions & 81 deletions utils/process/executor_pid_test.cpp → utils/process/executor_pid_test.c
View file
Original file line number Diff line number Diff line change
Expand Up @@ -69,23 +69,28 @@ Abort trap (core dumped)

#include <sys/stat.h>

#include <atf-c++.hpp>
#include <atf-c.h>

#include <errno.h>
#include <fcntl.h>
#include <signal.h>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <unistd.h>

#include <cerrno>
#include <cstring>

void
leak_work_dir()
{
int fd;

ATF_REQUIRE((fd = open("unforgettable", O_CREAT|O_EXCL|O_WRONLY, 0600))
>= 0);
ATF_REQUIRE_EQ(0, fchflags(fd, UF_NOUNLINK));
fd = open("unforgettable", O_CREAT|O_EXCL|O_WRONLY, 0600);
ATF_REQUIRE_MSG(fd != -1,
"open(..., O_CREAT|O_EXCL|O_WRONLY, 0600) failed unexpectedly: %s",
strerror(errno));
ATF_REQUIRE_EQ_MSG(0, fchflags(fd, SF_NOUNLINK),
"fchflags(..., UF_NOUNLINK): failed unexpectedly: %s",
strerror(errno));
ATF_REQUIRE_EQ(0, close(fd));
}

Expand Down Expand Up @@ -130,79 +135,76 @@ clean_up()
(void)system("chflags -R nouunlink ../..");
}

ATF_TEST_CASE_WITHOUT_HEAD(leak_0);
ATF_TEST_CASE_BODY(leak_0) { leak_work_dir(); }
ATF_TEST_CASE_WITHOUT_HEAD(leak_1);
ATF_TEST_CASE_BODY(leak_1) { leak_work_dir(); }
ATF_TEST_CASE_WITHOUT_HEAD(leak_2);
ATF_TEST_CASE_BODY(leak_2) { leak_work_dir(); }
ATF_TEST_CASE_WITHOUT_HEAD(leak_3);
ATF_TEST_CASE_BODY(leak_3) { leak_work_dir(); }
ATF_TEST_CASE_WITHOUT_HEAD(leak_4);
ATF_TEST_CASE_BODY(leak_4) { leak_work_dir(); }
ATF_TEST_CASE_WITHOUT_HEAD(leak_5);
ATF_TEST_CASE_BODY(leak_5) { leak_work_dir(); }
ATF_TEST_CASE_WITHOUT_HEAD(leak_6);
ATF_TEST_CASE_BODY(leak_6) { leak_work_dir(); }
ATF_TEST_CASE_WITHOUT_HEAD(leak_7);
ATF_TEST_CASE_BODY(leak_7) { leak_work_dir(); }
ATF_TEST_CASE_WITHOUT_HEAD(leak_8);
ATF_TEST_CASE_BODY(leak_8) { leak_work_dir(); }
ATF_TEST_CASE_WITHOUT_HEAD(leak_9);
ATF_TEST_CASE_BODY(leak_9) { leak_work_dir(); }

ATF_TEST_CASE_WITHOUT_HEAD(pid_wrap);
ATF_TEST_CASE_BODY(pid_wrap) { wrap_pids(); }

ATF_TEST_CASE_WITHOUT_HEAD(pid_wrap_0);
ATF_TEST_CASE_BODY(pid_wrap_0) { test_work_dir_reuse(); }
ATF_TEST_CASE_WITHOUT_HEAD(pid_wrap_1);
ATF_TEST_CASE_BODY(pid_wrap_1) { test_work_dir_reuse(); }
ATF_TEST_CASE_WITHOUT_HEAD(pid_wrap_2);
ATF_TEST_CASE_BODY(pid_wrap_2) { test_work_dir_reuse(); }
ATF_TEST_CASE_WITHOUT_HEAD(pid_wrap_3);
ATF_TEST_CASE_BODY(pid_wrap_3) { test_work_dir_reuse(); }
ATF_TEST_CASE_WITHOUT_HEAD(pid_wrap_4);
ATF_TEST_CASE_BODY(pid_wrap_4) { test_work_dir_reuse(); }
ATF_TEST_CASE_WITHOUT_HEAD(pid_wrap_5);
ATF_TEST_CASE_BODY(pid_wrap_5) { test_work_dir_reuse(); }
ATF_TEST_CASE_WITHOUT_HEAD(pid_wrap_6);
ATF_TEST_CASE_BODY(pid_wrap_6) { test_work_dir_reuse(); }
ATF_TEST_CASE_WITHOUT_HEAD(pid_wrap_7);
ATF_TEST_CASE_BODY(pid_wrap_7) { test_work_dir_reuse(); }
ATF_TEST_CASE_WITHOUT_HEAD(pid_wrap_8);
ATF_TEST_CASE_BODY(pid_wrap_8) { test_work_dir_reuse(); }
ATF_TEST_CASE_WITHOUT_HEAD(pid_wrap_9);
ATF_TEST_CASE_BODY(pid_wrap_9) { test_work_dir_reuse(); }

ATF_TEST_CASE_WITHOUT_HEAD(really_clean_up);
ATF_TEST_CASE_BODY(really_clean_up) { clean_up(); }

ATF_INIT_TEST_CASES(tcs)
#define LEAK_WORKDIR_TC(name) \
ATF_TC_WITHOUT_HEAD(name); \
ATF_TC_BODY(name, tc) { \
leak_work_dir(); \
}

LEAK_WORKDIR_TC(leak_0)
LEAK_WORKDIR_TC(leak_1)
LEAK_WORKDIR_TC(leak_2)
LEAK_WORKDIR_TC(leak_3)
LEAK_WORKDIR_TC(leak_4)
LEAK_WORKDIR_TC(leak_5)
LEAK_WORKDIR_TC(leak_6)
LEAK_WORKDIR_TC(leak_7)
LEAK_WORKDIR_TC(leak_8)
LEAK_WORKDIR_TC(leak_9)

ATF_TC_WITHOUT_HEAD(pid_wrap);
ATF_TC_BODY(pid_wrap, tc) { wrap_pids(); }

#define PID_WRAP_TC(name) \
ATF_TC_WITHOUT_HEAD(name); \
ATF_TC_BODY(name, tc) { \
test_work_dir_reuse(); \
}

PID_WRAP_TC(pid_wrap_0)
PID_WRAP_TC(pid_wrap_1)
PID_WRAP_TC(pid_wrap_2)
PID_WRAP_TC(pid_wrap_3)
PID_WRAP_TC(pid_wrap_4)
PID_WRAP_TC(pid_wrap_5)
PID_WRAP_TC(pid_wrap_6)
PID_WRAP_TC(pid_wrap_7)
PID_WRAP_TC(pid_wrap_8)
PID_WRAP_TC(pid_wrap_9)

ATF_TC_WITHOUT_HEAD(really_clean_up);
ATF_TC_BODY(really_clean_up, tc)
{
clean_up();
}

ATF_TP_ADD_TCS(tcs)
{
ATF_ADD_TEST_CASE(tcs, leak_0);
ATF_ADD_TEST_CASE(tcs, leak_1);
ATF_ADD_TEST_CASE(tcs, leak_2);
ATF_ADD_TEST_CASE(tcs, leak_3);
ATF_ADD_TEST_CASE(tcs, leak_4);
ATF_ADD_TEST_CASE(tcs, leak_5);
ATF_ADD_TEST_CASE(tcs, leak_6);
ATF_ADD_TEST_CASE(tcs, leak_7);
ATF_ADD_TEST_CASE(tcs, leak_8);
ATF_ADD_TEST_CASE(tcs, leak_9);

ATF_ADD_TEST_CASE(tcs, pid_wrap);

ATF_ADD_TEST_CASE(tcs, pid_wrap_0);
ATF_ADD_TEST_CASE(tcs, pid_wrap_1);
ATF_ADD_TEST_CASE(tcs, pid_wrap_2);
ATF_ADD_TEST_CASE(tcs, pid_wrap_3);
ATF_ADD_TEST_CASE(tcs, pid_wrap_4);
ATF_ADD_TEST_CASE(tcs, pid_wrap_5);
ATF_ADD_TEST_CASE(tcs, pid_wrap_6);
ATF_ADD_TEST_CASE(tcs, pid_wrap_7);
ATF_ADD_TEST_CASE(tcs, pid_wrap_8);
ATF_ADD_TEST_CASE(tcs, pid_wrap_9);

ATF_ADD_TEST_CASE(tcs, really_clean_up);
ATF_TP_ADD_TC(tcs, leak_0);
ATF_TP_ADD_TC(tcs, leak_1);
ATF_TP_ADD_TC(tcs, leak_2);
ATF_TP_ADD_TC(tcs, leak_3);
ATF_TP_ADD_TC(tcs, leak_4);
ATF_TP_ADD_TC(tcs, leak_5);
ATF_TP_ADD_TC(tcs, leak_6);
ATF_TP_ADD_TC(tcs, leak_7);
ATF_TP_ADD_TC(tcs, leak_8);
ATF_TP_ADD_TC(tcs, leak_9);

ATF_TP_ADD_TC(tcs, pid_wrap);

ATF_TP_ADD_TC(tcs, pid_wrap_0);
ATF_TP_ADD_TC(tcs, pid_wrap_1);
ATF_TP_ADD_TC(tcs, pid_wrap_2);
ATF_TP_ADD_TC(tcs, pid_wrap_3);
ATF_TP_ADD_TC(tcs, pid_wrap_4);
ATF_TP_ADD_TC(tcs, pid_wrap_5);
ATF_TP_ADD_TC(tcs, pid_wrap_6);
ATF_TP_ADD_TC(tcs, pid_wrap_7);
ATF_TP_ADD_TC(tcs, pid_wrap_8);
ATF_TP_ADD_TC(tcs, pid_wrap_9);

ATF_TP_ADD_TC(tcs, really_clean_up);

return atf_no_error();
}

0 comments on commit 8be05bc

Please sign in to comment.