Skip to content

[release/1.6] ci: Use Vagrant on ubuntu-latest-4-cores#9332

Merged
samuelkarp merged 1 commit intorelease/1.6from
vagrant-on-linux
Nov 4, 2023
Merged

[release/1.6] ci: Use Vagrant on ubuntu-latest-4-cores#9332
samuelkarp merged 1 commit intorelease/1.6from
vagrant-on-linux

Conversation

@kzys
Copy link
Copy Markdown
Member

@kzys kzys commented Nov 3, 2023

This Vagrant on Mac job has been broken for a while. Instead this change moves Vagrant to larger Linux workers that provide nested virtualization.

Our main branch did something similar (moving to Cirrus CI to larger GitHub-hosted runners) already as #8919.

@kzys kzys marked this pull request as draft November 3, 2023 17:14
@kzys kzys force-pushed the vagrant-on-linux branch 2 times, most recently from ed44294 to 5758377 Compare November 3, 2023 17:27
@samuelkarp samuelkarp changed the title ci: Use Vagrant on ubuntu-latest-4-cores [release/1.6] ci: Use Vagrant on ubuntu-latest-4-cores Nov 3, 2023
@kzys kzys force-pushed the vagrant-on-linux branch 3 times, most recently from 075d15f to b0d695f Compare November 3, 2023 18:30
@samuelkarp
Copy link
Copy Markdown
Member

Opened #9334 for the flake TestIssue9103/should_be_stopped_status_if_init_has_been_killed.

@kzys kzys force-pushed the vagrant-on-linux branch from b0d695f to 7c642aa Compare November 3, 2023 19:34
@estesp
Copy link
Copy Markdown
Member

estesp commented Nov 3, 2023

Wow; 13m for a Vagrant run. That's a huge improvement

@kzys kzys force-pushed the vagrant-on-linux branch from 7c642aa to 0eabbbd Compare November 3, 2023 21:36
This Vagrant on Mac job has been broken for a while. Instead this
change moves Vagrant to larger Linux workers that provide nested
virtualization.

Our main branch did something similar (moving to Cirrus CI to larger
GitHub-hosted runners) already as #8919.

Signed-off-by: Kazuyoshi Kato <[email protected]>
@kzys kzys force-pushed the vagrant-on-linux branch from 0eabbbd to 0985f7a Compare November 3, 2023 22:17
@kzys kzys marked this pull request as ready for review November 3, 2023 22:49
@samuelkarp
Copy link
Copy Markdown
Member

Merging anyway, tracking the CI failure in #9334.

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.

4 participants