-
Notifications
You must be signed in to change notification settings - Fork 553
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
tools/cmake_test: remove accidental env override #17027
tools/cmake_test: remove accidental env override #17027
Conversation
/dt |
/ci-repeat 1 |
/dt |
ducktape was retried in https://buildkite.com/redpanda/redpanda/builds/46105#018e3799-a497-4b64-9106-d5c0667ec996 ducktape was retried in https://buildkite.com/redpanda/redpanda/builds/46484#018e5a8e-13a7-4ed8-9e5e-6b575d192630 |
Requires #17026 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice find
This nuked the above set environment variables. Noticed this because unit tests were succeeded with UBSAN warnings which meant that `halt_on_error=1:abort_on_error=1` didn't take effect.
cd4db58
to
40e1797
Compare
new failures in https://buildkite.com/redpanda/redpanda/builds/46484#018e5a9f-c206-47f8-8256-d18f20ecf000:
new failures in https://buildkite.com/redpanda/redpanda/builds/46491#018e5ba2-0ad4-476e-b077-e69bceb7fcad:
new failures in https://buildkite.com/redpanda/redpanda/builds/46491#018e5bb4-b8be-475c-b87d-963e63fb695b:
|
/dt |
/backport v23.3.x |
/backport v23.2.x |
This nuked the above set environment variables. Noticed this because unit tests were succeeded with UBSAN warnings which meant that
halt_on_error=1:abort_on_error=1
didn't take effect.Backports Required
Release Notes