[SCM] PostGIS branch master updated. 3.7.0beta1-175-g08f9832fb3

git at osgeo.org git at osgeo.org
Sun Aug 2 01:00:37 PDT 2026


This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "PostGIS".

The branch, master has been updated
       via  08f9832fb36e066a57e53f285cf33914e9061aaf (commit)
       via  9067ee7ba05ea014c12f40582715339bd11043f2 (commit)
      from  7bc10724ea488d17b005ef88d7135c9ce34d0558 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 08f9832fb36e066a57e53f285cf33914e9061aaf
Merge: 7bc10724ea 9067ee7ba0
Author: Darafei Praliaskouski <komzpa at gmail.com>
Date:   Sun Aug 2 01:00:36 2026 -0700

    Merge pull request 'ci: stop ignoring failures in the usan-clang tier' (!645) from Komzpa/postgis:ci/usan-clang-cu-misc-20260801 into master
    
    `qa/usan-clang` is muted, so its result cannot fail a pipeline. A check that cannot fail is not a
    check; this removes the mute so the tier speaks for itself.
    
    Reviewed-on: https://gitea.osgeo.org/postgis/postgis/pulls/645


commit 9067ee7ba05ea014c12f40582715339bd11043f2
Author: Darafei Praliaskouski <me at komzpa.net>
Date:   Sat Aug 1 04:21:27 2026 +0400

    ci: stop ignoring failures in the usan-clang QA step
    
    The step was muted with this reason:
    
        # Current master trips this sanitizer in liblwgeom/cunit/cu_misc.c;
        # keep the signal visible without blocking unrelated pull requests.
    
    Both halves have stopped being true. The cu_misc.c finding was fixed by
    a73d06ef7 and the recipe exits 0 on fresh master, so there is nothing left to
    shield anyone from. And the mute is not protecting unrelated pull requests: it
    hides failures in the pull requests that cause them.
    
    Across the 200 most recent pipelines the step ran 140 times to completion --
    133 success, 7 failure -- and every one of those seven was ignored, leaving its
    pull request green. They are not sanitizer diagnostics; not one log contains a
    runtime error: line. The step runs the QA suite inside a sanitizer build, so
    what it swallowed were ordinary test failures:
    
        pipeline 6232, PR 434  topogeo_addlinestring, totopogeom          Failed: 2
        pipeline 6336, PR 429  topogeo_addlinestring_robust               Failed: 1
        pipeline 6331, PR 595  raster overview extents                    Failed: 1
    
    Each was reproduced locally and each is branch-caused: master at f0c84f8c6
    passes all three. So the step was reporting correctly and only the mute was
    discarding the report.
    
    Worth stating plainly, because anyone checking today will find those pull
    requests green: all three branches have since moved on and their current heads
    pass. The mute did not cause permanent damage. What it cost was time -- the
    authors were never shown a failure they had introduced, and each was found
    again later by someone else. That is the argument for removing it, not a claim
    that defects are sitting there now.
    
    Expect this to turn red any pull request that introduces a QA failure, which is
    the point.

diff --git a/.woodpecker/qa.yml b/.woodpecker/qa.yml
index 24fdf4e4da..1ebab3437d 100644
--- a/.woodpecker/qa.yml
+++ b/.woodpecker/qa.yml
@@ -33,9 +33,6 @@ steps:
   usan-clang:
     image: *test-image
     depends_on: autogen
-    # Current master trips this sanitizer in liblwgeom/cunit/cu_misc.c;
-    # keep the signal visible without blocking unrelated pull requests.
-    failure: ignore
     when:
       - matrix:
           QA_MODE: usan_clang

-----------------------------------------------------------------------

Summary of changes:
 .woodpecker/qa.yml | 3 ---
 1 file changed, 3 deletions(-)


hooks/post-receive
-- 
PostGIS


More information about the postgis-tickets mailing list