-
Notifications
You must be signed in to change notification settings - Fork 0
oobservability #168
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Closed
Closed
oobservability #168
Changes from all commits
Commits
Show all changes
22 commits
Select commit
Hold shift + click to select a range
efdf00d
docs(plans): track observability strategy plan, revised at 673bdfa0
Bnjoroge1 2e91532
docs(plans): add observability strategy HTML with architecture diagra…
Bnjoroge1 a30f691
fix(server): scrub capability tokens from INFO/WARN logs
Bnjoroge1 19ca6fa
docs(internal): track observability signal/security contract
Bnjoroge1 da68f73
feat(observability): add observability crate and unify process init
Bnjoroge1 cdab10d
feat(server,cli): add liveness, readiness, aggregate status and metri…
Bnjoroge1 faf3283
chore: remove plan stage markers from code comments
Bnjoroge1 6e4a82c
feat(server): instrument HTTP and store with bounded metrics
Bnjoroge1 b050d95
feat(server): record job terminal transitions and queue wait
Bnjoroge1 c3877c6
feat(server): record queue wait and broker poll on successful acquire
Bnjoroge1 8e80f2a
feat(server): record session create/delete transitions
Bnjoroge1 797dcab
feat(observability): add concurrency decision counter
Bnjoroge1 e065899
feat(vm): add host sampler stub and VM fleet registry
Bnjoroge1 1f7f90a
feat(pool): wire pool_status preparing flag alongside legacy signal
Bnjoroge1 da5ae18
feat(contrib): add pinned single-node OpenObserve reference profile
Bnjoroge1 d52cc9a
feat(server): report per-component storage bytes in status
Bnjoroge1 7703429
feat(server): report github configured state in status
Bnjoroge1 5415dcb
fix(server): always instrument store, even without explicit observabi…
Bnjoroge1 84a2551
feat(observability): add bounded OTLP/HTTP JSON exporter and wire lif…
Bnjoroge1 9085a12
fix(server): classify termination reasons and report the host binary …
Bnjoroge1 5808ba4
feat(observability): export metrics and traces over OTLP
Bnjoroge1 4faa700
fix(server): keep the reason prose on the log and classify external-h…
Bnjoroge1 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,43 @@ | ||
| # Optional reference telemetry backend. Preloop never starts this — it is | ||
| # opt-in, loopback-bound, and pinned by digest. Credentials come from the | ||
| # environment (systemd LoadCredential / .env outside version control), | ||
| # never from this file. | ||
| # | ||
| # Upstream: https://github.com/openobserve/openobserve (AGPL-3.0). | ||
| # Run the stock image as a separate process; do not vendor or modify it. | ||
| services: | ||
| openobserve: | ||
| # Digest-pinned: a floating tag is not an immutable input. | ||
| image: public.ecr.aws/zinclabs/openobserve@sha256:88fb692ac791d3eaff69653a4a4686f1c7eceb9e105491d58d29ac2739560b3b | ||
| container_name: preloop-openobserve | ||
| restart: unless-stopped | ||
| # Loopback only. The OSS build has no SSO/RBAC — never put the UI on the | ||
| # public webhook origin. Front it with operator auth if shared. | ||
| ports: | ||
| - "127.0.0.1:5080:5080" | ||
| environment: | ||
| ZO_ROOT_USER_EMAIL: ${ZO_ROOT_USER_EMAIL:-admin@preloop.local} | ||
| ZO_ROOT_USER_PASSWORD: ${ZO_ROOT_USER_PASSWORD:-ChangeMe.Preloop1} | ||
| ZO_DATA_DIR: /data | ||
| # Short retention: this is operational telemetry (hours/days), not a | ||
| # data lake. Losing SQLite metadata makes the install inoperable, so | ||
| # back up the volume if you rely on it. | ||
| ZO_COMPACT_DATA_RETENTION_DAYS: "7" | ||
| ZO_TELEMETRY: "false" | ||
| volumes: | ||
| - openobserve-data:/data | ||
| # Measured caps: do not starve the VM pool. Re-measure on your host. | ||
| deploy: | ||
| resources: | ||
| limits: | ||
| cpus: "1.0" | ||
| memory: 2G | ||
| healthcheck: | ||
| test: ["CMD", "sh", "-c", "wget -qO- http://127.0.0.1:5080/healthz || exit 1"] | ||
| interval: 10s | ||
| timeout: 3s | ||
| retries: 10 | ||
| start_period: 20s | ||
|
|
||
| volumes: | ||
| openobserve-data: | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🔒 Security & Privacy | 🟠 Major | ⚡ Quick win
Require the root password.
Line 20 starts OpenObserve with a known administrator password when
ZO_ROOT_USER_PASSWORDis unset. Loopback binding does not protect against local users, compromised local processes, or port forwarding.Proposed fix
📝 Committable suggestion
🤖 Prompt for AI Agents