Skip to content

Commit 5a27ea0

Browse files
committed
vz: make ssh-over-vsock error less scary
Signed-off-by: Akihiro Suda <[email protected]>
1 parent 2f98baa commit 5a27ea0

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed

hack/test-templates.sh

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -339,7 +339,7 @@ if [[ -n ${CHECKS["ssh-over-vsock"]} ]]; then
339339
INFO "Testing .ssh.overVsock=true configuration"
340340
limactl stop "${NAME}"
341341
# Detection of the SSH server on VSOCK may fail; however, a failing log indicates that controlling detection via the environment variable works as expected.
342-
if ! limactl start --set '.ssh.overVsock=true' "${NAME}" 2>&1 | grep -i -E "(started vsock forwarder|Failed to detect SSH server on vsock)"; then
342+
if ! limactl start --set '.ssh.overVsock=true' "${NAME}" 2>&1 | grep -i -E "(started vsock forwarder|SSH server does not seem running on vsock port)"; then
343343
set +x
344344
diagnose "${NAME}"
345345
ERROR ".ssh.overVsock=true did not enable vsock forwarder"
@@ -348,7 +348,7 @@ if [[ -n ${CHECKS["ssh-over-vsock"]} ]]; then
348348
INFO 'Testing .ssh.overVsock=null configuration'
349349
limactl stop "${NAME}"
350350
# Detection of the SSH server on VSOCK may fail; however, a failing log indicates that controlling detection via the environment variable works as expected.
351-
if ! limactl start --set '.ssh.overVsock=null' "${NAME}" 2>&1 | grep -i -E "(started vsock forwarder|Failed to detect SSH server on vsock)"; then
351+
if ! limactl start --set '.ssh.overVsock=null' "${NAME}" 2>&1 | grep -i -E "(started vsock forwarder|SSH server does not seem running on vsock port)"; then
352352
set +x
353353
diagnose "${NAME}"
354354
ERROR ".ssh.overVsock=null did not enable vsock forwarder"

pkg/driver/vz/vm_darwin.go

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -116,7 +116,9 @@ func startVM(ctx context.Context, inst *limatype.Instance, sshLocalPort int) (vm
116116
logrus.Infof("Detected SSH server is listening on the vsock port; changed %s to proxy for the vsock port", hostAddress)
117117
usernetSSHLocalPort = 0 // disable gvisor ssh port forwarding
118118
} else {
119-
logrus.WithError(err).Warn("Failed to detect SSH server on vsock port, falling back to usernet forwarder")
119+
logrus.WithError(err).WithField("hostAddress", hostAddress).
120+
Debugf("Failed to start vsock forwarder (systemd is older than v256?)")
121+
logrus.Info("SSH server does not seem running on vsock port, using usernet forwarder")
120122
}
121123
} else {
122124
logrus.WithError(err).Warn("Failed to wait for the guest SSH server to become available, falling back to usernet forwarder")

0 commit comments

Comments
 (0)