llvm / llvm/llvm-project

[lldb-dap] Remove (or trim) the DAP log on test failures

Open
#188,705 11 comments 0 reactions 0 assignees View on GitHub
lldb-dap
Dominant language
LLVM
Stars
40.5k
Forks
18.7k
PR merge metrics
PR metrics pending

Description

Whenever a DAP test fails it seems to dump the entire protocol communication log to the output. Can we avoid doing so as it creates noise when looking at test logs?

Question to `lldb-dap` maintainers: Is there some portion of the current log output that's critical to see on CI? How often is seeing the entire communication history useful when debugging test failures?

CC @da-viper @ashgti @walter-erquinigo @JDevlieghere

Contributor guide

Open the contributing guide

Research direction

Start by locating the lldb-dap tests and the code that prints protocol communication when a test fails. Review the existing failure output and determine which diagnostic portion must remain on CI; done means failures no longer dump the entire communication history while preserving useful debugging context.

Written by the indexing model from the issue text.

Assessment

Domain
devtools, testing-qa
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.