Skip to content

remote/client: suppress gRPC error message on shutdown#1605

Merged
Bastian-Krause merged 1 commit intolabgrid-project:masterfrom
jluebbe:suppress-grpc-error-message
Feb 12, 2025
Merged

remote/client: suppress gRPC error message on shutdown#1605
Bastian-Krause merged 1 commit intolabgrid-project:masterfrom
jluebbe:suppress-grpc-error-message

Conversation

@jluebbe
Copy link
Member

@jluebbe jluebbe commented Feb 12, 2025

This is a workround for the gRPC issue grpc/grpc#38679.

On Python 3.12, an empty exception message is printed from gRPC during shutdown, although nothing seems to go wrong. As this is confusing for users, suppress the message by closing stderr. When in debug mode, we keep it open.

@jluebbe jluebbe added the grpc label Feb 12, 2025
@jluebbe jluebbe requested a review from Emantor February 12, 2025 11:17
Emantor
Emantor previously approved these changes Feb 12, 2025
This is a workround for the gRPC issue
grpc/grpc#38679.

Since Python 3.12, an empty exception message is printed from gRPC during
shutdown, although nothing seems to go wrong. As this is confusing for
users, suppress the message by closing stderr. When in debug mode, we
keep it open.

Signed-off-by: Jan Luebbe <jlu@pengutronix.de>
@codecov
Copy link

codecov bot commented Feb 12, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 55.8%. Comparing base (9636068) to head (2dc889c).
Report is 52 commits behind head on master.

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@          Coverage Diff           @@
##           master   #1605   +/-   ##
======================================
  Coverage    55.8%   55.8%           
======================================
  Files         170     170           
  Lines       13376   13378    +2     
======================================
+ Hits         7468    7470    +2     
  Misses       5908    5908           
Flag Coverage Δ
3.10 56.1% <100.0%> (+<0.1%) ⬆️
3.11 56.1% <100.0%> (+<0.1%) ⬆️
3.12 56.1% <100.0%> (+<0.1%) ⬆️
3.13 56.1% <100.0%> (+<0.1%) ⬆️
3.9 56.1% <100.0%> (+<0.1%) ⬆️

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

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

@Bastian-Krause Bastian-Krause merged commit e8aadae into labgrid-project:master Feb 12, 2025
11 checks passed
@jluebbe jluebbe deleted the suppress-grpc-error-message branch February 12, 2025 11:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants