Skip to content

CI: add vz #1511

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

Merged
merged 2 commits into from
Jun 6, 2023
Merged

CI: add vz #1511

merged 2 commits into from
Jun 6, 2023

Conversation

AkihiroSuda
Copy link
Member

No description provided.

@AkihiroSuda
Copy link
Member Author

[vz](https://github.com/lima-vm/lima/actions/runs/4799814791/jobs/8541941830?pr=1511#logs)
cancelled 33 minutes ago in 25m 56s

macos-13 infra seems still unstable

@AkihiroSuda AkihiroSuda marked this pull request as draft April 25, 2023 19:02
@AkihiroSuda
Copy link
Member Author

@AkihiroSuda AkihiroSuda force-pushed the ci-vz branch 3 times, most recently from 046da26 to 5824ae4 Compare May 11, 2023 05:14
@AkihiroSuda AkihiroSuda removed this from the v0.16.0 milestone May 22, 2023
@AkihiroSuda AkihiroSuda force-pushed the ci-vz branch 3 times, most recently from dedadbf to 9212566 Compare June 1, 2023 07:36
@balajiv113
Copy link
Member

@AkihiroSuda - Looks like failure is related to number of cpus being 4 by default. The macos-13 runner has 3 cores and looks like its not able to allocate required resources.

I tried with using 1 cpu and the vz flow was fully running
https://github.com/balajiv113/lima/actions/runs/5163763720/jobs/9302272793

@AkihiroSuda
Copy link
Member Author

https://github.com/balajiv113/lima/actions/runs/5163763720/jobs/9302272793

Thanks, let's change the default dynamically:

@AkihiroSuda AkihiroSuda force-pushed the ci-vz branch 2 times, most recently from 9a50011 to 50dc5f0 Compare June 3, 2023 13:36
@balajiv113
Copy link
Member

Even setting to 3 cores didn't work.
Mostly now its related to the resource allocation from the runner only. Nothing much from our end.

@AkihiroSuda AkihiroSuda force-pushed the ci-vz branch 3 times, most recently from 1c91465 to e4e8603 Compare June 3, 2023 15:01
Signed-off-by: Akihiro Suda <[email protected]>
@AkihiroSuda AkihiroSuda marked this pull request as ready for review June 5, 2023 17:40
@AkihiroSuda AkihiroSuda added this to the v0.17.0 milestone Jun 5, 2023
@AkihiroSuda AkihiroSuda requested a review from balajiv113 June 5, 2023 17:41
@AkihiroSuda AkihiroSuda merged commit c0334c5 into lima-vm:master Jun 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants