Skip to content

feat: add tracing - #274

Open
Patrick Derks (TrayserCassa) wants to merge 1 commit into
mainfrom
add-tracing
Open

Patrick Derks (TrayserCassa) wants to merge 1 commit into
mainfrom
add-tracing

Conversation

@TrayserCassa

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions

github-actions Bot commented Oct 7, 2026 •

Copy link
Copy Markdown
Contributor

Preview helm chart

helm upgrade --install shopware-operator \
  oci://ghcr.io/shopware/shopware-operator-preview/operator \
  --version 0.0.0-add-tracing.ge310ddf
Chart version 0.0.0-add-tracing.ge310ddf
Image tag add-tracing
Commit e310ddf

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟡 Changes recommended

Jaeger exposes unauthenticated ports beyond localhost, and invalid UTF-8 diagnostics can prevent trace-batch export.

Review effort: Balanced
Findings: 1 High severity · 2 Medium severity

Open (3)
What changed in this PR

Adds optional OpenTelemetry tracing across the operator’s reconciliation and Kubernetes operations.

Changes:

  • Adds OTLP exporters, span helpers, and tracing tests.
  • Instruments controllers, state management, and pod execution.
  • Adds Helm configuration, documentation, and local Jaeger support.
File Description
Makefile Starts Jaeger for local tracing.
internal/​tracing/​tracing.go Configures exporters and span helpers.
internal/​tracing/​tracing_test.go Tests tracing helpers.
internal/​tracing/​kubernetes.go Instruments Kubernetes HTTP requests.
internal/​tracing/​kubernetes_test.go Tests request tracing and filtering.
internal/​manager/​status.go Traces store status updates.
internal/​manager/​queue.go Traces queue updates and errors.
internal/​manager/​manager.go Traces state and resource handlers.
internal/​manager/​base/​resources.go Adds resource reconciliation spans.
internal/​k8s/​exec.go Traces pod execution diagnostics.
internal/​deployment/​queue.go Traces queue-stat collection.
internal/​controller/​storesnapshot_base_reconciler.go Traces snapshot reconciliation and status.
internal/​controller/​storeexec_status.go Records execution status events.
internal/​controller/​storeexec_controller.go Adds execution reconciliation spans.
internal/​controller/​storedebuginstance_status.go Records debug-instance status events.
internal/​controller/​storedebuginstance_controller.go Adds debug-instance reconciliation spans.
internal/​controller/​store_controller.go Traces store reconciliation and errors.
internal/​config/​config.go Adds the tracing enable flag.
helm/​values.yaml Defines tracing settings.
helm/​templates/​deployment.yaml Passes tracing environment variables.
helm/​README.md Documents tracing configuration.
go.sum Updates dependency checksums.
go.mod Adds tracing dependencies and upgrades.
cmd/​main.go Initializes and shuts down tracing.
api/​v1/​storedebuginstance_status.go Adds latest-condition access.
api/​v1/​execstatus.go Adds latest-condition access.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread Makefile
Comment thread internal/k8s/exec.go Outdated
Comment thread internal/tracing/tracing.go Outdated
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