Skip to content

Doc: Account for both COS Lite and COS - #1593

Merged
roosterfish merged 1 commit into
canonical:mainfrom
roosterfish:fix_cos_integrations
Oct 13, 2026
Merged

roosterfish merged 1 commit into
canonical:mainfrom
roosterfish:fix_cos_integrations

Conversation

@roosterfish

Copy link
Copy Markdown
Contributor

COS Lite uses Prometheus whereas COS uses Mimir as metrics backend.

@roosterfish
roosterfish requested a lite review from Copilot October 13, 2026 07:52
@github-actions github-actions Bot added the Documentation Documentation needs updating label Oct 13, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

The collector relation still names prometheus, so the documented COS path fails when using mimir.

2 open findings
What changed in this PR

This PR updates the MicroCloud charm metrics guide to cover both COS Lite and COS metrics backends.

Changes:

  • Replaces the Prometheus application name in the offer and consume commands with a backend placeholder.
  • Adds a note explaining how to choose the backend application.
File Description
doc/​how-to/​charm_metrics.md Updates cross-model metrics instructions for COS Lite and COS.

🧠 Review effort: Lite


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread doc/how-to/charm_metrics.md
Comment thread doc/how-to/charm_metrics.md
COS Lite uses Prometheus whereas COS uses Mimir as metrics backend.

Signed-off-by: Julian Pelizäus <julian.pelizaeus@canonical.com>
@roosterfish
roosterfish requested a review from MusicDin October 13, 2026 09:03
@roosterfish
roosterfish merged commit 48ec732 into canonical:main Oct 13, 2026
19 checks passed
@roosterfish
roosterfish deleted the fix_cos_integrations branch October 13, 2026 09:57
roosterfish added a commit that referenced this pull request Oct 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Documentation Documentation needs updating

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants