-
Notifications
You must be signed in to change notification settings - Fork 151
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
[DOCUMENTATION] Update terms in documentation #448
Conversation
docs/user/terms.rst
Outdated
A service in Prometheus is a group of related Projects and are assigned to a | ||
specific shard | ||
A service in Prometheus is a group of related Projects and Rules and are the | ||
main top-level object. |
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.
Could we rephrase this sentence? It is a little bit difficult to understand.
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.
What would you recommend then?
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.
What would you recommend then?
I am not sure. Maybe this?
"A Service in Prometheus is a group of related Projects and Rules. Those three are the
main top-level objects."
Are there other top-level objects that are not considered "main"?
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.
I realized I need to fix this anyways, because service
is a Promgen concept and not Prometheus.
A service in Promgen is a group of related Projects and Rules.
Service is typically the top-level object in Promgen.
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.
I realized I didn't approve this PR before. The comments were optional, by the way.
Some of these terms have not been updated in a while and need to be updated to show the current relationship between them.
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.
Looks good to me 👍🏻
Some of these terms have not been updated in a while and need to be updated to show the current relationship between them.