-
Notifications
You must be signed in to change notification settings - Fork 228
docs(SaaS): Add Cluster Metrics endpoint documentation for Camunda 8 SaaS monitoring #7688
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
Conversation
|
👋 🤖 🤔 Hello, @zaycodes! Did you make your changes in all the right places? These files were changed only in docs/. You might want to duplicate these changes in versioned_docs/version-8.8/.
You may have done this intentionally, but we wanted to point it out in case you didn't. You can read more about the versioning within our docs in our documentation guidelines. |
...s/saas/monitoring/cluster-metrics-endpoint/configure-monitoring-systems-to-scrape-metrics.md
Outdated
Show resolved
Hide resolved
...s/saas/monitoring/cluster-metrics-endpoint/configure-monitoring-systems-to-scrape-metrics.md
Outdated
Show resolved
Hide resolved
...s/saas/monitoring/cluster-metrics-endpoint/configure-monitoring-systems-to-scrape-metrics.md
Show resolved
Hide resolved
...s/saas/monitoring/cluster-metrics-endpoint/configure-monitoring-systems-to-scrape-metrics.md
Show resolved
Hide resolved
...s/saas/monitoring/cluster-metrics-endpoint/configure-monitoring-systems-to-scrape-metrics.md
Outdated
Show resolved
Hide resolved
...s/saas/monitoring/cluster-metrics-endpoint/configure-monitoring-systems-to-scrape-metrics.md
Outdated
Show resolved
Hide resolved
docs/components/saas/monitoring/cluster-metrics-endpoint/index.md
Outdated
Show resolved
Hide resolved
docs/components/saas/monitoring/cluster-metrics-endpoint/index.md
Outdated
Show resolved
Hide resolved
docs/components/saas/monitoring/cluster-metrics-endpoint/index.md
Outdated
Show resolved
Hide resolved
docs/components/saas/monitoring/cluster-metrics-endpoint/index.md
Outdated
Show resolved
Hide resolved
docs/components/saas/monitoring/cluster-metrics-endpoint/index.md
Outdated
Show resolved
Hide resolved
docs/components/saas/monitoring/cluster-metrics-endpoint/set-up-cluster-metrics-endpoint.md
Outdated
Show resolved
Hide resolved
docs/components/saas/monitoring/cluster-metrics-endpoint/set-up-cluster-metrics-endpoint.md
Outdated
Show resolved
Hide resolved
docs/components/saas/monitoring/cluster-metrics-endpoint/set-up-cluster-metrics-endpoint.md
Show resolved
Hide resolved
docs/components/saas/monitoring/cluster-metrics-endpoint/index.md
Outdated
Show resolved
Hide resolved
docs/components/saas/monitoring/cluster-metrics-endpoint/index.md
Outdated
Show resolved
Hide resolved
docs/components/saas/monitoring/cluster-metrics-endpoint/set-up-cluster-metrics-endpoint.md
Outdated
Show resolved
Hide resolved
docs/components/saas/monitoring/cluster-metrics-endpoint/set-up-cluster-metrics-endpoint.md
Outdated
Show resolved
Hide resolved
| <!-- TODO: Credential behavior \ | ||
| Confirm if there is a limit on the number of credentials per cluster --> |
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.
@ColRad Can you confirm 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.
No there is no limit on credentials. The user can create as many as they like
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.
mesellings
left a comment
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.
@zaycodes nice work on this PR - I've committed a few changes, but otherwise looks good to go! 🚀
|
@mesellings help re-approve after back-porting the changes. Thanks! |
mesellings
left a comment
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.
Re-approved @zaycodes - great work 🚀
|
🌱 Have we thought about adding a very simple cc @sourcehawk, @zaycodes |
The curl command would be: |
|
@zaycodes @sourcehawk confirmed we can backport this to the alpha 3 release notes. Do you prefer to include it in this PR, or should I open a separate one? Let me know, thanks! |
I've added it to this PR, thanks! |
afgambin
left a comment
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! Thanks for adding it to the RN :)
sourcehawk
left a comment
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.
Beautiful! 🎉
Description
This PR introduces a new Monitoring section under Camunda 8 SaaS and adds documentation for the Cluster Metrics endpoint.
Epic - https://github.com/camunda/product-hub/issues/2229
The documentation covers:
When should this change go live?
bugorsupportlabel)available & undocumentedlabel)holdlabel)low priolabel)PR Checklist
{type}(scope): {description}commit message(s)My changes are for an upcoming minor release and are in the
/docsdirectory (version 8.9).My changes are for an already released minor and are in a
/versioned_docsdirectory.I included my new page in the sidebar file(s).
@camunda/tech-writersunless working with an embedded writer.