Re-enable composefs, this time with a proposed patch to avoid test failures
authorSimon McVittie <smcv@debian.org>
Wed, 24 Jan 2024 14:50:04 +0000 (14:50 +0000)
committerSimon McVittie <smcv@debian.org>
Wed, 24 Jan 2024 14:50:04 +0000 (14:50 +0000)
We can't run its unit tests on a Debian 12 kernel with /var/tmp on tmpfs,
but they do pass on a sid kernel or if /var/tmp is disk-backed.

debian/patches/series
debian/patches/tests-Skip-composefs-test-if-var-tmp-does-not-support-use.patch [new file with mode: 0644]
debian/rules

index 78a4e0547fbb9feebe11efa44337bcd1ba1df821..60f6179ea36e04a52013adfb9d96abdaab439f5a 100644 (file)
@@ -1,3 +1,4 @@
+tests-Skip-composefs-test-if-var-tmp-does-not-support-use.patch
 debian/Skip-test-pull-repeated-during-CI.patch
 debian/test-sysroot-Skip-on-s390x-by-default.patch
 debian/Skip-test-admin-deploy-uboot.sh-on-s390x.patch
diff --git a/debian/patches/tests-Skip-composefs-test-if-var-tmp-does-not-support-use.patch b/debian/patches/tests-Skip-composefs-test-if-var-tmp-does-not-support-use.patch
new file mode 100644 (file)
index 0000000..dbda0d1
--- /dev/null
@@ -0,0 +1,26 @@
+From: Simon McVittie <smcv@debian.org>
+Date: Wed, 24 Jan 2024 13:55:12 +0000
+Subject: tests: Skip composefs test if /var/tmp does not support user xattrs
+
+Otherwise, this test fails on Debian 12 (Linux 6.1) kernels if /var/tmp
+is a tmpfs. Some autobuilders put the entire build chroot on a tmpfs,
+to speed up builds.
+
+Signed-off-by: Simon McVittie <smcv@debian.org>
+Forwarded: https://github.com/ostreedev/ostree/pull/3145
+---
+ tests/test-composefs.sh | 1 +
+ 1 file changed, 1 insertion(+)
+
+diff --git a/tests/test-composefs.sh b/tests/test-composefs.sh
+index 4b91973..d05579f 100755
+--- a/tests/test-composefs.sh
++++ b/tests/test-composefs.sh
+@@ -24,6 +24,7 @@ if ! ${CMD_PREFIX} ostree --version | grep -q -e '- composefs'; then
+     exit 0
+ fi
++skip_without_user_xattrs
+ setup_test_repository "bare-user"
index 6095fef48c5cdb1ce2abab10e6a756496a1ed1a6..e465d7209a94033c805e2196623698adec208feb 100755 (executable)
@@ -17,6 +17,7 @@ endif
 configure_options = \
        --enable-installed-tests \
        --with-avahi \
+       --with-composefs \
        --with-curl \
        --with-dracut \
        --with-grub2 \
@@ -29,7 +30,6 @@ configure_options = \
        --with-selinux \
        --with-systemdsystemgeneratordir=/usr/lib/systemd/system-generators \
        --with-systemdsystemunitdir=/usr/lib/systemd/system \
-       --without-composefs \
        --without-smack \
        $(NULL)