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

Skip jq install if possible #53

Merged
merged 4 commits into from
Feb 24, 2025
Merged

Skip jq install if possible #53

merged 4 commits into from
Feb 24, 2025

Conversation

nirs
Copy link
Owner

@nirs nirs commented Feb 24, 2025

This minimize the time to start the VM in the CI, making it easier to debug startup issue, when we don't have ssh access to the guest. The time moves to the next step preparing the VM, but it is easier to debug.

nirs added 2 commits February 24, 2025 03:53
Instead of passing 4 arguments trough multiple functions.
Instead of 2 parameters.
This minimize the time to start the VM in the CI, making it easier to
debug startup issue, when we don't have ssh access to the guest. The
time moves to the next step preparing the VM, but it is easier to debug
that step with ssh access.

Before:
- start vm: 2m15s
- prepare vm: 15s

After:
- start vm: 1m33s
- prepare vm: 1m1s
@nirs nirs added this to the v0.5.0 milestone Feb 24, 2025
In the CI this completes now in 1m30s, so 3m is long enough to avoid
flaky tests. Locally this takes less than 10 seconds.
@nirs nirs merged commit d59889f into main Feb 24, 2025
5 checks passed
@nirs nirs deleted the skip-jq-install branch February 24, 2025 02:20
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.

1 participant