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

[clickhouse] Dependency update breaks build when cross-compiling #18863

Closed
atoulme opened this issue Feb 21, 2023 · 3 comments · Fixed by #18864
Closed

[clickhouse] Dependency update breaks build when cross-compiling #18863

atoulme opened this issue Feb 21, 2023 · 3 comments · Fixed by #18864
Assignees
Labels
bug Something isn't working exporter/clickhouse

Comments

@atoulme
Copy link
Contributor

atoulme commented Feb 21, 2023

Component(s)

exporter/clickhouse

What happened?

Build fails when cross-compiling to ppc64le or win386.

cd ./cmd/otelcontribcol && GO111MODULE=on CGO_ENABLED=0 go build -trimpath -o ../../bin/otelcontribcol_linux_ppc64le \
	-ldflags "-X github.com/open-telemetry/opentelemetry-collector-contrib/internal/otelcontribcore/internal/version.Version=9d0020e" -tags "" .
# github.com/ClickHouse/clickhouse-go/v2/lib/column
Error: /home/runner/go/pkg/mod/github.com/!click!house/clickhouse-go/v2@v2.6.2/lib/column/fixed_string.go:100:40: too many arguments in call to binary.Str2Bytes
	have (string, int)
	want (string)
Error: /home/runner/go/pkg/mod/github.com/!click!house/clickhouse-go/v2@v2.6.2/lib/column/fixed_string.go:116:42: too many arguments in call to binary.Str2Bytes
	have (string, int)
	want (string)
Error: /home/runner/go/pkg/mod/github.com/!click!house/clickhouse-go/v2@v2.6.2/lib/column/fixed_string.go:142:31: too many arguments in call to binary.Str2Bytes
	have (string, int)
	want (string)
Error: /home/runner/go/pkg/mod/github.com/!click!house/clickhouse-go/v2@v2.6.2/lib/column/fixed_string.go:147:33: too many arguments in call to binary.Str2Bytes
	have (string, int)
	want (string)
Error: /home/runner/go/pkg/mod/github.com/!click!house/clickhouse-go/v2@v2.6.2/lib/column/string.go:74:50: too many arguments in call to binary.Str2Bytes
	have (string, int)
	want (string)
make: *** [Makefile:259: otelcontribcol] Error 2
Error: Process completed with exit code 2.

https://github.com/open-telemetry/opentelemetry-collector-contrib/actions/runs/4236028542/jobs/7360771718

https://github.com/open-telemetry/opentelemetry-collector-contrib/actions/runs/4235817053/jobs/7360591861

Collector version

latest main

Environment information

No response

OpenTelemetry Collector configuration

No response

Log output

No response

Additional context

No response

@atoulme atoulme added bug Something isn't working needs triage New item requiring triage exporter/clickhouse labels Feb 21, 2023
@github-actions
Copy link
Contributor

Pinging code owners:

See Adding Labels via Comments if you do not have permissions to add labels yourself.

@atoulme atoulme removed the needs triage New item requiring triage label Feb 21, 2023
@github-actions
Copy link
Contributor

Pinging code owners for exporter/clickhouse: @hanjm @dmitryax @Frapschen. See Adding Labels via Comments if you do not have permissions to add labels yourself.

@codeboten codeboten self-assigned this Feb 21, 2023
codeboten pushed a commit to codeboten/opentelemetry-collector-contrib that referenced this issue Feb 21, 2023
Fixes open-telemetry#18863

Signed-off-by: Alex Boten <aboten@lightstep.com>
codeboten pushed a commit that referenced this issue Feb 21, 2023
Fixes #18863

Signed-off-by: Alex Boten <aboten@lightstep.com>
@atoulme
Copy link
Contributor Author

atoulme commented Feb 22, 2023

Following up with a report to clickhouse here: ClickHouse/clickhouse-go#913

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working exporter/clickhouse
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants