Skip to content

basic otel setup#2964

Open
Brendonovich wants to merge 5 commits intospacedriveapp:mainfrom
Brendonovich:basic-otel
Open

basic otel setup#2964
Brendonovich wants to merge 5 commits intospacedriveapp:mainfrom
Brendonovich:basic-otel

Conversation

@Brendonovich
Copy link
Contributor

@Brendonovich Brendonovich commented Jan 10, 2026 •
edited
Loading

Adds OpenTelemetry to sd-daemon. Enabling OTEL can be done by adding the following to spacedrive.json:

"telemetry": {
  "enabled": true,
  "endpoint": "http://localhost:4318/v1/traces",
  "service_name": "spacedrive-daemon",
  "headers": {},
  "timeout_secs": 10
}

To run a basic OTEL stack I'd recommend using Grafana:

docker run -p 3000:3000 -p 4317:4317 -p 4318:4318 --rm -it docker.io/grafana/otel-lgtm

Happy to change any of how this is configured

tembo[bot]
tembo bot reviewed Jan 10, 2026
View reviewed changes
@jamiepine @tembo
Update core/Cargo.toml
5840618 
Co-authored-by: tembo[bot] <208362400+tembo[bot]@users.noreply.github.com>
@cursor
Copy link

cursor bot commented Jan 10, 2026 •
edited
Loading

PR Summary

Introduces distributed tracing and consolidates config utilities.

  • Adds optional OpenTelemetry tracing in sd-core (telemetry feature): new config.telemetry block (schema v5), OTLP exporter, Spacedrive-rooted span filter, and graceful shutdown in daemon bootstrap
  • New sd-config crate provides default_data_dir; replaces prior implementations in core and Tauri, and updates dependencies accordingly
  • Instruments core paths with tracing::instrument (RPC execute_json_operation, file copy, directory listing, location add, search, tags)
  • Server: enables telemetry feature passthrough and adds tower-http TraceLayer for HTTP request spans
  • Dev DX: dev-with-daemon.ts checks for existing daemon before building/starting; minor UI cleanup (removes cache demo export)

Written by Cursor Bugbot for commit 5840618. Configure here.

@jamiepine
Copy link
Member

jamiepine commented Jan 11, 2026

@cursor resolve the merge conflicts

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

Reviewers

@tembo tembo[bot] tembo[bot] left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Brendonovich @jamiepine