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
feat(cli): add open telemetry support for Tracetest agent #3662
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
cli/config/configurator.go
Outdated
@@ -162,6 +162,19 @@ func (e invalidServerErr) Render() { | |||
} | |||
|
|||
func (c Configurator) populateConfigWithVersionInfo(ctx context.Context, cfg Config) (_ Config, _ error, isOSS bool) { | |||
cliVersion := Version | |||
if cliVersion == "dev" { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Helper code to help us to run the agent locally when the cliVersion is dev
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we point the agent to a different environment from a dev build too? I want to make sure this will allows us to be flexible enough when debugging
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes! Added an update to allow these config to be configured by env vars.
This PR adds support for OTel Collector on Tracetest Agent.
Changes
Checklist