Skip to content

Re-enable LeakSanitizer runs on Ubuntu 26.04 - #24767

Open
sevan wants to merge 3 commits into
Perl:bleadfrom
sevan:asan-ubuntu-2604-with-leaksanitizer
Open

sevan wants to merge 3 commits into
Perl:bleadfrom
sevan:asan-ubuntu-2604-with-leaksanitizer

Conversation

@sevan

@sevan sevan commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

ubuntu-latest target currently means Ubuntu 24.04, whilst the latest actual release of Ubuntu is 26.04.
Target 26.04 to see if LeakSanitzer still struggles when GDBM support is included.
Part of PR #24765

  • This set of changes does not require a perldelta entry.

ubuntu-latest target currently means Ubuntu 24.04, whilst the latest
actual release of Ubuntu is 26.04.
Target 26.04 to see if LeakSanitzer still struggles when GDBM support
is included.
Part of PR Perl#24765
@sevan

sevan commented Aug 27, 2026

Copy link
Copy Markdown
Contributor Author
op/kvhslice.t ........................................................ ok
==23616==LeakSanitizer has encountered a fatal error.
==23616==HINT: For debugging, try setting environment variable LSAN_OPTIONS=verbosity=1:log_threads=1
==23616==HINT: LeakSanitizer does not work under ptrace (strace, gdb, etc)
op/fork.t ............................................................ ok

@sevan

sevan commented Aug 27, 2026

Copy link
Copy Markdown
Contributor Author

aha, the PERL_UNICODE job built with sanitizers sheds a little light. Note the warning on the first line:

==29183==WARNING: TestPTrace() failed to fork (errno 11)
==29183==LeakSanitizer has encountered a fatal error.
==29183==HINT: For debugging, try setting environment variable LSAN_OPTIONS=verbosity=1:log_threads=1
==29183==HINT: LeakSanitizer does not work under ptrace (strace, gdb, etc)
op/fork.t ............................................................ ok

@jkeenan

jkeenan commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

ubuntu-latest target currently means Ubuntu 24.04, whilst the latest actual release of Ubuntu is 26.04. Target 26.04 to see if LeakSanitzer still struggles when GDBM support is included. Part of PR #24765

  1. Do we need two separate pull requests (this and GH Re-enable LeakSanitizer runs聽#24765)?

  2. Who decides what ubuntu-latest points to? Canonical? GitHub? Us? (I ask because I'm a long-time Ubuntu user and have become accustomed to the pacing of their releases. Though the latest actual release may be 26.04, I haven't actually received prompts to upgrade from 24.04. I expect that will happen in a month or two.)

It results in LeakSanitizer erroring.
==29183==WARNING: TestPTrace() failed to fork (errno 11)
==29183==LeakSanitizer has encountered a fatal error.
==29183==HINT: For debugging, try setting environment variable LSAN_OPTIONS=verbosity=1:log_threads=1
==29183==HINT: LeakSanitizer does not work under ptrace (strace, gdb, etc)
op/fork.t ............................................................ oK
@sevan

sevan commented Aug 27, 2026

Copy link
Copy Markdown
Contributor Author

We need one pull request, but it's just easier from the CI run side to raise 2 pull requests and also it remains here as evidence, given that it is security sensitive.
Regarding who decides, it's Microsoft. See the readme in the github.com/actions/runner-images repo.

@sevan

sevan commented Aug 27, 2026

Copy link
Copy Markdown
Contributor Author

Ok, I dropped the install if libgdbm & libdb, and the error remains. So the issue is not related to libgdbm nor libdb.
It's always when it comes to run op/fork.t it seems, from the logs.

@sevan

sevan commented Aug 27, 2026

Copy link
Copy Markdown
Contributor Author

PR #24765 is the change I want to land. I'm just using this PR to test against Ubuntu 26.04 and will close it regardless of what happens to #24765.

@tonycoz

tonycoz commented Aug 30, 2026

Copy link
Copy Markdown
Contributor

PR #24765 is the change I want to land. I'm just using this PR to test against Ubuntu 26.04 and will close it regardless of what happens to #24765.

You can enable CI in your own fork, there's no need to make a PR:

image

and then set DO_EXTENDED_TESTING:

image image

This will show the status of CI with a little cross or tick when you switch to that branch:

image image

@jkeenan jkeenan added the Infrastructure Things needed to maintain Perl development label Aug 31, 2026
@sevan

sevan commented Aug 31, 2026

Copy link
Copy Markdown
Contributor Author

You can enable CI in your own fork, there's no need to make a PR:

Thanks for the pointer. I had disabled it all because when you're syncing the fork with upstream, it triggers a CI run, and then you get a load of unwanted emails.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Infrastructure Things needed to maintain Perl development

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants