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

ci/cirrus: use vagrant from hashicorp repo, bump Fedora to 38, bump bats #3848

Merged
merged 3 commits into from
Apr 27, 2023

Conversation

kolyshkin
Copy link
Contributor

@kolyshkin kolyshkin commented Apr 26, 2023

1. Use vagrant from hashicorp repo.

A version of vagrant available from the stock repos (2.2.19) is too old and contains a bug that prevents downloading Fedora 38 image. The error looks like this:

    default: Downloading: https://vagrantcloud.com/fedora/boxes/38-cloud-base/versions/38.20230413.1/providers/libvirt.box

An error occurred while downloading the remote file. The error
message, if any, is reproduced below. Please fix this error and try
again.

HTTP/2 stream 0 was not closed cleanly: PROTOCOL_ERROR (err 1)

Apparently, this is caused by hashicorp/vagrant#12921 (plus a combination of some server-side changes, I guess).

The code in question was added 5 years ago in vagrant 2.1.3 (hashicorp/vagrant@447e58d) , and it was fixed (by hashicorp/vagrant@c7ef689) in vagrant 2.3.1.

As Ubuntu 22.04 comes with vagrant 2.2.19, we hit the issue.

The solution is to use packages from hashicorp repo, which currently has vagrant 2.3.4.
This resolves the problem of downloading the latest Fedora image.

Also, vagrant-libvirt plugin from Ubuntu repos is not working with
vagrant from hashicorp, so switch to using "vagrant plugin install".
The downside it, this takes extra 4 minutes or so in our CI, and I
am not sure how to cache it or speed it up.

2. Bump Fedora to 38.

3. ci: bump bats 1.8.2 -> 1.9.0

As Fedora 38 uses bats 1.9.0, let's switch to this version in other
places.

.cirrus.yml Outdated Show resolved Hide resolved
@kolyshkin
Copy link
Contributor Author

Ahh, the next problem is vagrant-libvirt plugin from ubuntu repo is not recognized by vagrant from the hashicorp repo (hashicorp/vagrant#12657).

@kolyshkin kolyshkin force-pushed the bump-vagrant branch 8 times, most recently from 792b9a0 to c9b6cf1 Compare April 27, 2023 01:37
@kolyshkin kolyshkin changed the title ci/cirrus: use vagrant from hashicorp repo ci/cirrus: use vagrant from hashicorp repo, bump Fedora to 38, bump bats Apr 27, 2023
.cirrus.yml Outdated Show resolved Hide resolved
A version of vagrant available from the stock repos (2.2.19) is too old
and contains a bug that prevents downloading Fedora 38 image (see [1]).

Use packages from hashicorp repo, which currently has vagrant 2.3.4.
This resolves the problem of downloading the latest Fedora image.

Also, vagrant-libvirt plugin from Ubuntu repos is not working with
vagrant from hashicorp, so switch to using "vagrant plugin install".
The downside it, this takes extra 4 minutes or so in our CI, and I
am not sure how to cache it or speed it up.

[1] opencontainers#3835 (comment)

Signed-off-by: Kir Kolyshkin <[email protected]>
Signed-off-by: Kir Kolyshkin <[email protected]>
As Fedora 38 uses bats 1.9.0, let's switch to this version in other
places.

Signed-off-by: Kir Kolyshkin <[email protected]>
@kolyshkin kolyshkin marked this pull request as ready for review April 27, 2023 01:57
@kolyshkin
Copy link
Contributor Author

Filed a bug to Ubuntu: https://bugs.launchpad.net/vagrant/+bug/2017828

@kolyshkin
Copy link
Contributor Author

Do we want to backport it? I'm not entirely sure. @AkihiroSuda if you feel like it, just add a label.

@AkihiroSuda AkihiroSuda added backport/1.1-pr A backport PR to release-1.1 backport/1.1-todo A PR in main branch which needs to be backported to release-1.1 and removed backport/1.1-pr A backport PR to release-1.1 labels Apr 27, 2023
Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@thaJeztah thaJeztah merged commit 39fe1c3 into opencontainers:main Apr 27, 2023
@kolyshkin
Copy link
Contributor Author

Backported to 1.1 in #3878.

@kolyshkin kolyshkin added backport/1.1-done A PR in main branch which has been backported to release-1.1 and removed backport/1.1-todo A PR in main branch which needs to be backported to release-1.1 labels May 22, 2023
@AkihiroSuda
Copy link
Member

Filed a bug to Ubuntu: https://bugs.launchpad.net/vagrant/+bug/2017828

How can we get Ubuntu to look at this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ci backport/1.1-done A PR in main branch which has been backported to release-1.1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants