-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
[1.1] Fix CI #3729
[1.1] Fix CI #3729
Conversation
ci / test failures are being addressed by #3723 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ci / test failures are being addressed by #3723
Do we need to combine both PRs to get green CI?
Apparently, the Fedora 35 is not supported now, and the image is removed from servers, so vagrant can no longer find it. Bump to Fedora 37 to fix CI. Signed-off-by: Kir Kolyshkin <[email protected]>
Currently only amd64 and arm64v8 tarball have been checked in testdata, while busybox bundle is downloaded on fly, and supports multiple architectures. To enable integration tests for more architectures, the hello world bundle is replaced by busybox one. Signed-off-by: Shengjing Zhu <[email protected]> (cherry picked from commit 66bf371)
Fix issue 3699 Signed-off-by: Akihiro Suda <[email protected]>
Signed-off-by: Tianon Gravi <[email protected]> (cherry picked from commit 6d28928)
Yes, just did that here. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
@AkihiroSuda ptal |
CI fixes for
release-1.1
branch.Seems that the F35 vagrant image was removed, so we need to switch to a supported Fedora version (37). This is roughly the same as backporting Vagrantfile.fedora: upgrade Fedora to 36 #3442 and Vagrantfile.fedora: upgrade Fedora to 37 #3670.
Backports of tests/integration/get-images.sh: fix busybox.tar.xz URL #3700, Explicitly pin busybox and debian downloads #3701 and tests: replace local hello world bundle with busybox bundle #3526 (stolen from [1.1] Fix CI #3723).
Closes: #3723.