Skip to content

Commit

Permalink
Avoid warning about OS with vz driver
Browse files Browse the repository at this point in the history
This commit fix lima-vm#1768 by:

1. add `"OS"` to the know list since lima-vm#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: CDFMLR <[email protected]>
  • Loading branch information
cdfmlr committed Aug 31, 2023
1 parent 7977464 commit 324c4a3
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion pkg/vz/vz_driver_darwin.go
Original file line number Diff line number Diff line change
Expand Up @@ -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) {
Expand Down

0 comments on commit 324c4a3

Please sign in to comment.