Skip to content

driver/rawnetworkinterfacedriver: improve debugging experience - #1952

Open
Bastian-Krause wants to merge 4 commits into
labgrid-project:masterfrom
Bastian-Krause:bst/raw-net-debugging
Open

driver/rawnetworkinterfacedriver: improve debugging experience#1952
Bastian-Krause wants to merge 4 commits into
labgrid-project:masterfrom
Bastian-Krause:bst/raw-net-debugging

Conversation

@Bastian-Krause

Copy link
Copy Markdown
Member

Description
Small collection of patches allowing to see what commands are actually executed (via processwrapper and via the labgrid-raw-interface sudo helper).

Checklist

  • PR has been tested

/cc @JoshuaWatt

…ocess directly

Where possible, use labgrid's processwrapper. It logs what command is
executed and what it emitted. This helps when debugging.

Signed-off-by: Bastian Krause <bst@pengutronix.de>
This helps debugging.

Signed-off-by: Bastian Krause <bst@pengutronix.de>
This allows querying what command would have been executed.

Signed-off-by: Bastian Krause <bst@pengutronix.de>
…-raw-interface helper will execute

The driver and helper indirection make it really hard to track what
actual commands are executed where. Use the labgrid-raw-interface
helper's --dry-run argument to query and log what will be executed
where.

Signed-off-by: Bastian Krause <bst@pengutronix.de>
@Bastian-Krause
Bastian-Krause force-pushed the bst/raw-net-debugging branch from 75dbcd0 to 2c7acdb Compare August 7, 2026 09:24
@codecov

codecov Bot commented Aug 7, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 3.12500% with 31 lines in your changes missing coverage. Please review.
✅ Project coverage is 60.9%. Comparing base (424a0b7) to head (2c7acdb).
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
labgrid/driver/rawnetworkinterfacedriver.py 3.1% 31 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff            @@
##           master   #1952     +/-   ##
========================================
- Coverage    61.0%   60.9%   -0.1%     
========================================
  Files         182     182             
  Lines       14898   14919     +21     
========================================
  Hits         9096    9096             
- Misses       5802    5823     +21     
Flag Coverage Δ
3.10 60.9% <3.1%> (-0.2%) ⬇️
3.11 60.9% <3.1%> (-0.2%) ⬇️
3.12 60.9% <3.1%> (-0.1%) ⬇️
3.13 60.9% <3.1%> (-0.1%) ⬇️
3.14 60.9% <3.1%> (-0.1%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant