Skip to content

feat: integrate OTEL/Jaeger #3815

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

Open
wants to merge 42 commits into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
42 commits
Select commit Hold shift + click to select a range
b51afd1
wip
Feb 24, 2025
8ffdcc0
Merge remote-tracking branch 'remote/master' into feat/integrate-otel…
hthieu1110 Feb 24, 2025
4aab27d
Merge remote-tracking branch 'remote/master' into feat/integrate-otel…
Feb 25, 2025
5c373c6
wip: integrate with templo
Feb 25, 2025
8c4d553
Merge branch 'feat/integrate-otel-jaeger' of https://github.com/hthie…
hthieu1110 Feb 25, 2025
4616d29
wip: fix connect otel -> tempo
Feb 26, 2025
2fb245e
feat: added datasource + dashboard for tracing
Feb 26, 2025
5392a01
feat: added dashboard json config files
Feb 26, 2025
3181a85
chore: change visu type
Feb 26, 2025
af2c4b5
chore: revert removed part from docker-compose
hthieu1110 Feb 26, 2025
53b4724
wip
hthieu1110 Feb 27, 2025
71090c1
wip: add test traces
hthieu1110 Feb 28, 2025
ce12b5c
Merge remote-tracking branch 'remote/master' into feat/integrate-otel…
hthieu1110 Feb 28, 2025
93e2489
Merge remote-tracking branch 'remote/master' into feat/integrate-otel…
hthieu1110 Mar 3, 2025
754b15f
chore: remove unused code
hthieu1110 Mar 3, 2025
620e10a
feat: add traces for consensus
hthieu1110 Mar 4, 2025
bbc0cb5
Update dashboard
hthieu1110 Mar 5, 2025
9eda04a
Merge remote-tracking branch 'remote/master' into feat/integrate-otel…
hthieu1110 Mar 5, 2025
c25bd50
fix: go mod tidy
hthieu1110 Mar 5, 2025
9ea2b16
Merge branch 'master' into feat/integrate-otel-jaeger
hthieu1110 Mar 6, 2025
21ad30c
chore: remove unused method
hthieu1110 Mar 12, 2025
ae163be
Merge branch 'feat/integrate-otel-jaeger' of https://github.com/hthie…
hthieu1110 Mar 12, 2025
a0671d0
Merge remote-tracking branch 'remote/master' into feat/integrate-otel…
hthieu1110 Mar 12, 2025
7d76652
fix: addressed feedback
hthieu1110 Apr 10, 2025
b9778f0
fix: addressed feedback
hthieu1110 Apr 10, 2025
3e9dd7a
fix: un-expose port
hthieu1110 Apr 10, 2025
e0a7020
Update tm2/pkg/bft/consensus/state.go
hthieu1110 Apr 11, 2025
e8590fb
refactor: separate telemetry config
hthieu1110 Apr 14, 2025
f8437e6
chore: rename file
hthieu1110 Apr 14, 2025
254102f
Merge remote-tracking branch 'remote/master' into feat/integrate-otel…
hthieu1110 Apr 17, 2025
4366fb4
chore: rename tracer
hthieu1110 Apr 17, 2025
ee5e2e3
chore: change telemetry exporter name
hthieu1110 Apr 17, 2025
1b614b3
refactor: refactor tracing messages
hthieu1110 Apr 17, 2025
68bb23e
Merge branch 'master' into feat/integrate-otel-jaeger
hthieu1110 May 15, 2025
23377fa
Merge remote-tracking branch 'remote/master' into feat/integrate-otel…
hthieu1110 May 26, 2025
e1bc8ce
Merge branch 'master' into feat/integrate-otel-jaeger
hthieu1110 May 26, 2025
f495cd0
Merge branch 'feat/integrate-otel-jaeger' of https://github.com/hthie…
hthieu1110 May 26, 2025
8511a4b
chore: fix lint
hthieu1110 May 26, 2025
06d10f7
wip: address feedbacks
hthieu1110 May 28, 2025
caa91b6
fix: addressed feedbacks
hthieu1110 May 29, 2025
152b84f
Merge remote-tracking branch 'remote/master' into feat/integrate-otel…
hthieu1110 May 29, 2025
e6abffc
chore: tidy
hthieu1110 May 29, 2025
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions contribs/gnodev/go.mod
Original file line number Diff line number Diff line change
Expand Up @@ -89,6 +89,9 @@ require (
go.opentelemetry.io/otel v1.34.0 // indirect
go.opentelemetry.io/otel/exporters/otlp/otlpmetric/otlpmetricgrpc v1.34.0 // indirect
go.opentelemetry.io/otel/exporters/otlp/otlpmetric/otlpmetrichttp v1.34.0 // indirect
go.opentelemetry.io/otel/exporters/otlp/otlptrace v1.34.0 // indirect
go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracegrpc v1.34.0 // indirect
go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracehttp v1.34.0 // indirect
go.opentelemetry.io/otel/metric v1.34.0 // indirect
go.opentelemetry.io/otel/sdk v1.34.0 // indirect
go.opentelemetry.io/otel/sdk/metric v1.34.0 // indirect
Expand Down
6 changes: 6 additions & 0 deletions contribs/gnodev/go.sum

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

3 changes: 3 additions & 0 deletions contribs/gnofaucet/go.mod
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,9 @@ require (
go.opentelemetry.io/otel v1.34.0 // indirect
go.opentelemetry.io/otel/exporters/otlp/otlpmetric/otlpmetricgrpc v1.34.0 // indirect
go.opentelemetry.io/otel/exporters/otlp/otlpmetric/otlpmetrichttp v1.34.0 // indirect
go.opentelemetry.io/otel/exporters/otlp/otlptrace v1.34.0 // indirect
go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracegrpc v1.34.0 // indirect
go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracehttp v1.34.0 // indirect
go.opentelemetry.io/otel/metric v1.34.0 // indirect
go.opentelemetry.io/otel/sdk v1.34.0 // indirect
go.opentelemetry.io/otel/sdk/metric v1.34.0 // indirect
Expand Down
6 changes: 6 additions & 0 deletions contribs/gnofaucet/go.sum

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

3 changes: 3 additions & 0 deletions contribs/gnogenesis/go.mod
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,9 @@ require (
go.opentelemetry.io/otel v1.34.0 // indirect
go.opentelemetry.io/otel/exporters/otlp/otlpmetric/otlpmetricgrpc v1.34.0 // indirect
go.opentelemetry.io/otel/exporters/otlp/otlpmetric/otlpmetrichttp v1.34.0 // indirect
go.opentelemetry.io/otel/exporters/otlp/otlptrace v1.34.0 // indirect
go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracegrpc v1.34.0 // indirect
go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracehttp v1.34.0 // indirect
go.opentelemetry.io/otel/metric v1.34.0 // indirect
go.opentelemetry.io/otel/sdk v1.34.0 // indirect
go.opentelemetry.io/otel/sdk/metric v1.34.0 // indirect
Expand Down
8 changes: 8 additions & 0 deletions contribs/gnogenesis/go.sum

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

3 changes: 3 additions & 0 deletions contribs/gnohealth/go.mod
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,9 @@ require (
go.opentelemetry.io/otel v1.34.0 // indirect
go.opentelemetry.io/otel/exporters/otlp/otlpmetric/otlpmetricgrpc v1.34.0 // indirect
go.opentelemetry.io/otel/exporters/otlp/otlpmetric/otlpmetrichttp v1.34.0 // indirect
go.opentelemetry.io/otel/exporters/otlp/otlptrace v1.34.0 // indirect
go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracegrpc v1.34.0 // indirect
go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracehttp v1.34.0 // indirect
go.opentelemetry.io/otel/metric v1.34.0 // indirect
go.opentelemetry.io/otel/sdk v1.34.0 // indirect
go.opentelemetry.io/otel/sdk/metric v1.34.0 // indirect
Expand Down
8 changes: 8 additions & 0 deletions contribs/gnohealth/go.sum

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

3 changes: 3 additions & 0 deletions contribs/gnokeykc/go.mod
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,9 @@ require (
go.opentelemetry.io/otel v1.34.0 // indirect
go.opentelemetry.io/otel/exporters/otlp/otlpmetric/otlpmetricgrpc v1.34.0 // indirect
go.opentelemetry.io/otel/exporters/otlp/otlpmetric/otlpmetrichttp v1.34.0 // indirect
go.opentelemetry.io/otel/exporters/otlp/otlptrace v1.34.0 // indirect
go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracegrpc v1.34.0 // indirect
go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracehttp v1.34.0 // indirect
go.opentelemetry.io/otel/metric v1.34.0 // indirect
go.opentelemetry.io/otel/sdk v1.34.0 // indirect
go.opentelemetry.io/otel/sdk/metric v1.34.0 // indirect
Expand Down
8 changes: 8 additions & 0 deletions contribs/gnokeykc/go.sum

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

3 changes: 3 additions & 0 deletions contribs/gnomigrate/go.mod
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,9 @@ require (
go.opentelemetry.io/otel v1.34.0 // indirect
go.opentelemetry.io/otel/exporters/otlp/otlpmetric/otlpmetricgrpc v1.34.0 // indirect
go.opentelemetry.io/otel/exporters/otlp/otlpmetric/otlpmetrichttp v1.34.0 // indirect
go.opentelemetry.io/otel/exporters/otlp/otlptrace v1.34.0 // indirect
go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracegrpc v1.34.0 // indirect
go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracehttp v1.34.0 // indirect
go.opentelemetry.io/otel/metric v1.34.0 // indirect
go.opentelemetry.io/otel/sdk v1.34.0 // indirect
go.opentelemetry.io/otel/sdk/metric v1.34.0 // indirect
Expand Down
8 changes: 8 additions & 0 deletions contribs/gnomigrate/go.sum

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions examples/gno.land/p/moul/xmath/generator.go
Original file line number Diff line number Diff line change
Expand Up @@ -158,7 +158,7 @@ func main() {
}

// Write source file
if err := os.WriteFile("xmath.gen.gno", formattedSource, 0644); err != nil {
if err := os.WriteFile("xmath.gen.gno", formattedSource, 0o644); err != nil {
log.Fatal(err)
}

Expand All @@ -176,7 +176,7 @@ func main() {
}

// Write test file
if err := os.WriteFile("xmath.gen_test.gno", formattedTest, 0644); err != nil {
if err := os.WriteFile("xmath.gen_test.gno", formattedTest, 0o644); err != nil {
log.Fatal(err)
}

Expand Down
29 changes: 27 additions & 2 deletions gno.land/cmd/gnoland/start.go
Original file line number Diff line number Diff line change
Expand Up @@ -228,8 +228,33 @@
}

// Initialize telemetry
if err := telemetry.Init(*cfg.Telemetry); err != nil {
return fmt.Errorf("unable to initialize telemetry, %w", err)
// Return the providers so we can gracefully shutdown them if needed
metricsProvider, err := telemetry.InitMetrics(*cfg.Telemetry)
if err != nil {
return fmt.Errorf("unable to initialize metrics, %w", err)
}

Check warning on line 235 in gno.land/cmd/gnoland/start.go

View check run for this annotation

Codecov / codecov/patch

gno.land/cmd/gnoland/start.go#L234-L235

Added lines #L234 - L235 were not covered by tests

tracesProvider, err := telemetry.InitTraces(*cfg.Telemetry)
if err != nil {
return fmt.Errorf("unable to initialize traces, %w", err)
}

Check warning on line 240 in gno.land/cmd/gnoland/start.go

View check run for this annotation

Codecov / codecov/patch

gno.land/cmd/gnoland/start.go#L239-L240

Added lines #L239 - L240 were not covered by tests

if cfg.Telemetry.GracefulShutdownTelemetry {
if metricsProvider != nil {
defer func() {
if err := metricsProvider.Shutdown(ctx); err != nil {
logger.Error("unable to shutdown metrics provider", "error", err)
}

Check warning on line 247 in gno.land/cmd/gnoland/start.go

View check run for this annotation

Codecov / codecov/patch

gno.land/cmd/gnoland/start.go#L243-L247

Added lines #L243 - L247 were not covered by tests
}()
}

if tracesProvider != nil {
defer func() {
if err := tracesProvider.Shutdown(ctx); err != nil {
logger.Error("unable to shutdown traces provider", "error", err)
}

Check warning on line 255 in gno.land/cmd/gnoland/start.go

View check run for this annotation

Codecov / codecov/patch

gno.land/cmd/gnoland/start.go#L251-L255

Added lines #L251 - L255 were not covered by tests
}()
}
}

// Print the starting graphic
Expand Down
4 changes: 3 additions & 1 deletion go.mod
Original file line number Diff line number Diff line change
Expand Up @@ -33,9 +33,11 @@ require (
go.opentelemetry.io/otel v1.34.0
go.opentelemetry.io/otel/exporters/otlp/otlpmetric/otlpmetricgrpc v1.34.0
go.opentelemetry.io/otel/exporters/otlp/otlpmetric/otlpmetrichttp v1.34.0
go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracehttp v1.34.0
go.opentelemetry.io/otel/metric v1.34.0
go.opentelemetry.io/otel/sdk v1.34.0
go.opentelemetry.io/otel/sdk/metric v1.34.0
go.opentelemetry.io/otel/trace v1.34.0
go.uber.org/multierr v1.11.0
go.uber.org/zap v1.27.0
go.uber.org/zap/exp v0.3.0
Expand Down Expand Up @@ -63,7 +65,7 @@ require (
github.com/zondax/hid v0.9.2 // indirect
github.com/zondax/ledger-go v0.14.3 // indirect
go.opentelemetry.io/auto/sdk v1.1.0 // indirect
go.opentelemetry.io/otel/trace v1.34.0 // indirect
go.opentelemetry.io/otel/exporters/otlp/otlptrace v1.34.0 // indirect
go.opentelemetry.io/proto/otlp v1.5.0 // indirect
golang.org/x/sys v0.32.0 // indirect
golang.org/x/text v0.24.0 // indirect
Expand Down
4 changes: 4 additions & 0 deletions go.sum

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading
Loading