Skip to content

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

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.

Sign up for GitHub

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

Jump to bottom
Merged
daltonbohning merged 1 commit into from
Jan 9, 2026
Merged

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

daltonbohning merged 1 commit into from
Jan 9, 2026
+7 −0

Conversation

@kjacque
Copy link
Contributor

@kjacque kjacque commented Jan 6, 2026 •
edited
Loading

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).

@kjacque
DAOS-18343 test: Suppress Go runtime false positive
8d0db08 
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 6, 2026 23:48
@kjacque kjacque self-assigned this Jan 6, 2026
@kjacque kjacque requested review from a team as code owners January 6, 2026 23:48
@github-actions
Copy link

github-actions bot commented Jan 6, 2026 •
edited
Loading

Ticket title is 'NLT valgrind - uninitialized value in function ShadowSet'
Status is 'In Review'
https://daosio.atlassian.net/browse/DAOS-18343

@daosbuild3
Copy link
Collaborator

daosbuild3 commented Jan 7, 2026

Test stage NLT on EL 8.8 completed with status UNSTABLE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/daos//view/change-requests/job/PR-17346/1/testReport/

@kjacque
Copy link
Contributor Author

kjacque commented Jan 7, 2026

The NLT valgrind issue run into above is DAOS-17416.

@kjacque kjacque requested review from knard38 and mjmac January 8, 2026 21:31
@kjacque kjacque added the forced-landing The PR has known failures or has intentionally reduced testing, but should still be landed. label Jan 9, 2026
@kjacque kjacque requested a review from a team January 9, 2026 16:57
@daltonbohning daltonbohning merged commit ac24ca6 into master Jan 9, 2026
45 of 48 checks passed
@daltonbohning daltonbohning deleted the kjacque/nlt-valgrind-shadowset branch January 9, 2026 17:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tanabarr tanabarr tanabarr approved these changes

@knard38 knard38 knard38 approved these changes

@mjmac mjmac Awaiting requested review from mjmac

Assignees

@kjacque kjacque

Labels

forced-landing The PR has known failures or has intentionally reduced testing, but should still be landed.

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@kjacque @daosbuild3 @tanabarr @knard38 @daltonbohning