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

- `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]>
lima-vm · Oct 16, 2024 · 559edd0 · 559edd0
1 parent 64c3c41
commit 559edd0
Show file tree

Hide file tree

Showing 11 changed files with 46 additions and 171 deletions.
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -217,18 +217,17 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
+        # Most templates use 9p as the mount type
         template:
         - alpine.yaml
-        - debian.yaml
+        - debian.yaml  # reverse-sshfs
         - fedora.yaml
         # cloud-init 24.3.1-1 package has a regression: https://github.com/lima-vm/lima/issues/2714
         # - archlinux.yaml
         - opensuse.yaml
-        - experimental/net-user-v2.yaml
-        - experimental/9p.yaml
         - docker.yaml
         - ../hack/test-templates/alpine-iso-9p-writable.yaml  # Covers alpine-iso.yaml
-        - ../hack/test-templates/test-misc.yaml
+        - ../hack/test-templates/test-misc.yaml  # net-user-v2
     steps:
     - uses: actions/checkout@eef61447b9ff4aafe5dcd4e0bbf5d482be7e7871  # v4.2.1
       with:
@@ -345,13 +344,15 @@ jobs:
       run: make
     - name: Install
       run: make install
+    - name: "Adjust LIMACTL_CREATE_ARGS"
+      run: echo "LIMACTL_CREATE_ARGS=${LIMACTL_CREATE_ARGS} --network=lima:shared" >>$GITHUB_ENV
     - name: "Inject `no_timer_check` to kernel cmdline"
       # workaround to https://github.com/lima-vm/lima/issues/84
-      run: ./hack/inject-cmdline-to-template.sh templates/vmnet.yaml no_timer_check
-    - name: Cache image used by vmnet.yaml
+      run: ./hack/inject-cmdline-to-template.sh templates/default.yaml no_timer_check
+    - name: Cache image used by default .yaml
       uses: ./.github/actions/setup_cache_for_template
       with:
-        template: templates/vmnet.yaml
+        template: templates/default.yaml
     - name: Install test dependencies
       run: brew install qemu bash coreutils iperf3
     - name: Install socket_vmnet
@@ -376,7 +377,7 @@ jobs:
         timeout_minutes: 30
         retry_on: error
         max_attempts: 3
-        command: ./hack/test-templates.sh templates/vmnet.yaml
+        command: ./hack/test-templates.sh templates/default.yaml
     - if: always()
       uses: ./.github/actions/upload_failure_logs_if_exists
 

diff --git a/cmd/limactl/start.go b/cmd/limactl/start.go
@@ -132,10 +132,20 @@ func loadOrCreateInstance(cmd *cobra.Command, args []string, createOnly bool) (*
 		templateName := filepath.Join(templateURL.Host, templateURL.Path)
 		logrus.Debugf("interpreting argument %q as a template name %q", arg, templateName)
 		switch templateName {
+		case "experimental/vz":
+			logrus.Warn("template://experimental/vz was merged into the default template in Lima v1.0. See also <https://lima-vm.io/docs/config/vmtype/>.")
 		case "experimental/riscv64":
 			logrus.Warn("template://experimental/riscv64 was merged into the default template in Lima v1.0. Use `limactl create --arch=riscv64 template://default` instead.")
 		case "experimental/armv7l":
 			logrus.Warn("template://experimental/armv7l was merged into the default template in Lima v1.0. Use `limactl create --arch=armv7l template://default` instead.")
+		case "vmnet":
+			logrus.Warn("template://vmnet was removed in Lima v1.0. Use `limactl create --network=lima:shared template://default` instead. See also <https://lima-vm.io/docs/config/network/>.")
+		case "experimental/net-user-v2":
+			logrus.Warn("template://experimental/net-user-v2 was removed in Lima v1.0. Use `limactl create --network=lima:user-v2 template://default` instead. See also <https://lima-vm.io/docs/config/network/>.")
+		case "experimental/9p":
+			logrus.Warn("template://experimental/9p was 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/>.")
+		case "experimental/virtiofs-linux":
+			logrus.Warn("template://experimental/virtiofs-linux was removed in Lima v1.0. Use `limactl create --mount-type=virtiofs template://default` instead. See also <https://lima-vm.io/docs/config/mount/>.")
 		}
 		if st.instName == "" {
 			// e.g., templateName = "deprecated/centos-7" , st.instName = "centos-7"

diff --git a/examples/README.md b/examples/README.md
@@ -49,11 +49,6 @@ Container orchestration:
 - [`experimental/u7s`](./experimental/u7s.yaml): [Usernetes](https://github.com/rootless-containers/usernetes): Rootless Kubernetes
 
 Optional feature enablers:
-- [`vmnet`](./vmnet.yaml): ⭐enable [`vmnet.framework`](../docs/network.md)
-- [`experimental/9p`](./experimental/9p.yaml): [experimental] use 9p mount type
-- [`experimental/virtiofs-linux`](./experimental/9p.yaml): [experimental] use virtiofs mount type for Linux
-- [`experimental/net-user-v2`](./experimental/net-user-v2.yaml): [experimental] user-v2 network
-  to enable VM-to-VM communication without root privilege
 - [`experimental/vnc`](./experimental/vnc.yaml): [experimental] use vnc display and xorg server
 - [`experimental/alsa`](./experimental/alsa.yaml): [experimental] use alsa and default audio device
 
@@ -68,8 +63,13 @@ Lost+found:
 - ~`nomad`~: Removed in Lima v0.17.1, as Nomad is [no longer free software](https://github.com/hashicorp/nomad/commit/b3e30b1dfa185d9437a25830522da47b91f78816)
 - ~`centos-stream-8`~: Remove in Lima v0.23.0, as CentOS Stream 8 reached [EOL](https://blog.centos.org/2023/04/end-dates-are-coming-for-centos-stream-8-and-centos-linux-7/).
 - ~`deprecated/centos-7`~: Remove in Lima v0.23.0, as CentOS 7 reached [EOL](https://blog.centos.org/2023/04/end-dates-are-coming-for-centos-stream-8-and-centos-linux-7/).
+- ~`experimental/vz`~: Merged into the default template in Lima v1.0. See also <https://lima-vm.io/docs/config/vmtype/>.
 - ~`experimental/armv7l`~: Merged into the `default` template in Lima v1.0. Use `limactl create --arch=armv7l template://default`.
 - ~`experimental/riscv64`~: Merged into the `default` template in Lima v1.0. Use `limactl create --arch=riscv64 template://default`.
+- ~`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-linux template://default` instead. See also <https://lima-vm.io/docs/config/mount/>.
 
 ## Tier
 

diff --git a/examples/experimental/9p.yaml b/examples/experimental/9p.yaml
diff --git a/examples/experimental/net-user-v2.yaml b/examples/experimental/net-user-v2.yaml
diff --git a/examples/experimental/virtiofs-linux.yaml b/examples/experimental/virtiofs-linux.yaml
diff --git a/examples/experimental/vz.yaml b/examples/experimental/vz.yaml
diff --git a/examples/vmnet.yaml b/examples/vmnet.yaml
diff --git a/hack/common.inc.sh b/hack/common.inc.sh
@@ -22,3 +22,5 @@ if [[ ${BASH_VERSINFO:-0} -lt 4 ]]; then
 	ERROR "Bash version is too old: ${BASH_VERSION}"
 	exit 1
 fi
+
+: "${LIMA_HOME:=$HOME/.lima}"
diff --git a/hack/test-templates.sh b/hack/test-templates.sh
@@ -56,9 +56,6 @@ case "$NAME" in
 	# ● run-r2b459797f5b04262bfa79984077a65c7.service                                       loaded failed failed    /usr/bin/systemctl start man-db-cache-update
 	CHECKS["systemd-strict"]=
 	;;
-"vmnet")
-	CHECKS["vmnet"]=1
-	;;
 "test-misc")
 	CHECKS["disk"]=1
 	CHECKS["snapshot-online"]="1"
@@ -67,10 +64,6 @@ case "$NAME" in
 	CHECKS["provision-ansible"]="1"
 	CHECKS["param-env-variables"]="1"
 	;;
-"net-user-v2")
-	CHECKS["port-forwards"]=""
-	CHECKS["user-v2"]=1
-	;;
 "docker")
 	CONTAINER_ENGINE="docker"
 	;;
@@ -81,6 +74,22 @@ if limactl ls -q | grep -q "$NAME"; then
 	exit 1
 fi
 
+# Create ${NAME}-tmp to inspect the enabled features.
+# TODO: skip downloading and converting the image here.
+# Probably `limactl create` should have "dry run" mode that just generates `lima.yaml`.
+# shellcheck disable=SC2086
+"${LIMACTL_CREATE[@]}" ${LIMACTL_CREATE_ARGS} --set ".additionalDisks=null" --name="${NAME}-tmp" "$FILE"
+case "$(yq '.networks[].lima' "${LIMA_HOME}/${NAME}-tmp/lima.yaml")" in
+"shared")
+	CHECKS["vmnet"]=1
+	;;
+"user-v2")
+	CHECKS["port-forwards"]=""
+	CHECKS["user-v2"]=1
+	;;
+esac
+limactl rm -f "${NAME}-tmp"
+
 if [[ -n ${CHECKS["port-forwards"]} ]]; then
 	tmpconfig="$HOME/lima-config-tmp"
 	mkdir -p "${tmpconfig}"

diff --git a/hack/test-templates/test-misc.yaml b/hack/test-templates/test-misc.yaml
@@ -1,6 +1,7 @@
 # The test template for testing misc configurations:
 # - disk
 # - snapshots
+# - user-v2 network
 # - (More to come)
 #
 # This template requires Lima v1.0.0-alpha.0 or later.
@@ -58,3 +59,6 @@ probes:
 # $ limactl disk create data --size 10G
 additionalDisks:
 - "data"
+
+networks:
+- lima: user-v2