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

[load] Load tests fail to run #38316

Closed
atoulme opened this issue Mar 3, 2025 · 2 comments
Closed

[load] Load tests fail to run #38316

atoulme opened this issue Mar 3, 2025 · 2 comments
Labels
ci-cd CI, CD, testing, build issues

Comments

@atoulme
Copy link
Contributor

atoulme commented Mar 3, 2025

Component(s)

No response

Describe the issue you're reporting

See https://github.com/open-telemetry/opentelemetry-collector-contrib/actions/runs/13637121634/job/38118527106

Error: /home/ghrunner/go/pkg/mod/go.opentelemetry.io/collector/exporter/exporterhelper/xexporterhelper@v0.120.0/profiles.go:37:9: cannot use &profilesRequest{…} (value of type *profilesRequest) as exporterhelper.Request value in return statement: *profilesRequest does not implement request.Request (wrong type for method MergeSplit)
		have MergeSplit(context.Context, exporterbatcher.MaxSizeConfig, exporterhelper.Request) ([]exporterhelper.Request, error)
		want MergeSplit(context.Context, exporterbatcher.SizeConfig, request.Request) ([]request.Request, error)
Error: /home/ghrunner/go/pkg/mod/go.opentelemetry.io/collector/exporter/exporterhelper/xexporterhelper@v0.1[20](https://github.com/open-telemetry/opentelemetry-collector-contrib/actions/runs/13637121634/job/38118527106#step:8:21).0/profiles.go:44:91: undefined: exporterqueue.Unmarshaler
Error: /home/ghrunner/go/pkg/mod/go.opentelemetry.io/collector/exporter/exporterhelper/xexporterhelper@v0.120.0/profiles.go:55:43: impossible type assertion: req.(*profilesRequest)
	*profilesRequest does not implement exporterhelper.Request (wrong type for method MergeSplit)
		have MergeSplit(context.Context, exporterbatcher.MaxSizeConfig, exporterhelper.Request) ([]exporterhelper.Request, error)
		want MergeSplit(context.Context, exporterbatcher.SizeConfig, request.Request) ([]request.Request, error)
Error: /home/ghrunner/go/pkg/mod/go.opentelemetry.io/collector/exporter/exporterhelper/xexporterhelper@v0.120.0/profiles.go:63:9: cannot use req (variable of type *profilesRequest) as exporterhelper.Request value in return statement: *profilesRequest does not implement request.Request (wrong type for method MergeSplit)
		have MergeSplit(context.Context, exporterbatcher.MaxSizeConfig, exporterhelper.Request) ([]exporterhelper.Request, error)
		want MergeSplit(context.Context, exporterbatcher.SizeConfig, request.Request) ([]request.Request, error)
Error: /home/ghrunner/go/pkg/mod/go.opentelemetry.io/collector/exporter/exporterhelper/xexporterhelper@v0.120.0/profiles.go:98:12: undefined: internal.WithMarshaler
Error: /home/ghrunner/go/pkg/mod/go.opentelemetry.io/collector/exporter/exporterhelper/xexporterhelper@v0.120.0/profiles.go:98:62: undefined: internal.WithUnmarshaler
Error: /home/ghrunner/go/pkg/mod/go.opentelemetry.io/collector/exporter/exporterhelper/xexporterhelper@v0.120.0/profiles_batch.go:18:15: impossible type assertion: r2.(*profilesRequest)
	*profilesRequest does not implement exporterhelper.Request (wrong type for method MergeSplit)
		have MergeSplit(context.Context, exporterbatcher.MaxSizeConfig, exporterhelper.Request) ([]exporterhelper.Request, error)
		want MergeSplit(context.Context, exporterbatcher.SizeConfig, request.Request) ([]request.Request, error)
Error: /home/ghrunner/go/pkg/mod/go.opentelemetry.io/collector/exporter/exporterhelper/xexporterhelper@v0.120.0/profiles_batch.go:26:25: invalid operation: cfg.MaxSizeItems == 0 (mismatched types *int and untyped int)
Error: /home/ghrunner/go/pkg/mod/go.opentelemetry.io/collector/exporter/exporterhelper/xexporterhelper@v0.120.0/profiles_batch.go:27:35: cannot use req (variable of type *profilesRequest) as exporterhelper.Request value in array or slice literal: *profilesRequest does not implement request.Request (wrong type for method MergeSplit)
		have MergeSplit(context.Context, exporterbatcher.MaxSizeConfig, exporterhelper.Request) ([]exporterhelper.Request, error)
		want MergeSplit(context.Context, exporterbatcher.SizeConfig, request.Request) ([]request.Request, error)
Error: /home/ghrunner/go/pkg/mod/go.opentelemetry.io/collector/exporter/exporterhelper/xexporterhelper@v0.120.0/profiles_batch.go:40:25: invalid operation: req.ItemsCount() > cfg.MaxSizeItems (mismatched types int and *int)
Error: /home/ghrunner/go/pkg/mod/go.opentelemetry.io/collector/exporter/exporterhelper/xexporterhelper@v0.120.0/profiles_batch.go:40:25: too many errors
# go.opentelemetry.io/collector/processor/memorylimiterprocessor
Error: /home/ghrunner/go/pkg/mod/go.opentelemetry.io/collector/processor/memorylimiterprocessor@v0.120.1-0.20250303102058-a9bca17f1a4c/factory.go:51:[23](https://github.com/open-telemetry/opentelemetry-collector-contrib/actions/runs/13637121634/job/38118527106#step:8:24): undefined: memorylimiter.NewDefaultConfig
make: *** [Makefile:397: oteltestbedcol] Error 1
Error: Process completed with exit code 2.
@atoulme atoulme added needs triage New item requiring triage ci-cd CI, CD, testing, build issues and removed needs triage New item requiring triage labels Mar 3, 2025
@atoulme
Copy link
Contributor Author

atoulme commented Mar 3, 2025

An indirect dependency on xexporterhelper, pointing to an older version, breaks the go build. The fix is to update all otel packages to 0.121.0 which should have uniform imports. This is because xexporterhelper is new, I guess.

@songy23
Copy link
Member

songy23 commented Mar 4, 2025

@songy23 songy23 closed this as completed Mar 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci-cd CI, CD, testing, build issues
Projects
None yet
Development

No branches or pull requests

2 participants