git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH v2 8/8] ci: improve reftable test coverage

From
Patrick Steinhardt <ps@pks.im>
Date
Oct 9, 2026, 11:32 UTC
Message-ID
<20261009-pks-ci-housekeeping-v2-8-6863d58ef691@pks.im>
In-Reply-To
<20261009-pks-ci-housekeeping-v2-0-6863d58ef691@pks.im>

The "linux-reftable" job exercises Git with reftables as its default backend. But this job is arguably redundant because we already have the "linux-reftable-leaks" job that exercises reftables with the leak sanitizer enabled, and it is unlikely that we will catch any extra bugs with the leak sanitizer disabled.

One may thus be triggered to just drop the "linux-reftable" job in favor of the leak-checking variant. But the latter disables a couple of other tests that exercise git-p4, git-svn and other tools. Those tools may not be an essential part of Git nowadays, but I don't quite feel comfortable with reducing test coverage.

Instead, adapt the job to run reftables together with SHA256. which is an area that we don't have test coverage for.

Signed-off-by: Patrick Steinhardt <ps@pks.im>
---
 .github/workflows/main.yml | 2 +-
 .gitlab-ci.yml             | 2 +-
 ci/run-build-and-tests.sh  | 6 +++++-
 3 files changed, 7 insertions(+), 3 deletions(-)
diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml
index 5b772dab55..0939833e57 100644
--- a/.github/workflows/main.yml
+++ b/.github/workflows/main.yml
@@ -405,7 +405,7 @@ jobs:
         - jobname: linux-sha256
           image: ubuntu:rolling
           cc: clang
-        - jobname: linux-reftable
+        - jobname: linux-reftable-sha256
           image: ubuntu:rolling
           cc: clang
         - jobname: linux-exotic
diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index de40434ae3..b814963ff7 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -39,7 +39,7 @@ test:linux:
       - jobname: linux-sha256
         image: ubuntu:rolling
         CC: clang
-      - jobname: linux-reftable
+      - jobname: linux-reftable-sha256
         image: ubuntu:rolling
         CC: clang
       - jobname: linux-exotic
diff --git a/ci/run-build-and-tests.sh b/ci/run-build-and-tests.sh
index 9381ff8893..c6e61d5794 100755
--- a/ci/run-build-and-tests.sh
+++ b/ci/run-build-and-tests.sh
@@ -40,7 +40,11 @@ linux-exotic)
 linux-sha256)
 	export GIT_TEST_DEFAULT_HASH=sha256
 	;;
-linux-reftable|linux-reftable-leaks|osx-reftable)
+linux-reftable-sha256)
+	export GIT_TEST_DEFAULT_HASH=sha256
+	export GIT_TEST_DEFAULT_REF_STORAGE_FORMAT=reftable
+	;;
+linux-reftable-leaks|osx-reftable)
 	export GIT_TEST_DEFAULT_REF_STORAGE_FORMAT=reftable
 	;;
 
-- 
2.56.0.170.g584c36229d.dirty
Previous: Patrick Steinhardt
Message 27 of 27 in “ci: some housekeeping and modernizations”
  1. 0/8 ci: some housekeeping and modernizationsPatrick Steinhardt, Oct 8, 2026
  2. 1/8 t5004: skip SHA-1-only test in SHA-256 repositoryPatrick Steinhardt, Oct 8, 2026
  3. Junio C HamanoOct 8, 2026
  4. Patrick SteinhardtOct 9, 2026
  5. 2/8 ci: fix "fedora-breaking-changes-meson" jobPatrick Steinhardt, Oct 8, 2026
  6. 3/8 ci: drop unused "linux-clang" logicPatrick Steinhardt, Oct 8, 2026
  7. 4/8 ci: switch away from unsupported i386/ubuntu imagePatrick Steinhardt, Oct 8, 2026
  8. Todd ZullingerOct 8, 2026
  9. Junio C HamanoOct 8, 2026
  10. Patrick SteinhardtOct 9, 2026
  11. 5/8 ci: rename linux-TEST-vars jobPatrick Steinhardt, Oct 8, 2026
  12. 6/8 ci: switch away from EOL'd Ubuntu version in linux-exoticPatrick Steinhardt, Oct 8, 2026
  13. 7/8 ci: drop now-dead Python 2 coveragePatrick Steinhardt, Oct 8, 2026
  14. 8/8 ci: drop redundant linux-reftable jobPatrick Steinhardt, Oct 8, 2026
  15. Junio C HamanoOct 8, 2026
  16. Patrick SteinhardtOct 9, 2026
  17. Junio C HamanoOct 9, 2026
  18. 0/8 ci: some housekeeping and modernizationsPatrick Steinhardt, Oct 9, 2026
  19. 1/8 t5004: skip SHA-1-only test in SHA-256 repositoryPatrick Steinhardt, Oct 9, 2026
  20. 2/8 ci: fix "fedora-breaking-changes-meson" jobPatrick Steinhardt, Oct 9, 2026
  21. 3/8 ci: drop unused "linux-clang" logicPatrick Steinhardt, Oct 9, 2026
  22. 4/8 ci: switch away from unsupported i386/ubuntu imagePatrick Steinhardt, Oct 9, 2026
  23. 5/8 ci: rename linux-TEST-vars jobPatrick Steinhardt, Oct 9, 2026
  24. SZEDER GáborOct 9, 2026
  25. 6/8 ci: switch away from EOL'd Ubuntu version in linux-exoticPatrick Steinhardt, Oct 9, 2026
  26. 7/8 ci: drop now-dead Python 2 coveragePatrick Steinhardt, Oct 9, 2026
  27. 8/8 ci: improve reftable test coveragePatrick Steinhardt, Oct 9, 2026

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.