Skip to content

Commit

Permalink
qemu: use 9p by default
Browse files Browse the repository at this point in the history
Templates for RHEL-like distros are updated to continue using reverse-sshfs,
as 9p is not available on RHEL-like distros.

Close issue 971

Signed-off-by: Akihiro Suda <[email protected]>
  • Loading branch information
AkihiroSuda committed Oct 26, 2023
1 parent 3ead893 commit 933ec09
Show file tree
Hide file tree
Showing 11 changed files with 22 additions and 10 deletions.
7 changes: 0 additions & 7 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -171,13 +171,6 @@ jobs:
retry_on: error
max_attempts: 3
command: ./hack/test-templates.sh templates/default.yaml
- name: "Test experimental/9p.yaml"
uses: nick-invision/retry@v2
with:
timeout_minutes: 30
retry_on: error
max_attempts: 3
command: ./hack/test-templates.sh templates/experimental/9p.yaml
- name: "Test test-misc.yaml"
uses: nick-invision/retry@v2
with:
Expand Down
2 changes: 2 additions & 0 deletions examples/almalinux-8.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,8 @@ images:
arch: "x86_64"
- location: "https://repo.almalinux.org/almalinux/8/cloud/aarch64/images/AlmaLinux-8-GenericCloud-latest.aarch64.qcow2"
arch: "aarch64"
# 9p is not available
mountType: "reverse-sshfs"
mounts:
- location: "~"
- location: "/tmp/lima"
Expand Down
2 changes: 2 additions & 0 deletions examples/almalinux-9.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,8 @@ images:
arch: "x86_64"
- location: "https://repo.almalinux.org/almalinux/9/cloud/aarch64/images/AlmaLinux-9-GenericCloud-latest.aarch64.qcow2"
arch: "aarch64"
# 9p is not available
mountType: "reverse-sshfs"
mounts:
- location: "~"
- location: "/tmp/lima"
Expand Down
2 changes: 2 additions & 0 deletions examples/centos-stream-8.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,8 @@ images:
arch: "x86_64"
- location: "https://cloud.centos.org/centos/8-stream/aarch64/images/CentOS-Stream-GenericCloud-8-latest.aarch64.qcow2"
arch: "aarch64"
# 9p is not available
mountType: "reverse-sshfs"
mounts:
- location: "~"
- location: "/tmp/lima"
Expand Down
2 changes: 2 additions & 0 deletions examples/centos-stream-9.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,8 @@ images:
arch: "x86_64"
- location: "https://cloud.centos.org/centos/9-stream/aarch64/images/CentOS-Stream-GenericCloud-9-latest.aarch64.qcow2"
arch: "aarch64"
# 9p is not available
mountType: "reverse-sshfs"
mounts:
- location: "~"
- location: "/tmp/lima"
Expand Down
2 changes: 1 addition & 1 deletion examples/default.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -100,7 +100,7 @@ mounts:

# Mount type for above mounts, such as "reverse-sshfs" (from sshocker), "9p" (EXPERIMENTAL (until Lima v1.0), from QEMU’s virtio-9p-pci, aka virtfs),
# or "virtiofs" (EXPERIMENTAL, needs `vmType: vz`)
# 🟢 Builtin default: "reverse-sshfs" (for QEMU), "virtiofs" (for vz)
# 🟢 Builtin default: "9p" (for QEMU, since Lima v1.0), "virtiofs" (for vz)
mountType: null

# Lima disks to attach to the instance. The disks will be accessible from inside the
Expand Down
2 changes: 2 additions & 0 deletions examples/deprecated/centos-7.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,8 @@ images:
containerd:
system: false
user: false
# 9p is not available
mountType: "reverse-sshfs"
mounts:
- location: "~"
- location: "/tmp/lima"
Expand Down
2 changes: 2 additions & 0 deletions examples/oraclelinux-8.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,8 @@ images:
- location: "https://yum.oracle.com/templates/OracleLinux/OL8/u8/aarch64/OL8U8_aarch64-kvm-b42.qcow"
arch: "aarch64"
digest: "sha256:8fe33ebf15780a2d3917a80b3bddabac834f97ac62fc677d35480ebc80cd6db5"
# 9p is not available
mountType: "reverse-sshfs"
mounts:
- location: "~"
- location: "/tmp/lima"
Expand Down
2 changes: 2 additions & 0 deletions examples/oraclelinux-9.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,8 @@ images:
- location: "https://yum.oracle.com/templates/OracleLinux/OL9/u2/aarch64/OL9U2_aarch64-kvm-b39.qcow"
arch: "aarch64"
digest: "sha256:48351bc318b59ab6d54a2f22d11a28891cd54b9ec2fbbcdf83a7c8483b856221"
# 9p is not available
mountType: "reverse-sshfs"
mounts:
- location: "~"
- location: "/tmp/lima"
Expand Down
2 changes: 2 additions & 0 deletions hack/test-templates/test-misc.yaml
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
# The test template for testing misc configurations:
# - disk
# - reverse-sshfs
# - (More to come)
#
# This template requires Lima v0.14.0 or later.
Expand All @@ -18,6 +19,7 @@ images:
- location: "https://cloud-images.ubuntu.com/releases/22.04/release/ubuntu-22.04-server-cloudimg-arm64.img"
arch: "aarch64"

mountType: "reverse-sshfs"
mounts:
- location: "~"
- location: "/tmp/lima"
Expand Down
7 changes: 5 additions & 2 deletions pkg/limayaml/defaults.go
Original file line number Diff line number Diff line change
Expand Up @@ -518,9 +518,12 @@ func FillDefault(y, d, o *LimaYAML, filePath string) {
y.MountType = o.MountType
}
if y.MountType == nil || *y.MountType == "" {
if *y.VMType == VZ {
switch *y.VMType {
case VZ:
y.MountType = pointer.String(VIRTIOFS)
} else {
case QEMU:
y.MountType = pointer.String(NINEP)
default:
y.MountType = pointer.String(REVSSHFS)
}
}
Expand Down

0 comments on commit 933ec09

Please sign in to comment.