mirrors/velero
1
0
Fork 0
mirror of https://github.com/vmware-tanzu/velero.git synced 2026-01-06 21:36:30 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Merge pull request #5110 from reasonerjt/plugin-panic-trace

Browse Source 
Dump stack trace when the plugin server handles panic
This commit is contained in:
Xun Jiang/Bruce Jiang
2022-07-11 22:47:17 +08:00
committed by GitHub
parent 77c1549d4d 9102f53131
commit eaf97e7510
2 changed files with 5 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
changelogs/unreleased/5110-reasonerjt Normal file
View File

@@ -0,0 +1 @@
Dump stack trace when the plugin server handles panic

5
pkg/plugin/framework/handle_panic.go
View File

@@ -17,6 +17,8 @@ limitations under the License.
package framework
import (
"runtime/debug"
"github.com/pkg/errors"
"google.golang.org/grpc/codes"
)
@@ -38,7 +40,8 @@ func handlePanic(p interface{}) error {
if _, ok := panicErr.(stackTracer); ok {
err = panicErr
} else {
err = errors.Wrap(panicErr, "plugin panicked")
errWithStacktrace := errors.Errorf("%v, stack trace: %s", panicErr, debug.Stack())
err = errors.Wrap(errWithStacktrace, "plugin panicked")
}
}