VAULT-39930 Add certificate counters to license usage reporting #1147
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add mention of PKI certificate counts to the automated and manual license usage reporting docs.
Contributor
Vercel Previews Deployed
|
Contributor
Broken Link CheckerNo broken links found! 🎉 |
rculpepper
approved these changes
Oct 21, 2025
schavis
requested changes
Oct 21, 2025
content/vault/v1.21.x (rc)/content/docs/license/utilization/auto-reporting.mdx
Outdated
Show resolved
Hide resolved
content/vault/v1.21.x (rc)/content/docs/license/utilization/auto-reporting.mdx
Outdated
Show resolved
Hide resolved
content/vault/v1.21.x (rc)/content/docs/license/utilization/auto-reporting.mdx
Outdated
Show resolved
Hide resolved
content/vault/v1.21.x (rc)/content/docs/license/utilization/auto-reporting.mdx
Outdated
Show resolved
Hide resolved
content/vault/v1.21.x (rc)/content/docs/license/utilization/auto-reporting.mdx
Outdated
Show resolved
Hide resolved
| snapshot | ||
| - `snapshot_id` - A unique identifier for this particular snapshot | ||
| - `process_id` - An identifier for the system that produced this snapshot | ||
| - `timestamp` - The date and time for this snapshot |
Contributor
There was a problem hiding this comment.
Suggested change
| - `timestamp` - The date and time for this snapshot | |
| - `timestamp` - The date and time for the generated snapshot. |
| - `metrics` - A map of representations of snapshot metrics contained within | ||
| this snapshot | ||
| - `key` - The key name associated with this metric | ||
| - `kind` - The kind of metric (feature, counter, sum, or mean) |
Contributor
There was a problem hiding this comment.
Suggested change
| - `kind` - The kind of metric (feature, counter, sum, or mean) | |
| - `kind` - The kind of metric (feature, counter, sum, or mean). |
Comment on lines
+85
to
+88
| - `-message` `(string: "")` - Provide context about the conditions under | ||
| which the report was generated and submitted. This message is not included | ||
| in the license utilization bundle but will be included in the vault server | ||
| logs. (optional) |
Contributor
There was a problem hiding this comment.
Suggested change
| - `-message` `(string: "")` - Provide context about the conditions under | |
| which the report was generated and submitted. This message is not included | |
| in the license utilization bundle but will be included in the vault server | |
| logs. (optional) | |
| - `-message` `(string: "")` - Provides context about the conditions under | |
| which Vault generated and submitted the report. Vault writes the | |
| context message to the server logs, but does not include the message | |
| in the license utilization bundle. |
Style correction: write in active voice
Comment on lines
+91
to
+92
| historical snapshots. If no snapshots were persisted in the last 24 hrs, it | ||
| takes a snapshot and exports it to a bundle. (optional) |
Contributor
There was a problem hiding this comment.
Suggested change
| historical snapshots. If no snapshots were persisted in the last 24 hrs, it | |
| takes a snapshot and exports it to a bundle. (optional) | |
| historical snapshots. Vault takes a snapshot and exports the snapshot | |
| to a bundle if it has not persisted a snapshot in the last 24 hrs. |
Style correction: write in active voice
content/vault/v1.21.x (rc)/content/docs/license/utilization/manual-reporting.mdx
Outdated
Show resolved
Hide resolved
Co-authored-by: Sarah Chavis <62406755+schavis@users.noreply.github.com>
Contributor
Author
|
@schavis Committed suggestions that were directly related to the content I edited, or where I felt comfortable changing without further research. Other changes, like "this snapshot", I won't make right now, as I'd need to do further research to confirm their accuracy (e.g., IIUC, there can be multiple snapshots within a single payload). These are marked as changes within this PR largely because of document-wide formatting but I didn't actually edit the content. |
Contributor
@gsantos-hc Makes sense. "Ignore whitespace" was unchecked for some reason so it made it look like you had edited the whole page 🤦🏽♀️ |
schavis
approved these changes
Oct 22, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🎫 VAULT-39930
Add mention of PKI certificate counts to the automated and manual license usage reporting docs.