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

Correct GOOS for get-deps build on MacOS #4068

Merged
merged 1 commit into from
Jul 16, 2024

Conversation

andrewd-zededa
Copy link
Contributor

No description provided.

tools/get-deps/Makefile Outdated Show resolved Hide resolved
tools/get-deps/Makefile Outdated Show resolved Hide resolved
@andrewd-zededa
Copy link
Contributor Author

latest push is a rebase, we'll see how it does

@andrewd-zededa andrewd-zededa marked this pull request as ready for review July 12, 2024 19:53
@andrewd-zededa
Copy link
Contributor Author

@deitch pushed up your requested change for GOOS. I also realized I had not built HV=kubevirt for this branch, and I see that get-deps was looking for the non-existent pkg/kernel due to the external-boot-image containing eve-kernel. For the moment I added a no-op pkg/kernel dep. Let me know what you think. @rene adding you on here for an additional set of eyes on my get-deps workaround.

Fix kubevirt build looking for pkg/kernel dep.

Signed-off-by: Andrew Durbin <andrewd@zededa.com>
@andrewd-zededa
Copy link
Contributor Author

cleaned up spaces at end of lines for yetus

@deitch
Copy link
Contributor

deitch commented Jul 15, 2024

I restarted the failed test, it was a weird AWS issue.

@deitch
Copy link
Contributor

deitch commented Jul 15, 2024

Ugh, I will have to re-approve once the run is successful, as eden tests do not like it until it is.

@deitch
Copy link
Contributor

deitch commented Jul 15, 2024

Eden tests are running now.

@deitch
Copy link
Contributor

deitch commented Jul 16, 2024

There is an eden failure, but it is completely unrelated to this change, so I am ok merging it in. We should resolve that issue, though.

@deitch deitch merged commit 6cff366 into lf-edge:master Jul 16, 2024
35 of 36 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants