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

trap SIGINT to suppress noisy stack traces #113

Merged
merged 27 commits into from
Mar 10, 2023
Merged

trap SIGINT to suppress noisy stack traces #113

merged 27 commits into from
Mar 10, 2023

Conversation

salsferrazza
Copy link
Collaborator

Control-C'ing out of a transcoder process run in the foreground exits with a very noisy stack trace that inhibits immediate diagnostics of the last few messages. This CL traps SIGINT and exists more gracefully. It also provides the basic entry point for #13 and #111.

@salsferrazza salsferrazza requested a review from mservidio March 7, 2023 01:34
@salsferrazza salsferrazza requested a review from t8dogg as a code owner March 7, 2023 01:34
mservidio
mservidio previously approved these changes Mar 7, 2023
@salsferrazza salsferrazza marked this pull request as draft March 7, 2023 14:37
Copy link
Member

@t8dogg t8dogg left a comment

Choose a reason for hiding this comment

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

Useful and comprehensible, LGTM

mservidio and others added 2 commits March 7, 2023 12:50
@mservidio
Copy link
Collaborator

@salsferrazza needs some linting clean up, I touched up a couple of the issues

Copy link
Member

@t8dogg t8dogg left a comment

Choose a reason for hiding this comment

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

The linting LGTM @mservidio

@salsferrazza salsferrazza marked this pull request as ready for review March 10, 2023 00:37
transcoder/message/MessageParser.py Outdated Show resolved Hide resolved
transcoder/main.py Outdated Show resolved Hide resolved
@mservidio
Copy link
Collaborator

One other thought, the logic in main.py will only be a feature add for cli usage, but not library usage. Moving this into the MessageParser instance may be more beneficial.

@mservidio mservidio changed the title trap SIGINT to suppress noisy stack traces WIP: trap SIGINT to suppress noisy stack traces Mar 10, 2023
@mservidio mservidio changed the title WIP: trap SIGINT to suppress noisy stack traces trap SIGINT to suppress noisy stack traces Mar 10, 2023
@mservidio
Copy link
Collaborator

@salsferrazza re-factored a little, take a look...

@mservidio mservidio merged commit 33b4347 into main Mar 10, 2023
@mservidio mservidio deleted the trap_sigint branch March 10, 2023 15:53
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.

3 participants