Skip to content

Comments

feat: Check otelcol logs in COS tests#170

Draft
MichaelThamm wants to merge 4 commits intomainfrom
feat/check-otelcol-logs
Draft

feat: Check otelcol logs in COS tests#170
MichaelThamm wants to merge 4 commits intomainfrom
feat/check-otelcol-logs

Conversation

@MichaelThamm
Copy link
Contributor

@MichaelThamm MichaelThamm commented Dec 18, 2025

Issue

Our tests do not have great coverage, e.g. only checking that catalogue apps are reachable. We need more coverage on internal workings of COS.

Solution

Since otelcol is the aggregator (all data flows through it) within the COS model, it can indicate correct working operations. We can inspect this by checking the pebble logs. Check for:

  • error
  • warn

Checklist

  • I have added or updated relevant documentation.
  • PR title makes an appropriate release note and follows conventional commits syntax.
  • Merge target is the correct branch, and relevant tandem backport PRs opened.

Context

Testing Instructions

Upgrade Notes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant