fix: wrap handleCommonOptions error with NetworkNotReadyErrorMsg

behzad-mir · Copilot · behzad-mir · commit c357fdc694f8 · 2026-05-18T15:53:28.000-07:00
Wrap the error returned from handleCommonOptions in newNetworkImpl with
NetworkNotReadyErrorMsg so kubelet treats transient failures (e.g.
ENETUNREACH from route installation) as network-not-ready and retries
the ADD call in ~1 second.

Co-authored-by: Copilot &lt;223556219+Copilot@users.noreply.github.com&gt;
diff --git a/network/network_linux.go b/network/network_linux.go
@@ -116,7 +116,7 @@ func (nm *networkManager) newNetworkImpl(nwInfo *EndpointInfo, extIf *externalIn
 	err := nm.handleCommonOptions(ifName, nwInfo)
 	if err != nil {
 		logger.Error("handleCommonOptions failed with", zap.Error(err))
-		return nil, err
+		return nil, errors.Wrap(err, NetworkNotReadyErrorMsg)
 	}
 
 	// Create the network object.

Original file line number	Diff line number	Diff line change
`@@ -116,7 +116,7 @@ func (nm networkManager) newNetworkImpl(nwInfo EndpointInfo, extIf *externalIn`
`116`	`116`	`err := nm.handleCommonOptions(ifName, nwInfo)`
`117`	`117`	`if err != nil {`
`118`	`118`	`logger.Error("handleCommonOptions failed with", zap.Error(err))`
`119`		`- return nil, err`
	`119`	`+ return nil, errors.Wrap(err, NetworkNotReadyErrorMsg)`
`120`	`120`	`}`
`121`	`121`
`122`	`122`	`// Create the network object.`