Stall ns notifications until gws have inited (#426)

arkodg · web-flow · commit 96e7f5495cb8 · 2022-09-26T15:57:05.000-07:00
* Fixes a regression caused by #420 which stalled namespace notifications until the httproute resource were initialized, but the namespaces are required to be populated to built listener context which affects infra IR creation. Signed-off-by: Arko Dasgupta <arko@tetrate.io> Signed-off-by: Arko Dasgupta <arko@tetrate.io>
diff --git a/internal/gatewayapi/runner/runner.go b/internal/gatewayapi/runner/runner.go
@@ -62,7 +62,7 @@ func (r *Runner) subscribeAndTranslate(ctx context.Context) {
 		case <-servicesCh:
 			r.waitUntilAllGAPIInitialized()
 		case <-namespacesCh:
-			r.waitUntilAllGAPIInitialized()
+			r.waitUntilGCAndGatewaysInitialized()
 		}
 		r.Logger.Info("received a notification")
 		// Load all resources required for translation

Original file line number	Diff line number	Diff line change
`@@ -62,7 +62,7 @@ func (r *Runner) subscribeAndTranslate(ctx context.Context) {`
`62`	`62`	`case <-servicesCh:`
`63`	`63`	`r.waitUntilAllGAPIInitialized()`
`64`	`64`	`case <-namespacesCh:`
`65`		`- r.waitUntilAllGAPIInitialized()`
	`65`	`+ r.waitUntilGCAndGatewaysInitialized()`
`66`	`66`	`}`
`67`	`67`	`r.Logger.Info("received a notification")`
`68`	`68`	`// Load all resources required for translation`