From 1db84376283927e99d4f822191698a8d9b80eb2e Mon Sep 17 00:00:00 2001 From: Leonard Cohnen Date: Tue, 9 Jan 2024 12:33:31 +0100 Subject: [PATCH] strict: retry getting node labels --- daemon/cmd/datapath.go | 28 +++++++++++++++++++--------- 1 file changed, 19 insertions(+), 9 deletions(-) diff --git a/daemon/cmd/datapath.go b/daemon/cmd/datapath.go index abaf648b5c7e1..3b1f209eefb87 100644 --- a/daemon/cmd/datapath.go +++ b/daemon/cmd/datapath.go @@ -46,6 +46,7 @@ import ( "github.com/cilium/cilium/pkg/node" "github.com/cilium/cilium/pkg/option" "github.com/cilium/cilium/pkg/source" + "github.com/cilium/cilium/pkg/time" ) // LocalConfig returns the local configuration of the daemon's nodediscovery. @@ -508,18 +509,27 @@ func setupStrictModeMap(lns *node.LocalNodeStore) error { return fmt.Errorf("unable to parse control plane label selector: %w", err) } - localNode, err := lns.Get(context.Background()) - if err != nil { - return fmt.Errorf("unable to get local node: %w", err) - } - - if sel.Matches(k8sLabels.Set(localNode.Labels)) { - for _, nodeCIDR := range option.Config.EncryptionStrictModeNodeCIDRs { - if err := strictmap.UpdateContext(nodeCIDR, 0, 0, 2379, 2380); err != nil { - return fmt.Errorf("updating strict mode map: %w", err) + allowEtcd := func() { + for i := 0; i < 10; i++ { + localNode, err := lns.Get(context.Background()) + if err != nil { + log.WithError(err).Error("unable to get local node") + } + log.Debugf("local node labels: %v", localNode.Labels) + if sel.Matches(k8sLabels.Set(localNode.Labels)) { + for _, nodeCIDR := range option.Config.EncryptionStrictModeNodeCIDRs { + if err := strictmap.UpdateContext(nodeCIDR, 0, 0, 2379, 2380); err != nil { + log.WithError(err).Fatal("updating strict mode map: %w", err) + } + } + log.Infoln("Added etcd ports to strict mode map") + return } + time.Sleep(2 * time.Second) } + log.Infoln("Didn't add etcd ports to strict mode map") } + go allowEtcd() return nil }