Skip to content

observability#4

Open
gsanseverino wants to merge 1 commit into
mainfrom
observability
Open

observability#4
gsanseverino wants to merge 1 commit into
mainfrom
observability

Conversation

@gsanseverino

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions

github-actions Bot commented May 20, 2026

Copy link
Copy Markdown
Contributor

QualOps Code Quality Analysis

Status: ⚠️ FAILED - Critical or high severity issues found

Summary

  • Total Issues: 5
  • Critical: 0 🔴
  • High: 1 🟠
  • Medium: 2 🟡
  • Low: 2 🟢
  • Files Analyzed: 12

🟠 High Issues (1)

  • package.json:13 - bug
    pnpm version in packageManager field (10.28.2) conflicts with the version activated in the Dockerfile (10.30.2), causing corepack to use the wrong version at build time

🟡 Medium Issues (2)

  • docker-compose.yml:21 - bug
    The agent service's depends_on: tempo has no healthcheck condition, so the agent can start before Tempo's OTLP receiver is ready to accept connections

  • docker-compose.yml:36 - maintainability
    Grafana image is unpinned (grafana/grafana:latest), making builds non-reproducible while every other image in the stack is version-pinned

📊 Full Report

View detailed report


Powered by QualOps

@gsanseverino
gsanseverino force-pushed the observability branch 2 times, most recently from 27f81ac to a5e4ea3 Compare May 20, 2026 16:26
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