-
Notifications
You must be signed in to change notification settings - Fork 99
Describe Satellite usage metrics collection #3846
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
base: master
Are you sure you want to change the base?
Conversation
Hi @adamruzicka, can you please review when you're available? |
The PR preview for 62d726d is available at theforeman-foreman-documentation-preview-pr-3846.surge.sh The following output files are affected by this PR: |
Hi @Lennonka, what do you think about the placement? Is Planning guide a good place to add this? |
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.
Two optional suggestions, otherwise looks good
What changes are you introducing?
Adding explanation on Satellite's usage metrics collection.
Why are you introducing these changes? (Explanation, links to references, issues, etc.)
theforeman/foreman_maintain#851
Anything else to add? (Considerations, potential downsides, alternative solutions you have explored, etc.)
Because this is specific to Satellite, I'm not using the usual attributes, such as {Team} or {foreman-maintain}, and I'm instead hard-coding them as Red{nbsp}Hat and satellite-maintain.
Checklists
Please cherry-pick my commits into: