Trigger target update if client list changes

Signed-off-by: Henry Robinson <[email protected]>
slackhq · Feb 3, 2025 · 24ec168 · 24ec168
1 parent 43f0574
commit 24ec168
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/go/vt/vtgateproxy/discovery.go b/go/vt/vtgateproxy/discovery.go
@@ -188,6 +188,7 @@ func (b *JSONGateResolverBuilder) start() error {
 
 	go func() {
 		var parseErr error
+		lastClientIndex := -1
 		for range b.ticker.C {
 			checkFileStat, err := os.Stat(b.jsonPath)
 			if err != nil {
@@ -213,10 +214,13 @@ func (b *JSONGateResolverBuilder) start() error {
 				continue
 			}
 			parseErr = nil
-			if !contentsChanged {
+			curClientIndex := getClientIndex()
+			clientsChanged := (lastClientIndex == curClientIndex)
+			if !contentsChanged && !clientsChanged {
 				parseCount.Add("unchanged", 1)
 				continue
 			}
+			lastClientIndex = curClientIndex
 			parseCount.Add("changed", 1)
 
 			var wg sync.WaitGroup