Skip to content
This repository has been archived by the owner on Sep 26, 2019. It is now read-only.

CLI stack traces when debugging #960

Merged
merged 2 commits into from
Feb 25, 2019
Merged

CLI stack traces when debugging #960

merged 2 commits into from
Feb 25, 2019

Conversation

shemnon
Copy link
Contributor

@shemnon shemnon commented Feb 25, 2019

PR description

Sometimes we get very short and unhelpful exceptions from deep in
pantheon ('null' for example). This change makes it so that if
--logging is at DEBUG, TRACE, or ALL then the stack trace that caused
the exit will also be printed.

This is done by providing our own impl of defaultExceptionHandler that
is aware of the logLevel in PantheonCommand.

Sometimes we get very short and unhelpful exceptions from deep in
pantheon ('null' for example).  This change makes it so that if
--logging is at DEBUG, TRACE, or ALL then the stack trace that caused
the exit will also be printed.

This is done by providing our own impl of defaultExceptionHandler that
is aware of the logLevel in PantheonCommand.
@shemnon
Copy link
Contributor Author

shemnon commented Feb 25, 2019

not 1.0 critical.

Copy link
Contributor

@ekellstrand ekellstrand left a comment

Choose a reason for hiding this comment

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

LGTM

# Conflicts:
#	pantheon/src/main/java/tech/pegasys/pantheon/Pantheon.java
#	pantheon/src/main/java/tech/pegasys/pantheon/cli/PantheonCommand.java
#	pantheon/src/test/java/tech/pegasys/pantheon/cli/CommandTestAbstract.java
@shemnon shemnon merged commit 21eb853 into PegaSysEng:master Feb 25, 2019
@shemnon shemnon deleted the CLI branch February 25, 2019 23:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants