Skip to content

DAOS-19333 build: Don't fail release builds on memcheck - #18747

Open
mjmac wants to merge 2 commits into
masterfrom
mjmac/DAOS-19333
Open

DAOS-19333 build: Don't fail release builds on memcheck#18747
mjmac wants to merge 2 commits into
masterfrom
mjmac/DAOS-19333

Conversation

@mjmac

@mjmac mjmac commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

The memcheck stage uses a separate instrumented build for Go
binaries regardless of the BUILD_TYPE value.

faults-enabled: false
Signed-off-by: Michael MacDonald github@macdonald.cx

@github-actions

Copy link
Copy Markdown

Ticket title is 'NLT can not be run with release binary'
Status is 'In Progress'
https://daosio.atlassian.net/browse/DAOS-19333

@daosbuild3

Copy link
Copy Markdown
Collaborator

The memcheck stage uses a separate instrumented build for Go
binaries regardless of the BUILD_TYPE value.

faults-enabled: false
Signed-off-by: Michael MacDonald <github@macdonald.cx>
@mjmac
mjmac force-pushed the mjmac/DAOS-19333 branch from fad9913 to d7da9f3 Compare July 30, 2026 16:33
Release builds compile out fault injection, so the stage has nothing
to test and fails on the missing test logs.

faults-enabled: false
Signed-off-by: Michael MacDonald <github@macdonald.cx>
@mjmac
mjmac marked this pull request as ready for review August 3, 2026 14:11
@mjmac
mjmac requested review from a team as code owners August 3, 2026 14:11
@mjmac

mjmac commented Aug 4, 2026

Copy link
Copy Markdown
Contributor Author

@grom72: I'm happier with this version. I used faults-enabled: false to test it, which if I'm understanding the build stuff correctly results in the same configuration as a tagged release build.

@mjmac
mjmac requested a review from kjacque August 4, 2026 17:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants