GH-51285: [CI] Fix AMD64 Conda Integration Test failure - #51287
Conversation
|
|
|
|
|
@zeroshade Could you or one of your fellow Go maintainers help on this? (if only by reviewing) |
|
It looks like mamba install go is now installing the nocgo variant of Go, so we may need to explicitly install the cgo-enabled variant. I’ll try that later. |
|
Starting with build 5, the Currently,
|
|
|
lidavidm
left a comment
There was a problem hiding this comment.
It's not clear to me but maybe go-cgo is the preferred way to get this? https://anaconda.org/channels/conda-forge/packages/go-cgo/overview (although this talks about recipes)
|
@lidavidm Thanks for the review. I'll try |
lidavidm
left a comment
There was a problem hiding this comment.
It appears integration passes!
|
@lidavidm Thanks. |
Rationale for this change
AMD64 Conda Integration Test failure
https://github.com/apache/arrow/actions/runs/34469696916/job/102846552090?pr=51284
The Conda integration test requires cgo, but
mamba install gocurrently selects thenocgovariant by default.What changes are included in this PR?
Selects the cgo-enabled Go package for the Conda integration test.
Before this change,
mambaselected thenocgovariant:After this change, mamba installs the Go package with cgo support:
Are these changes tested?
Yes.
Are there any user-facing changes?
No.