Skip to content

DAOS-18343 test: Suppress Go runtime false positive (#17346)#17456

Merged
daltonbohning merged 1 commit intorelease/2.6from
kjacque/2.6/nlt-valgrind-shadowset
Jan 29, 2026
Merged

DAOS-18343 test: Suppress Go runtime false positive (#17346)#17456
daltonbohning merged 1 commit intorelease/2.6from
kjacque/2.6/nlt-valgrind-shadowset

Conversation

@kjacque
Copy link
Contributor

@kjacque kjacque commented Jan 26, 2026

Suppress a valgrind false positive for Go runtime function ShadowSet.

Steps for the author:

  • Commit message follows the guidelines.
  • Appropriate Features or Test-tag pragmas were used.
  • Appropriate Functional Test Stages were run.
  • At least two positive code reviews including at least one code owner from each category referenced in the PR.
  • Testing is complete. If necessary, forced-landing label added and a reason added in a comment.

After all prior steps are complete:

  • Gatekeeper requested (daos-gatekeeper added as a reviewer).

Suppress a valgrind false positive for Go runtime function
ShadowSet.

Signed-off-by: Kris Jacque <[email protected]>
@kjacque kjacque requested review from a team January 26, 2026 22:34
@kjacque kjacque self-assigned this Jan 26, 2026
@kjacque kjacque requested a review from a team as a code owner January 26, 2026 22:34
@kjacque kjacque requested a review from a team as a code owner January 26, 2026 22:34
@github-actions
Copy link

Ticket title is 'NLT valgrind - uninitialized value in function ShadowSet'
Status is 'Awaiting backport'
Job should run at elevated priority (1)
https://daosio.atlassian.net/browse/DAOS-18343

@github-actions github-actions bot added the priority Ticket has high priority (automatically managed) label Jan 26, 2026
@daosbuild3
Copy link
Collaborator

@kjacque kjacque added the clean-cherry-pick Cherry-pick from another branch that did not require additional edits label Jan 29, 2026
@kjacque
Copy link
Contributor Author

kjacque commented Jan 29, 2026

The NLT failures include one known issue and a different false positive not related to this one:

@kjacque kjacque added the forced-landing The PR has known failures or has intentionally reduced testing, but should still be landed. label Jan 29, 2026
@kjacque kjacque requested a review from a team January 29, 2026 23:04
@daltonbohning daltonbohning merged commit eb42dec into release/2.6 Jan 29, 2026
48 of 50 checks passed
@daltonbohning daltonbohning deleted the kjacque/2.6/nlt-valgrind-shadowset branch January 29, 2026 23:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

clean-cherry-pick Cherry-pick from another branch that did not require additional edits forced-landing The PR has known failures or has intentionally reduced testing, but should still be landed. priority Ticket has high priority (automatically managed) release-2.6.5

Development

Successfully merging this pull request may close these issues.

4 participants