Skip to content

feat(OpenTelemetry): Integrate OpenTelemetry into agent #90

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Jun 6, 2025
Merged

feat(OpenTelemetry): Integrate OpenTelemetry into agent #90

merged 2 commits into from
Jun 6, 2025

Conversation

cedi
Copy link
Member

@cedi cedi commented Jun 6, 2025

  • integrate OpenTelemetry logging with zap logger for better observability
  • add OpenTelemetry gRPC middleware for enhanced tracing capabilities
  • document new OTLP exporter endpoint for better configuration guidance

@github-actions github-actions bot added enhancement New feature or request api API related changes cli CLI related changes agent agent related changes hardware changes related to hardware interactions util changes related to utilities labels Jun 6, 2025
@cedi cedi mentioned this pull request Jun 6, 2025
Closed
@github-actions github-actions bot added the documentation Improvements or additions to documentation label Jun 6, 2025
@cedi cedi requested a review from xvzf June 6, 2025 16:54
cedi added 2 commits June 6, 2025 22:28
@cedi
feat(OpenTelemetry): Integrate OpenTelemetry into agent
2ecbc0b 
- integrate OpenTelemetry logging with zap logger for better observability
- add OpenTelemetry gRPC middleware for enhanced tracing capabilities
- document new OTLP exporter endpoint for better configuration guidance
@cedi
docs: document OTEL env var
7941e6a
@cedi cedi force-pushed the feature/add_open_telemetry branch from 611e2ce to 7941e6a Compare June 6, 2025 20:28
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Jun 6, 2025

Merging this branch will not change overall coverage

Impacted Packages Coverage Δ 🤖
github.com/compute-blade-community/compute-blade-agent/cmd/agent 0.00% (ø)
github.com/compute-blade-community/compute-blade-agent/cmd/bladectl 0.00% (ø)
github.com/compute-blade-community/compute-blade-agent/cmd/bladectl/config 0.00% (ø)
github.com/compute-blade-community/compute-blade-agent/internal/agent 0.00% (ø)
github.com/compute-blade-community/compute-blade-agent/internal/api 0.00% (ø)
github.com/compute-blade-community/compute-blade-agent/pkg/agent 100.00% (ø)
github.com/compute-blade-community/compute-blade-agent/pkg/hal 0.00% (ø)
github.com/compute-blade-community/compute-blade-agent/pkg/log 0.00% (ø)
Coverage by file

Changed files (no unit tests)

Changed File Coverage Δ Total Covered Missed 🤖
github.com/compute-blade-community/compute-blade-agent/cmd/agent/main.go 0.00% (ø) 92 (+18) 0 92 (+18)
github.com/compute-blade-community/compute-blade-agent/cmd/bladectl/cmd_identify.go 0.00% (ø) 23 0 23
github.com/compute-blade-community/compute-blade-agent/cmd/bladectl/config/config.go 0.00% (ø) 22 0 22
github.com/compute-blade-community/compute-blade-agent/internal/agent/agent.go 0.00% (ø) 154 0 154
github.com/compute-blade-community/compute-blade-agent/internal/api/api.go 0.00% (ø) 50 (+1) 0 50 (+1)
github.com/compute-blade-community/compute-blade-agent/internal/api/options.go 0.00% (ø) 11 0 11
github.com/compute-blade-community/compute-blade-agent/pkg/agent/state.go 100.00% (ø) 31 31 0
github.com/compute-blade-community/compute-blade-agent/pkg/hal/hal_bcm2711.go 0.00% (ø) 188 0 188
github.com/compute-blade-community/compute-blade-agent/pkg/hal/hal_bcm2711_simulated.go 0.00% (ø) 0 0 0
github.com/compute-blade-community/compute-blade-agent/pkg/hal/hal_bcm2711_standardfanunit.go 0.00% (ø) 31 0 31
github.com/compute-blade-community/compute-blade-agent/pkg/hal/smartfanunit.go 0.00% (ø) 78 0 78
github.com/compute-blade-community/compute-blade-agent/pkg/log/interceptor_logger.go 0.00% (ø) 18 (+1) 0 18 (+1)
github.com/compute-blade-community/compute-blade-agent/pkg/log/logger.go 0.00% (ø) 6 0 6

Please note that the "Total", "Covered", and "Missed" counts above refer to code statements instead of lines of code. The value in brackets refers to the test coverage of that file in the old version of the code.

@cedi cedi merged commit 7ec49ce into main Jun 6, 2025
14 checks passed
@cedi cedi deleted the feature/add_open_telemetry branch June 6, 2025 20:43
@github-actions github-actions bot mentioned this pull request Jun 6, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@xvzf xvzf Awaiting requested review from xvzf

Assignees
No one assigned
Labels
agent agent related changes api API related changes cli CLI related changes documentation Improvements or additions to documentation enhancement New feature or request hardware changes related to hardware interactions util changes related to utilities
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@cedi