Update main.go to fix scale to zero functionality (#225)

swilson-fly · web-flow · commit 7a62edea6665 · 2024-04-08T15:11:58.000-05:00
Currently we're checking if the number of connections is greater-or-equal to 1. However the ConnectionCount query that runs doesn't exclude itself from the result set, so it always returns at least one. The >= 1 check was added here: #200. This PR reverts back to the previous `>1 ` check.
diff --git a/cmd/start/main.go b/cmd/start/main.go
@@ -154,7 +154,7 @@ func scaleToZeroWorker(ctx context.Context, node *flypg.Node) error {
 				continue
 			}
 			log.Printf("Current connection count is %d\n", current)
-			if current >= 1 {
+			if current > 1 {
 				continue
 			}
 			return fmt.Errorf("scale to zero condition hit")

Original file line number	Diff line number	Diff line change
`@@ -154,7 +154,7 @@ func scaleToZeroWorker(ctx context.Context, node *flypg.Node) error {`
`154`	`154`	`continue`
`155`	`155`	`}`
`156`	`156`	`log.Printf("Current connection count is %d\n", current)`
`157`		`- if current >= 1 {`
	`157`	`+ if current > 1 {`
`158`	`158`	`continue`
`159`	`159`	`}`
`160`	`160`	`return fmt.Errorf("scale to zero condition hit")`