Merge pull request #3225 from sebsoto/hnsNetworkLog

openshift-merge-bot[bot] · web-flow · commit f4500ad2622f · 2025-08-26T12:17:15.000Z
OCPBUGS-60866: Fix HNS network removal error log level
diff --git a/pkg/windows/windows.go b/pkg/windows/windows.go
@@ -970,7 +970,7 @@ func (vm *windows) ensureHNSNetworksAreRemoved() error {
 		err = wait.PollImmediate(retry.Interval, retry.Timeout, func() (bool, error) {
 			// reinitialize and retry on failure to avoid connection reset SSH errors
 			if err := vm.removeHNSNetwork(network); err != nil {
-				vm.log.V(1).Error(err, "error removing %s HNS network", "network", network)
+				vm.log.V(1).Info("error removing HNS network", "network", network, "err", err.Error())
 				if err := vm.reinitialize(); err != nil {
 					return false, fmt.Errorf("error reinitializing VM after removing %s HNS network: %w", network, err)
 				}
@@ -981,7 +981,7 @@ func (vm *windows) ensureHNSNetworksAreRemoved() error {
 			}
 			out, err := vm.Run(getHNSNetworkCmd(network), true)
 			if err != nil {
-				vm.log.V(1).Error(err, "error waiting for HNS network", "network", network)
+				vm.log.V(1).Info("error waiting for HNS network", "network", network, "err", err.Error())
 				return false, nil
 			}
 			return !strings.Contains(out, network), nil

Original file line number	Diff line number	Diff line change
`@@ -970,7 +970,7 @@ func (vm *windows) ensureHNSNetworksAreRemoved() error {`
`970`	`970`	`err = wait.PollImmediate(retry.Interval, retry.Timeout, func() (bool, error) {`
`971`	`971`	`// reinitialize and retry on failure to avoid connection reset SSH errors`
`972`	`972`	`if err := vm.removeHNSNetwork(network); err != nil {`
`973`		`- vm.log.V(1).Error(err, "error removing %s HNS network", "network", network)`
	`973`	`+ vm.log.V(1).Info("error removing HNS network", "network", network, "err", err.Error())`
`974`	`974`	`if err := vm.reinitialize(); err != nil {`
`975`	`975`	`return false, fmt.Errorf("error reinitializing VM after removing %s HNS network: %w", network, err)`
`976`	`976`	`}`
`@@ -981,7 +981,7 @@ func (vm *windows) ensureHNSNetworksAreRemoved() error {`
`981`	`981`	`}`
`982`	`982`	`out, err := vm.Run(getHNSNetworkCmd(network), true)`
`983`	`983`	`if err != nil {`
`984`		`- vm.log.V(1).Error(err, "error waiting for HNS network", "network", network)`
	`984`	`+ vm.log.V(1).Info("error waiting for HNS network", "network", network, "err", err.Error())`
`985`	`985`	`return false, nil`
`986`	`986`	`}`
`987`	`987`	`return !strings.Contains(out, network), nil`