Skip to content

DAOS-18086 control: Fix data race in drpc unit test#17979

Open
kjacque wants to merge 1 commit intomasterfrom
kjacque/drpc-utest-data-race
Open

DAOS-18086 control: Fix data race in drpc unit test#17979
kjacque wants to merge 1 commit intomasterfrom
kjacque/drpc-utest-data-race

Conversation

@kjacque
Copy link
Copy Markdown
Contributor

@kjacque kjacque commented Apr 10, 2026

This issue was only affecting unit tests.

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

This issue was only affecting unit tests.

Signed-off-by: Kris Jacque <kris.jacque@hpe.com>
@kjacque kjacque requested review from mjmac and tanabarr April 10, 2026 17:59
@kjacque kjacque self-assigned this Apr 10, 2026
@kjacque kjacque requested review from a team as code owners April 10, 2026 18:00
@kjacque
Copy link
Copy Markdown
Contributor Author

kjacque commented Apr 10, 2026

Scripted "go test -race" runs for this test 100 times in a row. I was able to reproduce the original issue within 100 runs on master, but not with this patch.

@github-actions
Copy link
Copy Markdown

Ticket title is 'Unit Tests / Unit Test on EL 8.8 / UTEST_control.drpc.TestServer_Listen_AcceptConnection: data race'
Status is 'In Review'
https://daosio.atlassian.net/browse/DAOS-18086

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