Page MenuHomePhabricator

[CI] Workaround ubsan failure in functional tests

Authored by Fabien on Sep 4 2019, 15:41.



This will allow to run the undefined behavior sanitizer on the
functional tests.
For now the abc-p2p-compactblocks functional test is failing with ubsan (a message is missed, with no log) and the node fails to shut down, which causes the build to hang until timeout. This test is excluded from the list in order to allow for the other tests to run with ubsan, waiting for a fix.

Depends on D3993.

Test Plan

Run the CI build-ubsan build configuration.

Diff Detail

rABC Bitcoin ABC
Lint Not Applicable
Tests Not Applicable

Event Timeline

jasonbcox requested changes to this revision.Sep 6 2019, 21:26
jasonbcox added a subscriber: jasonbcox.

A better description/rationale as we discussed earlier.

This revision now requires changes to proceed.Sep 6 2019, 21:26

Rebase, improve summary and add a comment asking for removal in the future.

It would probably be more useful to fix the undefined behavior.

I'd rather fix it, but so far I failed to do so.
This is most likely not an undefined behavior, because ubsan doesn't catch anything, I rather suspect some kind of timing issue exhibited by running ubsan.

This revision is now accepted and ready to land.Sep 9 2019, 16:35