Skip to content

[core] Deflake gcs_client_test.cc#34411

Merged
fishbone merged 1 commit intoray-project:masterfrom
cadedaniel:deflake-gcs-client-test
Apr 17, 2023
Merged

[core] Deflake gcs_client_test.cc#34411
fishbone merged 1 commit intoray-project:masterfrom
cadedaniel:deflake-gcs-client-test

Conversation

@cadedaniel
Copy link
Copy Markdown
Member

  • Hypothesis is that on_subscribe callback is invoked after test finishes; the reference to the stack-allocated atomic counter is no longer valid, causing asan failure.
  • I don't understand the intention of this test. The counter is actually unused. Should I re-add it on-heap and then assert a correct value? Couldn't determine from git blame the intention.

Signed-off-by: Cade Daniel <edacih@gmail.com>
@cadedaniel cadedaniel changed the title Deflake gcs_client_test.cc [core] Deflake gcs_client_test.cc Apr 14, 2023
@cadedaniel cadedaniel linked an issue Apr 14, 2023 that may be closed by this pull request
@cadedaniel cadedaniel marked this pull request as ready for review April 14, 2023 22:28
@cadedaniel cadedaniel requested a review from a team as a code owner April 14, 2023 22:28
@fishbone fishbone merged commit 0dafee9 into ray-project:master Apr 17, 2023
@cadedaniel cadedaniel deleted the deflake-gcs-client-test branch April 17, 2023 19:26
justinvyu pushed a commit to justinvyu/ray that referenced this pull request Apr 18, 2023
Hypothesis is that on_subscribe callback is invoked after test finishes; the reference to the stack-allocated atomic counter is no longer valid, causing asan failure.
@rkooo567
Copy link
Copy Markdown
Contributor

Screen Shot 2023-04-17 at 5 37 50 PM

hmm unfortunately, this doesn't seem to fix issues

elliottower pushed a commit to elliottower/ray that referenced this pull request Apr 22, 2023
Hypothesis is that on_subscribe callback is invoked after test finishes; the reference to the stack-allocated atomic counter is no longer valid, causing asan failure.

Signed-off-by: elliottower <elliot@elliottower.com>
ProjectsByJackHe pushed a commit to ProjectsByJackHe/ray that referenced this pull request May 4, 2023
Hypothesis is that on_subscribe callback is invoked after test finishes; the reference to the stack-allocated atomic counter is no longer valid, causing asan failure.

Signed-off-by: Jack He <jackhe2345@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CI] linux://:gcs_client_test is failing/flaky on master.

3 participants