From 780c798f32d7690cda5e39aa8f4fa9c9d02befbe Mon Sep 17 00:00:00 2001 From: CDFMLR <45259230+cdfmlr@users.noreply.github.com> Date: Thu, 31 Aug 2023 10:54:45 +0800 Subject: [PATCH] Avoid warning about OS with vz driver This commit fix #1768 by: 1. add `"OS"` to the know list since #1735 introduced this field. 2. make the log message more clear: `Ignoring: vmType vz: [...]` -> `Ignoring following configuration due to the use of vmType vz: [...]` Signed-off-by: Yuehchan Chueng --- pkg/vz/vz_driver_darwin.go | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/pkg/vz/vz_driver_darwin.go b/pkg/vz/vz_driver_darwin.go index bca7a9bbe933..d603308c2576 100644 --- a/pkg/vz/vz_driver_darwin.go +++ b/pkg/vz/vz_driver_darwin.go @@ -73,8 +73,9 @@ func (l *LimaVzDriver) Validate() error { "AdditionalDisks", "Audio", "Video", + "OS", ); len(unknown) > 0 { - logrus.Warnf("Ignoring: vmType %s: %+v", *l.Yaml.VMType, unknown) + logrus.Warnf("Ignoring following configuration due to the use of vmType %s: %+v", *l.Yaml.VMType, unknown) } if !limayaml.IsNativeArch(*l.Yaml.Arch) {