Skip to content
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

Builder should provide a more helpful error if you mismatch the module type #10151

Open
austinlparker opened this issue May 14, 2024 · 0 comments
Labels
area:builder area:pipeline collector-telemetry healthchecker and other telemetry collection issues

Comments

@austinlparker
Copy link
Member

Is your feature request related to a problem? Please describe.
When using the collector builder, the error message when modules are mismatched isn't very helpful. For example, I tried to add an extension to the receiver section and got the following error:

Error: failed to compile the OpenTelemetry Collector distribution: go subcommand failed with args '[build -trimpath -o otelcol -ldflags=-s -w]': exit status 1, error message: # go.opentelemetry.io/collector/cmd/builder
./components.go:70:3: cannot use oauth2clientauthextension.NewFactory() (value of type extension.Factory) as receiver.Factory value in argument to receiver.MakeFactoryMap: extension.Factory does not implement receiver.Factory (missing method CreateLogsReceiver)
./components.go:71:3: cannot use oidcauthextension.NewFactory() (value of type extension.Factory) as receiver.Factory value in argument to receiver.MakeFactoryMap: extension.Factory does not implement receiver.Factory (missing method CreateLogsReceiver)

Describe the solution you'd like
It'd be nice if we could infer what the module was supposed to be and tell you that you've put the wrong thing in the wrong place, or perhaps even tell you where it should go based on its interface.

@TylerHelmuth TylerHelmuth added area:builder area:pipeline collector-telemetry healthchecker and other telemetry collection issues labels May 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:builder area:pipeline collector-telemetry healthchecker and other telemetry collection issues
Projects
None yet
Development

No branches or pull requests

2 participants