-
Notifications
You must be signed in to change notification settings - Fork 476
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
Document bundle manager telemetry #3870
Document bundle manager telemetry #3870
Conversation
Signed-off-by: Andrew Harding <azdagron@gmail.com>
doc/telemetry.md
Outdated
| Type | Keys | Labels | Description | | ||
|--------------|------------------------------------------------|-------------------|---------------------------------------------------------------------------------------| | ||
| Call Counter | `rpc`, `<service>`, `<method>` | | Call counters over the [SPIRE Server RPCs](https://github.com/spiffe/spire-api-sdk). | | ||
| Counter | `bundle_manager`, `update`, `federated_bundle` | `trust_domain_id` | The bundle endpoint manager updated a bundle | |
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.
With the table re-formatting, this diff is all but worthless :) This is the line that was introduced.
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.
updated a bundle
is true, but may we say that it updated a federated bundle
?
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.
Good call. Fixed.
Signed-off-by: Andrew Harding <azdagron@gmail.com>
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!!!
* Document bundle manager telemetry Signed-off-by: Andrew Harding <azdagron@gmail.com>
No description provided.