github/multus-cni
1
0
Fork 1
mirror of https://github.com/k8snetworkplumbingwg/multus-cni.git synced 2025-08-11 13:03:59 +00:00
Code Issues Projects Releases Wiki Activity

adding error checking in network status creation as well

Browse Source
This commit is contained in:
rkamudhan 2018-08-01 15:56:46 +01:00 committed by Kuralamudhan Ramakrishnan
parent 88759d29de
commit 1ad25a890d
1 changed files with 8 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
multus/multus.go
View File

@ -246,7 +246,8 @@ func cmdAdd(args *skel.CmdArgs, exec invoke.Exec, kubeClient k8s.KubeClient) (cn
} }
//create the network status, only in case Multus as kubeconfig //create the network status, only in case Multus as kubeconfig
if n.Kubeconfig != "" && kc.Podnamespace != "kube-system" { if n.Kubeconfig != "" && kc != nil {
if kc.Podnamespace != "kube-system" {
delegateNetStatus, err := types.LoadNetworkStatus(tmpResult, delegate.Conf.Name, delegate.MasterPlugin) delegateNetStatus, err := types.LoadNetworkStatus(tmpResult, delegate.Conf.Name, delegate.MasterPlugin)
if err != nil { if err != nil {
return nil, fmt.Errorf("Multus: Err in setting networks status: %v", err) return nil, fmt.Errorf("Multus: Err in setting networks status: %v", err)
@ -255,6 +256,7 @@ func cmdAdd(args *skel.CmdArgs, exec invoke.Exec, kubeClient k8s.KubeClient) (cn
netStatus = append(netStatus, delegateNetStatus) netStatus = append(netStatus, delegateNetStatus)
} }
} }
}
if err != nil { if err != nil {
// Ignore errors; DEL must be idempotent anyway // Ignore errors; DEL must be idempotent anyway