Merge pull request #759 from kochetov-dmitrij/logs-for-single-zone

Print logs when adding a node in unmanaged zone
kubernetes · Sep 5, 2024 · 8929def · 8929def
2 parents 1de8477 + cec3bb1
commit 8929def
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/providers/gce/gce.go b/providers/gce/gce.go
@@ -24,6 +24,7 @@ import (
 	"fmt"
 	"io"
 	"runtime"
+	"slices"
 	"strconv"
 	"strings"
 	"sync"
@@ -827,6 +828,9 @@ func (g *Cloud) updateNodeZones(prevNode, newNode *v1.Node) {
 				g.nodeZones[newZone] = sets.NewString()
 			}
 			g.nodeZones[newZone].Insert(newNode.ObjectMeta.Name)
+			if !slices.Contains(g.managedZones, newZone) {
+				klog.Warningf("Initializing node %s in an unmanaged zone %s. Managed zones: %v", newNode.ObjectMeta.Name, newZone, g.managedZones)
+			}
 		}
 	}
 }

diff --git a/providers/gce/gce_instances.go b/providers/gce/gce_instances.go
@@ -739,6 +739,8 @@ func (g *Cloud) getFoundInstanceByNames(names []string) ([]*gceInstance, error)
 
 // Gets the named instance, returning cloudprovider.InstanceNotFound if the instance is not found
 func (g *Cloud) getInstanceByName(name string) (*gceInstance, error) {
+	klog.Infof("Searching node %s in managed zones %v", name, g.managedZones)
+
 	// Avoid changing behaviour when not managing multiple zones
 	for _, zone := range g.managedZones {
 		instance, err := g.getInstanceFromProjectInZoneByName(g.projectID, zone, name)