Skip to content
This repository has been archived by the owner on May 12, 2021. It is now read-only.

main: stacktrace/coredump on error #59

Merged
merged 1 commit into from
Mar 9, 2018

Conversation

jodh-intel
Copy link
Contributor

If the shim fails due to an internal error or if it receives a fatal
signal, write a stack trace to the log and exit.

If it was started with --debug also dump core.

Fixes #57.

Signed-off-by: James O. D. Hunt [email protected]

@jodh-intel
Copy link
Contributor Author

/cc @sboeuf, @bergwolf.

@sboeuf
Copy link

sboeuf commented Mar 7, 2018

@jodh-intel please fix the CI failures (Travis I mean).
LGTM

If the shim fails due to an internal error or if it receives
a fatal signal, write a stack trace to the log and exit.

If it was started with `--debug` also dump core.

Fixes kata-containers#57.

Signed-off-by: James O. D. Hunt <[email protected]>
@jodh-intel jodh-intel force-pushed the stacktrace-on-error branch from 4c98f49 to ab8eb53 Compare March 8, 2018 07:44
@jodh-intel
Copy link
Contributor Author

@sboeuf - Travis is happy now ;)

@sboeuf
Copy link

sboeuf commented Mar 8, 2018

@bergwolf PTAL

@bergwolf
Copy link
Member

bergwolf commented Mar 9, 2018

lgtm

Approved with PullApprove

@bergwolf
Copy link
Member

bergwolf commented Mar 9, 2018

Assuming Jenkins is still WiP... I'm merging it.

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.

4 participants