Skip to content

[lldb] Temporarily disable actor state test #10451

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.

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

Conversation

jasonmolenda
Copy link

The test added in "Add summary formatter for DefaultActorStorage (#10388)" is checking the actor's state, but it is failing sometimes. Temporarily disable until Dave can look into this, to unblock CI/PR testing.

rdar://148754221

The test added in "Add summary formatter for DefaultActorStorage (swiftlang#10388)"
is checking the actor's state, but it is failing sometimes.
Temporarily disable until Dave can look into this, to unblock
CI/PR testing.

rdar://148754221
@jasonmolenda jasonmolenda requested a review from a team as a code owner April 8, 2025 00:54
@jasonmolenda
Copy link
Author

@swift-ci test

Copy link

@felipepiovezan felipepiovezan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Safest would be to just skip the test, but if this also temporarily gets the job done, lgtm

@jasonmolenda
Copy link
Author

Yeah if the test was flakey I wasn't sure if it's safe to run anywhere, so i'd be adding skipifDarwin, skipifWindows, skipifLinux. maybe that is the better approach.

The test added in "Add summary formatter for DefaultActorStorage (swiftlang#10388)"
is checking the actor's state, but it is failing sometimes.
Temporarily disable until Dave can look into this, to unblock
CI/PR testing.

rdar://148754221
@jasonmolenda
Copy link
Author

@swift-ci test

@jasonmolenda jasonmolenda merged commit 0b855f5 into swiftlang:stable/20240723 Apr 8, 2025
2 checks passed
@jasonmolenda jasonmolenda deleted the disable-actor-status-testing branch April 8, 2025 16:27
jasonmolenda added a commit to jasonmolenda/llvm-project that referenced this pull request Apr 8, 2025
* [lldb] Temporarily disable actor state test

The test added in "Add summary formatter for DefaultActorStorage (swiftlang#10388)"
is checking the actor's state, but it is failing sometimes.
Temporarily disable until Dave can look into this, to unblock
CI/PR testing.

rdar://148754221

(cherry picked from commit 0b855f5)
adrian-prantl added a commit that referenced this pull request Apr 8, 2025
…-skip-swift-actors-test-62

[lldb] Temporarily disable actor state test (#10451)
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.

2 participants