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

templates: remove vz, vmnet, etc. in favor of the limactl CLI args #2736

Merged
merged 1 commit into from
Oct 17, 2024

Conversation

AkihiroSuda
Copy link
Member

@AkihiroSuda AkihiroSuda commented Oct 15, 2024

@jandubois
Copy link
Member

jandubois commented Oct 15, 2024

@AkihiroSuda Did you intend to include all the cidata changes in the last force-push? They seem unrelated to this PR.

Sorry, I was confused by other stuff being merged to master in between.

jandubois
jandubois previously approved these changes Oct 15, 2024
Copy link
Member

@jandubois jandubois left a comment

Choose a reason for hiding this comment

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

Thanks, LGTM

hack/test-templates.sh Outdated Show resolved Hide resolved
@AkihiroSuda

This comment was marked as resolved.

@jandubois
Copy link
Member

Failure:

ERROR: Ansible requires blocking IO on stdin/stdout/stderr. Non-blocking file handles detected: <stderr>

See also ansible/ansible#80108

@AkihiroSuda
Copy link
Member Author

Failure:

ERROR: Ansible requires blocking IO on stdin/stdout/stderr. Non-blocking file handles detected: <stderr>

See also ansible/ansible#80108

What should we do with this?
Is this relevant to merging net-user-v2.yaml into test-misc.yaml ?

@jandubois
Copy link
Member

What should we do with this?
Is this relevant to merging net-user-v2.yaml into test-misc.yaml ?

No, it is not. If re-running the test gets rid of the error, then let's merge this.

@AkihiroSuda
Copy link
Member Author

Green, but took 52 mins (usually 20 mins)

- `experimental/vz`: Merged into the default template in Lima v1.0.
  See also <https://lima-vm.io/docs/config/vmtype/>.
- `vmnet`: Removed in Lima v1.0.
  Use `limactl create --network=lima:shared template://default` instead.
  See also <https://lima-vm.io/docs/config/network/>.
- `experimental/net-user-v2`: Removed in Lima v1.0.
  Use `limactl create --network=lima:user-v2 template://default` instead.
  See also <https://lima-vm.io/docs/config/network/>.
- `experimental/9p`: Removed in Lima v1.0.
  Use `limactl create --vm-type=qemu --mount-type=9p template://default` instead.
  See also <https://lima-vm.io/docs/config/mount/>.
- `experimental/virtiofs-linux`: Removed in Lima v1.0.
  Use `limactl create --mount-type=virtiofs template://default` instead.
  See also <https://lima-vm.io/docs/config/mount/>.

Signed-off-by: Akihiro Suda <[email protected]>
@AkihiroSuda
Copy link
Member Author

Decoupled net-user-v2 from test-misc again

@AkihiroSuda AkihiroSuda merged commit 8370d05 into lima-vm:master Oct 17, 2024
27 checks passed
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.

2 participants