dapr: Use github.com/pkg/errors instead of fmt.errorf

In what area(s)?

/area runtime

Describe the proposal

we have used fmt.errorf to formatted error in the entire code bases, but this loses many information.

The other projects, such as Kubernetes, switched to https://github.com/pkg/errors to wrap errors in contextual information and still preserve the underlying error and originating stack trace.

instead of using fmt.errorf, we can use errors.Wrap and errors.Errorf in https://github.com/pkg/errors.

Ref: https://github.com/kubernetes-sigs/kubefed/issues/381

About this issue

  • Original URL
  • State: closed
  • Created 4 years ago
  • Comments: 25 (25 by maintainers)

Commits related to this issue

Most upvoted comments

I will take up actors and api next

I think, placement and sidecar injector still needs modification so the ticket should not be closed

@ShaileshSurya I can take up Sentry and placement and you can take up Operator and sidecar injector