Skip to content
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

Disable the test category lldb-server for lldb-remote-* builders #353

Merged

Conversation

slydiman
Copy link
Contributor

@slydiman slydiman commented Jan 7, 2025

API tests llvm-project/lldb/test/API/tools/lldb-server/* with the category lldb-server run lldb-server locally on the host with a random port. Sometimes we got errors like
https://lab.llvm.org/buildbot/#/builders/195/builds/3145
https://lab.llvm.org/buildbot/#/builders/197/builds/312
The patch llvm/llvm-project#118222 improved the situation but did not fix it completely.

API tests llvm-project/lldb/test/API/tools/lldb-server/* with the category `lldb-server` run lldb-server locally on the host with a random port.
Sometimes we got errors like
https://lab.llvm.org/buildbot/#/builders/195/builds/3145
https://lab.llvm.org/buildbot/#/builders/197/builds/312
The patch llvm/llvm-project#118222 improved the situation but did not fix it completely.
Copy link
Contributor

@vvereschaka vvereschaka left a comment

Choose a reason for hiding this comment

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

LGTM

@andreil99
Copy link
Contributor

LGTM.

Copy link
Contributor

@gkistanova gkistanova left a comment

Choose a reason for hiding this comment

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

LGTM.

@slydiman slydiman merged commit fcd4bb2 into llvm:main Jan 13, 2025
2 checks passed
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.

4 participants