-
Notifications
You must be signed in to change notification settings - Fork 8k
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
[Files] Added usage counters for delete, unshare and download #140091
[Files] Added usage counters for delete, unshare and download #140091
Conversation
419b44e
to
07a807f
Compare
393bfd1
to
abc5031
Compare
eb3aed2
to
6ae6c5c
Compare
Pinging @elastic/kibana-app-services (Team:AppServicesUx) |
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.
Hope we can have error codes/types as context in the error code counter names.
export function getCounters(prefix: string) { | ||
return { | ||
DELETE: `${prefix}_delete`, | ||
DELETE_ERROR: `${prefix}_delete_error`, |
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.
Would it be possible to get more context to the error counter names, such as an error code? I think we could get more value by using event names for errors in the form of: eventType:error:errorCode
.
Example: csv_download:error:403
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.
That's a really good idea @tsullivan!
I've mapped out some known errors that I think would be of interest. The download case is a little bit hard because the error space is not very well mapped out. Let me know what you think!
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.
LGTM
💚 Build Succeeded
Metrics [docs]
History
To update your PR or re-run it, just comment with: |
* main: [SearchBar] Allow show refresh button as an icon (elastic#141088) [Files] Added usage counters for delete, unshare and download (elastic#140091) Fix removing a single field formatter (elastic#141078) [APM] Move service metric config to kibana advanced settings (elastic#141147) Synthetics project update improvements (elastic#140990)
Summary
To better track usage, this PR introduces use of usage counters of a select set of actions to enrich telemetry.
Continuation from: #140079