Skip to content

ci: recognize glibc heap-corruption aborts in ci-annotate-errors - #38702

Open
ggevay wants to merge 1 commit into
MaterializeInc:mainfrom
ggevay:gabor/ci-annotate-glibc-aborts
Open

ci: recognize glibc heap-corruption aborts in ci-annotate-errors#38702
ggevay wants to merge 1 commit into
MaterializeInc:mainfrom
ggevay:gabor/ci-annotate-glibc-aborts

Conversation

@ggevay

@ggevay ggevay commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

Motivation

Nightly 18265's testdrive with FoundationDB 1 died with malloc(): unsorted double linked list corrupted on testdrive's stderr (PER-82). ERROR_RE in ci_annotate_errors.py has no alternative for glibc's heap-consistency messages, so the job was annotated as "failed, but no error in logs found", no ci-regexp could attach to it, and the failure history has no record of the signature. Part of QAR-158.

Description

Add the glibc heap-abort messages to ERROR_RE: malloc()/free()/realloc()/munmap_chunk():, double free or corruption, corrupted size vs. prev_size, corrupted double-linked list, and stack smashing detected. Adds test_glibc_heap_abort_is_an_error to ci_annotate_errors_test.py; it fails without the pattern change.

Verification

The new test and the existing annotator tests pass; black and ruff are clean on both files.

🤖 Generated with Claude Code

A process that dies of a glibc heap-consistency abort (malloc(): ...,
free(): ..., double free or corruption, stack smashing) leaves only that
one stderr line behind. ERROR_RE did not match it, so the job annotated
as "failed, but no error in logs found", no known-issue regexp could
attach, and the failure history could not record it.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@ggevay
ggevay marked this pull request as ready for review September 7, 2026 17:12
@ggevay
ggevay requested a review from a team as a code owner September 7, 2026 17:13
@ggevay ggevay added the T-testing Theme: tests or test infrastructure label Sep 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T-testing Theme: tests or test infrastructure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant