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

make test setup-envtest unable to find version for linux #7630

Closed
1 task
sk593 opened this issue May 23, 2024 · 2 comments · Fixed by #7632
Closed
1 task

make test setup-envtest unable to find version for linux #7630

sk593 opened this issue May 23, 2024 · 2 comments · Fixed by #7632
Assignees
Labels
bug Something is broken or not working as expected triaged This issue has been reviewed and triaged

Comments

@sk593
Copy link
Contributor

sk593 commented May 23, 2024

Steps to reproduce

Running into this error when installing the envtest tools during make test. This was caught during PR checks of the unit tests

/bin/sh: 1: /home/runner/go/bin/setup-envtest: not found
=> Installing Kubebuilder test tools...
go: downloading sigs.k8s.io/controller-runtime/tools/setup-envtest v0.0.0-20240522175850-2e9781e9fc60
go: downloading github.com/spf13/afero v1.6.0
go: downloading go.uber.org/zap v1.26.0
go: downloading sigs.k8s.io/yaml v1.3.0
go: downloading golang.org/x/text v0.14.0
go: downloading go.uber.org/multierr v1.10.0
=> Instructions:
=> Set environment variable KUBEBUILDER_ASSETS for tests.
=> KUBEBUILDER_ASSETS=""
unable to find a version that was supported for platform linux/amd64
KUBEBUILDER_ASSETS="" CGO_ENABLED=1 gotestsum --junitfile ./dist/unit_test/results.xml -- -v ./pkg/... -race -coverprofile ./dist/unit_test/ut_coverage.out

This is happening in test.mk where the controller-runtime package gets downloaded and used:

@echo "$(ARROW) Installing Kubebuilder test tools..."

This seems to be isolated to linux and it's not happening for other archiectures

Observed behavior

No response

Desired behavior

We'd like the latest package to download for linux without the need for a hardcoded version. It's working locally on macos.

Workaround

This is a blocking issue to PRs so we've hardcoded a previous version that we know works. We definitely want to remove the workaround so we don't have a hardcoded package version in our tests. There are a couple options for that.

  1. There's an open issue in the controller-runtime repo to address this bug: [bug] setup-envtest unable to find version for linux kubernetes-sigs/controller-runtime#2838. Once this gets fixed, we can revert the hardcoded version back to latest
  2. We can move this somewhere dependabot can manage it for us

rad Version

build test in PR

Operating system

No response

Additional context

No response

Would you like to support us?

  • Yes, I would like to support you

AB#12407

@sk593 sk593 added the bug Something is broken or not working as expected label May 23, 2024
@radius-triage-bot
Copy link

👋 @sk593 Thanks for filing this bug report.

A project maintainer will review this report and get back to you soon. If you'd like immediate help troubleshooting, please visit our Discord server.

For more information on our triage process please visit our triage overview

@willtsai willtsai added the triaged This issue has been reviewed and triaged label May 23, 2024
@radius-triage-bot
Copy link

👍 We've reviewed this issue and have agreed to add it to our backlog. Please subscribe to this issue for notifications, we'll provide updates when we pick it up.

We also welcome community contributions! If you would like to pick this item up sooner and submit a pull request, please visit our contribution guidelines and assign this to yourself by commenting "/assign" on this issue.

For more information on our triage process please visit our triage overview

sk593 added a commit that referenced this issue May 28, 2024
Reverts #7629

This was fixed as per the convo in
kubernetes-sigs/controller-runtime#2838. It
looks like the binaries weren't released for the version of Kubernetes
that we were using (they had up until 1.24 but we were using 1.23)

Fixes: #7630

---------

Signed-off-by: sk593 <shruthikumar@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something is broken or not working as expected triaged This issue has been reviewed and triaged
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants