minor changes

Signed-off-by: Bella Khizgiyaev <bkhizgiy@redhat.com>
kubev2v · Mar 12, 2024 · e10d287 · e10d287
1 parent 7fda24d
commit e10d287
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 7 deletions.
diff --git a/pkg/controller/plan/adapter/ova/kubevirtvmparser.go b/pkg/controller/plan/adapter/ova/kubevirtvmparser.go
@@ -1,6 +1,7 @@
 package ova
 
 import (
+	liberr "github.com/konveyor/forklift-controller/pkg/lib/error"
 	"gopkg.in/yaml.v2"
 )
 
@@ -38,7 +39,7 @@ type EFI struct {
 	SecureBoot bool `yaml:"secureBoot"`
 }
 
-func ReadConfFromYaml(yamlData []byte) (firmware string, err error) {
+func GetFirmwareFromYaml(yamlData []byte) (firmware string, err error) {
 	var vmi VirtualMachineInstance
 	if err = yaml.Unmarshal(yamlData, &vmi); err != nil {
 		return
@@ -52,5 +53,6 @@ func ReadConfFromYaml(yamlData []byte) (firmware string, err error) {
 		firmware = "efi"
 		return
 	}
+	err = liberr.New("Firmware type was not detected")
 	return
 }
diff --git a/pkg/controller/plan/kubevirt.go b/pkg/controller/plan/kubevirt.go
@@ -882,14 +882,12 @@ func (r *KubeVirt) UpdateVmByConvertedConfig(vm *plan.VMStatus, pod *core.Pod, s
 		return
 	}
 
-	vm.Firmware, err = yamlparser.ReadConfFromYaml(vmConf)
+	vm.Firmware, err = yamlparser.GetFirmwareFromYaml(vmConf)
 	if err != nil {
 		r.Log.Error(err, "failed to get firmware configuration")
 	}
 
-	r.Log.Info("Setting the vm firmware",
-		"vm",
-		vm.String())
+	r.Log.Info("Setting the vm firmware ", vm.Firmware, "vmId", vm.ID)
 
 	shutdownURL := fmt.Sprintf("http://%s:8080/shutdown", pod.Status.PodIP)
 	resp, err = http.Post(shutdownURL, "application/json", nil)

diff --git a/virt-v2v/cold/entrypoint.go b/virt-v2v/cold/entrypoint.go
@@ -264,8 +264,8 @@ func vmHandler(w http.ResponseWriter, r *http.Request) {
 
 	err := addFirmwareToYaml(yamlFilePath)
 	if err != nil {
-		fmt.Printf("Error setting yaml file: %v\n", err)
-		http.Error(w, "Error setting yaml file", http.StatusInternalServerError)
+		fmt.Printf("Error modifying vm configuration: %v\n", err)
+		http.Error(w, "Error modifying vm configuration:", http.StatusInternalServerError)
 	}
 
 	yamlData, err := os.ReadFile(yamlFilePath)