-
Notifications
You must be signed in to change notification settings - Fork 730
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
Add exhaustive list of error codes and description of metric flag #4422
Conversation
Signed-off-by: Jake Engelberg <[email protected]>
daprdocs/content/en/operations/observability/metrics/metrics-overview.md
Outdated
Show resolved
Hide resolved
Stale PR, paging all reviewers |
Stale PR, paging all reviewers |
Stale PR, paging all reviewers |
Signed-off-by: Jake Engelberg <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
quick review - thank you!
daprdocs/content/en/operations/observability/metrics/metrics-overview.md
Outdated
Show resolved
Hide resolved
Signed-off-by: Jake Engelberg <[email protected]>
ab17803
to
a44a96a
Compare
Stale PR, paging all reviewers |
@jake-engelberg - I realize that we need to make error codes concept much more visible to developers by adding a topic under https://docs.dapr.io/developing-applications/building-blocks/ called "API Error codes" describing what they are and also how you can get metrics for them (This can point to the error codes list and the metrics configuration) Also in this article we should mention that each SDK can handle and return the error code in the language specific way. For example this is Go https://docs.dapr.io/developing-applications/sdks/go/go-client/#error-handling but we also need to discuss with each of the SDK maintainers how to handle these. I suggest raising an issue into each of SDK repos for this to track this (if my understanding is correct) @hhunter-ms can help as needed. Hannah - This reference article https://docs.dapr.io/reference/api/error_codes/ should be called "API Error codes" in the index file. I did not realize this was buried under Dapr API reference. |
@@ -7,43 +7,150 @@ weight: 1400 | |||
--- | |||
|
|||
For http calls made to Dapr runtime, when an error is encountered, an error json is returned in http response body. The json contains an error code and an descriptive error message, e.g. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jake-engelberg this is HTTP and gRPC?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, it is both and we'll update the wording soon to reflect the auto generated chart with both codes which at the moment sometimes differ:
spec: | ||
metrics: | ||
enabled: true | ||
recordErrorCodes: true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You also need to;
- update this table in configuration setting https://docs.dapr.io/operations/configuration/configuration-overview/#metrics And in that table point to this article for the details.
- Update the configuration schema to add this under metrics section here https://docs.dapr.io/reference/resource-specs/configuration-schema/
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@hhunter-ms - Will do this. Do Jake you can ignore this comment.
Co-authored-by: Mark Fussell <[email protected]> Signed-off-by: Jake Engelberg <[email protected]>
@holopin-bot @jake-engelberg Thanks Jake! |
Congratulations @jake-engelberg, the maintainer of this repository has issued you a badge! Here it is: https://holopin.io/claim/cm5qxvv6811300cmj4flegr8q This badge can only be claimed by you, so make sure that your GitHub account is linked to your Holopin account. You can manage those preferences here: https://holopin.io/account. |
Thank you for helping make the Dapr documentation better!
Please follow this checklist before submitting:
In addition, please fill out the following to help reviewers understand this pull request:
Description
Added the exhaustive set of error codes dapr emits as well as how to enable metric recording for these error codes based on the associated PR below which implements this.
Issue reference
dapr/dapr#8256