Skip to content

AIX build fails to run with CGO_ENABLED=0 #1806

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.

Sign up for GitHub

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

Jump to bottom
Closed
atoulme opened this issue Mar 4, 2025 · 1 comment · Fixed by #1809
Closed

AIX build fails to run with CGO_ENABLED=0 #1806

atoulme opened this issue Mar 4, 2025 · 1 comment · Fixed by #1809
Labels
os:aix package:net

Comments

@atoulme
Copy link
Contributor

atoulme commented Mar 4, 2025

Describe the bug
See https://github.com/open-telemetry/opentelemetry-collector/actions/runs/13653662476/job/38167780904?pr=12540

On update to v4.25.2, we see a compilation error for aix:

pushd cmd/otelcorecol && CGO_ENABLED=0 go build -trimpath -o ../../bin/otelcorecol_aix_ppc64 \
	-tags  ./cmd/otelcorecol && popd
~/work/opentelemetry-collector/opentelemetry-collector/cmd/otelcorecol ~/work/opentelemetry-collector/opentelemetry-collector
# github.com/shirou/gopsutil/v4/net
Error: /home/runner/go/pkg/mod/github.com/shirou/gopsutil/[email protected]/net/net_aix_nocgo.go:[9](https://github.com/open-telemetry/opentelemetry-collector/actions/runs/13653662476/job/38167780904?pr=12540#step:5:10)2:10: not enough return values
	have ([]IOCountersStat)
	want ([]IOCountersStat, error)
make: *** [Makefile:159: otelcorecol] Error 1
@atoulme atoulme mentioned this issue Mar 4, 2025
Closed
1 task
@Lomanic
Copy link
Collaborator

Lomanic commented Mar 6, 2025

Can reproduce with GOOS=aix GOARCH=ppc64 CGO_ENABLED=0 go build ./net (probably also with CGO_ENABLED=1)

#1797 missed aix, because it was never added to the Makefile.

Lomanic added a commit to Lomanic/gopsutil that referenced this issue Mar 6, 2025
@Lomanic
[net][aix] Add missing return value in IOCountersWithContext()
fe3df2b 
Amends shirou#1797
Fixes shirou#1806
@Lomanic Lomanic mentioned this issue Mar 6, 2025
Merged
Lomanic added a commit to Lomanic/gopsutil that referenced this issue Mar 6, 2025
@Lomanic
[net][aix] Add missing return value in IOCountersWithContext()
6c9cea2 
Amends shirou#1797
Fixes shirou#1806
This was referenced Mar 19, 2025
Merged
Merged
github-merge-queue bot pushed a commit to open-telemetry/opentelemetry-collector that referenced this issue Mar 19, 2025
@jade-guiton-dd
Revert "fix(deps): update module github.com/shirou/gopsutil/v4 to v4.…
665b35f 
…25.2 (#12654)" (#12674)

#### Description

The cross-build test for the AIX platform (a tier 3 platform) is failing
on main after the upgrade #12654. This is due to a bug in that patch
release that was reported
[here](shirou/gopsutil#1806), but the fix
hasn't been released yet. For that reason, I suggest reverting the
upgrade until the next release.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
os:aix package:net
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[net][host][aix] Fix build and Makefile Lomanic/gopsutil
2 participants
@atoulme @Lomanic