Skip to content

feat: add OpenObserve template#750

Open
andrew-tinekov wants to merge 2 commits intoDokploy:canaryfrom
andrew-tinekov:feat/add-openobserve-template
Open

feat: add OpenObserve template#750
andrew-tinekov wants to merge 2 commits intoDokploy:canaryfrom
andrew-tinekov:feat/add-openobserve-template

Conversation

@andrew-tinekov
Copy link
Copy Markdown

@andrew-tinekov andrew-tinekov commented Mar 23, 2026

What is this PR about?

New PR of OpenObserve – a cloud-native observability platform that unifies logs, metrics, and traces into one solution.

Checklist

Before submitting this PR, please make sure that:

Screenshots or Videos

openobserve1 openobserve2

Greptile Summary

This PR adds a new OpenObserve template — a cloud-native observability platform for logs, metrics, and traces. The overall structure is sound, but the docker-compose.yml has several violations of the project's documented conventions (AGENTS.md) that need to be addressed before merging.

Key issues:

  • Missing version: "3.8" — AGENTS.md requires this as the first line of every docker-compose.yml.
  • ports used instead of expose — AGENTS.md explicitly prohibits ports in templates; expose must be used instead. Using ports maps container ports to the host, bypassing Dokploy's network isolation.
  • Unpinned Docker image (latest tag) — AGENTS.md requires pinning images to specific versions to prevent supply chain attacks and ensure deployment reproducibility. The "version": "latest" in meta.json should also be updated to match the pinned tag once chosen.
  • Minor: ZO_HTTP_PORT and ZO_GRPC_PORT are threaded through as env vars unnecessarily; since they are static values, they can be hardcoded directly in expose.

Confidence Score: 2/5

  • Not safe to merge until the portsexpose fix, missing version: "3.8", and image pinning are addressed.
  • Three concrete convention violations from AGENTS.md are present in the docker-compose.yml: missing version declaration, prohibited ports directive, and unpinned image tag. These are straightforward fixes but required by the project's own guidelines before the template can be merged.
  • blueprints/openobserve/docker-compose.yml requires all three fixes; meta.json needs the version updated once the image is pinned.

Reviews (1): Last reviewed commit: "feat: add openobserve template" | Re-trigger Greptile

Greptile also left 4 inline comments on this PR.

(2/5) Greptile learns from your feedback when you react with thumbs up/down!

Context used:

  • Context used - AGENTS.md (source)

@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Mar 23, 2026
@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 23, 2026

built with Refined Cloudflare Pages Action

⚡ Cloudflare Pages Deployment

Name Status Preview Last Commit
templates ✅ Ready (View Log) Visit Preview af9f579

@andrew-tinekov
Copy link
Copy Markdown
Author

andrew-tinekov commented Mar 23, 2026

About Validate and Process Meta.json – not related to my changes:

[2026-03-23T19:35:23.228Z] 🔧 Starting meta.json processing...
[2026-03-23T19:35:23.230Z] 🔧 Found 389 total entries
[2026-03-23T19:35:23.231Z] ⚠️ Duplicate ID found: "strapi" (Strapi)
[2026-03-23T19:35:23.234Z] ✅ Processing completed in 6ms
[2026-03-23T19:35:23.234Z] 🔧 Statistics:
[2026-03-23T19:35:23.234Z] 🔧   • Original entries: 389
[2026-03-23T19:35:23.234Z] 🔧   • Duplicates removed: 1
[2026-03-23T19:35:23.234Z] 🔧   • Final entries: 388
[2026-03-23T19:35:23.234Z] 🔧   • Schema violations: 0
[2026-03-23T19:35:23.234Z] ⚠️ Removed duplicates:
[2026-03-23T19:35:23.234Z] ⚠️   • "strapi" (Strapi)

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

Labels

new-template size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant