openstack: do not set empty features

ahadas · ahadas · commit 32222f7cf093 · 2023-11-07T18:51:03.000+02:00
The features are taken from the corresponding template in kubevirt, if
exists, we do not want to override them with empty settings.

Signed-off-by: Arik Hadas &lt;ahadas@redhat.com&gt;
diff --git a/pkg/controller/plan/adapter/openstack/builder.go b/pkg/controller/plan/adapter/openstack/builder.go
@@ -391,10 +391,8 @@ func (r *Builder) mapFirmware(vm *model.Workload, object *cnv.VirtualMachineSpec
 	default:
 		bootloader = &cnv.Bootloader{BIOS: &cnv.BIOS{}}
 	}
-	features := &cnv.Features{}
 	firmware := &cnv.Firmware{}
 	firmware.Bootloader = bootloader
-	object.Template.Spec.Domain.Features = features
 	object.Template.Spec.Domain.Firmware = firmware
 }
 

Original file line number	Diff line number	Diff line change
`@@ -391,10 +391,8 @@ func (r Builder) mapFirmware(vm model.Workload, object *cnv.VirtualMachineSpec`
`391`	`391`	`default:`
`392`	`392`	`bootloader = &cnv.Bootloader{BIOS: &cnv.BIOS{}}`
`393`	`393`	`}`
`394`		`- features := &cnv.Features{}`
`395`	`394`	`firmware := &cnv.Firmware{}`
`396`	`395`	`firmware.Bootloader = bootloader`
`397`		`- object.Template.Spec.Domain.Features = features`
`398`	`396`	`object.Template.Spec.Domain.Firmware = firmware`
`399`	`397`	`}`
`400`	`398`