Skip to content

Conversation

@simonpasquier
Copy link
Contributor

This change adds the -tags netgo,osusergo argument to Go commands to get fully-static binaries.

This change adds the `-tags netgo,osusergo` argument to Go commands to
get fully-static binaries.

Signed-off-by: Simon Pasquier <spasquie@redhat.com>
@simonpasquier simonpasquier requested a review from a team as a code owner November 8, 2023 08:21
@simonpasquier simonpasquier requested review from JoaoBraveCoding and sthaha and removed request for a team November 8, 2023 08:21
@openshift-ci openshift-ci bot requested a review from danielmellado November 8, 2023 08:21
@openshift-ci openshift-ci bot added the approved label Nov 8, 2023
@simonpasquier
Copy link
Contributor Author

Looking at https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/rhobs_observability-operator/381/pull-ci-rhobs-observability-operator-main-observability-operator-e2e/1721911199957782528 (which is a failed run), the logs indicate a problem with glibc:

./e2e.test: /lib64/libc.so.6: version `GLIBC_2.32' not found (required by ./e2e.test)

My suspicion is that the e2e.test binary isn't fully static.

@simonpasquier simonpasquier requested review from jan--f and removed request for JoaoBraveCoding and sthaha November 8, 2023 08:52
@jan--f
Copy link
Collaborator

jan--f commented Nov 8, 2023

/lgtm
looks like your suspicion is correct :)

@openshift-ci
Copy link

openshift-ci bot commented Nov 8, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jan--f, simonpasquier

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:
  • OWNERS [jan--f,simonpasquier]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-merge-bot openshift-merge-bot bot merged commit 1875113 into rhobs:main Nov 8, 2023
@simonpasquier simonpasquier deleted the fix-static-builds branch November 8, 2023 09:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants