[PR #750] feat: add OpenObserve template #4229

New Issue

GiteaMirror · 2026-04-30T06:49:43-05:00

GiteaMirror commented

2026-04-30 06:49:43 -05:00

📋 Pull Request Information

Original PR: https://github.com/Dokploy/templates/pull/750
Author: @andrew-tinekov
Created: 3/23/2026
Status: 🔄 Open

Base: canary ← Head: feat/add-openobserve-template

📝 Commits (2)

e19e979 feat: add openobserve template
af9f579 fix(openobserve): fix code violations

📊 Changes

4 files changed (+70 additions, -0 deletions)

View changed files

➕ blueprints/openobserve/docker-compose.yml (+18 -0)
➕ blueprints/openobserve/openobserve.svg (+14 -0)
➕ blueprints/openobserve/template.toml (+17 -0)
📝 meta.json (+21 -0)

📄 Description

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:

I have read the suggestions in the README.md file https://github.com/Dokploy/templates?tab=readme-ov-file#general-requirements-when-creating-a-template
I have tested the template in my instance, so the maintainers don't spend time trying to figure out what's wrong.
I have added tests that demonstrate that my correction works or that my new feature works.

Screenshots or Videos

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 ports→expose 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)

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/Dokploy/templates/pull/750 **Author:** [@andrew-tinekov](https://github.com/andrew-tinekov) **Created:** 3/23/2026 **Status:** 🔄 Open **Base:** `canary` ← **Head:** `feat/add-openobserve-template` --- ### 📝 Commits (2) - [`e19e979`](https://github.com/Dokploy/templates/commit/e19e97987eb0a3b4c176b536552adca173297c07) feat: add openobserve template - [`af9f579`](https://github.com/Dokploy/templates/commit/af9f579995d3efcc658eda4ee2eed686893321db) fix(openobserve): fix code violations ### 📊 Changes **4 files changed** (+70 additions, -0 deletions) <details> <summary>View changed files</summary> ➕ `blueprints/openobserve/docker-compose.yml` (+18 -0) ➕ `blueprints/openobserve/openobserve.svg` (+14 -0) ➕ `blueprints/openobserve/template.toml` (+17 -0) 📝 `meta.json` (+21 -0) </details> ### 📄 Description ## 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: - [x] I have read the suggestions in the README.md file https://github.com/Dokploy/templates?tab=readme-ov-file#general-requirements-when-creating-a-template - [x] I have tested the template in my instance, so the maintainers don't spend time trying to figure out what's wrong. - [x] I have added tests that demonstrate that my correction works or that my new feature works. ## Screenshots or Videos <img width="1512" height="982" alt="openobserve1" src="https://github.com/user-attachments/assets/fb477170-274d-4560-bbdd-cb24225d1c01" /> <img width="1512" height="982" alt="openobserve2" src="https://github.com/user-attachments/assets/9d62c9c1-ba55-4100-948f-89ba288f3f23" />  <h3>Greptile Summary</h3> 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`. <h3>Confidence Score: 2/5</h3> - Not safe to merge until the `ports`→`expose` 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. <sub>Reviews (1): Last reviewed commit: ["feat: add openobserve template"](https://github.com/dokploy/templates/commit/e19e97987eb0a3b4c176b536552adca173297c07) | [Re-trigger Greptile](https://app.greptile.com/api/retrigger?id=26085200)</sub> > Greptile also left **4 inline comments** on this PR. <sub>(2/5) Greptile learns from your feedback when you react with thumbs up/down!</sub> **Context used:** - Context used - AGENTS.md ([source](https://app.greptile.com/review/custom-context?memory=09330bde-2058-497c-9c64-ceae637fb5b2))  --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

GiteaMirror added the pull-request label 2026-04-30 06:49:43 -05:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: github-starred/templates#4229