Skip to content

Conversation

djeebus
Copy link
Contributor

@djeebus djeebus commented Oct 1, 2025

Note

Adds optional OpenTelemetry logging to clean-nfs-cache with a new --otel-collector-endpoint flag and Nomad wiring from secret, plus minor refactors.

  • Orchestrator (clean-nfs-cache):
    • OTel logging: Integrates OTel log core (resource, OTLP gRPC exporter) and supports --otel-collector-endpoint flag; logger initialized with optional OTel core.
    • Refactors: rename results.union to results.sum; simplify deletion logging; remove unused size-formatting and usage logs.
    • Constants: introduce serviceVersion, commitSHA, and serviceName grouping.
    • Deps: promote go.opentelemetry.io/otel/exporters/otlp/otlplog/otlploggrpc to direct dependency.
  • IaC/Nomad:
    • Pass otel_collector_endpoint from google_secret_manager_secret_version.grafana_logs_url into job template and conditionally append CLI arg in jobs/clean-nfs-cache.hcl.

Written by Cursor Bugbot for commit a554f56. This will update automatically on new commits. Configure here.

@e2b-request-same-site-reviewers e2b-request-same-site-reviewers bot requested review from tomassrnka and removed request for dobrac, ValentaTomas and jakubno October 1, 2025 21:01
@djeebus djeebus marked this pull request as draft October 1, 2025 21:02
cursor[bot]

This comment was marked as outdated.

@djeebus djeebus marked this pull request as ready for review October 6, 2025 19:51
cursor[bot]

This comment was marked as outdated.

@ValentaTomas ValentaTomas self-assigned this Oct 7, 2025
@ValentaTomas ValentaTomas removed the request for review from tomassrnka October 7, 2025 21:20
@djeebus djeebus merged commit b459c54 into main Oct 7, 2025
26 checks passed
@djeebus djeebus deleted the add-logging branch October 7, 2025 22:30
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.

2 participants