From 9f23cf5a6e34a4ae985d4487bd2ecad2463a2f53 Mon Sep 17 00:00:00 2001 From: Aaron U'Ren Date: Thu, 21 Sep 2023 12:57:35 -0500 Subject: [PATCH] fix(linux_networking.go): add better error messages --- pkg/controllers/proxy/linux_networking.go | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/pkg/controllers/proxy/linux_networking.go b/pkg/controllers/proxy/linux_networking.go index 522f0cc5..03c12f66 100644 --- a/pkg/controllers/proxy/linux_networking.go +++ b/pkg/controllers/proxy/linux_networking.go @@ -312,7 +312,8 @@ func (ln *linuxNetworking) ipvsAddFWMarkService(svcs []*ipvs.Service, fwMark uin err := ln.ipvsUpdateService(svc) if err != nil { - return nil, err + return nil, fmt.Errorf("failed to update persistence flags for service %s due to %v", + ipvsServiceString(svc), err) } klog.V(2).Infof("Updated persistence/session-affinity for service: %s", ipvsServiceString(svc)) @@ -323,7 +324,8 @@ func (ln *linuxNetworking) ipvsAddFWMarkService(svcs []*ipvs.Service, fwMark uin err := ln.ipvsUpdateService(svc) if err != nil { - return nil, err + return nil, fmt.Errorf("failed to update scheduler flags for service %s due to %v", + ipvsServiceString(svc), err) } klog.V(2).Infof("Updated scheduler flags for service: %s", ipvsServiceString(svc)) } @@ -332,7 +334,8 @@ func (ln *linuxNetworking) ipvsAddFWMarkService(svcs []*ipvs.Service, fwMark uin svc.SchedName = scheduler err := ln.ipvsUpdateService(svc) if err != nil { - return nil, fmt.Errorf("failed to update the scheduler for the service due to %v", err) + return nil, fmt.Errorf("failed to update the scheduler for the service %s due to %v", + ipvsServiceString(svc), err) } klog.V(2).Infof("Updated schedule for the service: %s", ipvsServiceString(svc)) }