Besides the change in the title, we also change a configuration knob, tracing.url to tracing.endpoint to be more technically correct.
There appear to be many more changes in this PR than are actually there. This is because Go's dependency manager, go modules, automatically adds to the repository the new dependencies needed (and removes the old ones) for the change from the Jaeger Trace exporter libraries to the OpenTelemetry Trace exporter libraries.
Thus, the go.mod, go.sum, and any vendor/* files can be ignored for this PR, as they are managed automatically by the go tools.
The only files to actually review are:
[ ] cmd/refresh-uids-from-ferry/main.go
[ ] cmd/refresh-uids-from-ferry/main_test.go
[ ] cmd/run-onboarding-managed-tokens/main.go
[ ] cmd/token-push/main.go
[ ] internal/tracing/tracing.go (where the biggest changes are - the above files just use the changes from this file)
retzkek
commented
1 month ago
Closes #96
Besides the change in the title, we also change a configuration knob,
tracing.urltotracing.endpointto be more technically correct.There appear to be many more changes in this PR than are actually there. This is because Go's dependency manager,
go modules, automatically adds to the repository the new dependencies needed (and removes the old ones) for the change from the Jaeger Trace exporter libraries to the OpenTelemetry Trace exporter libraries.Thus, the
go.mod,go.sum, and anyvendor/*files can be ignored for this PR, as they are managed automatically by thegotools.The only files to actually review are: