Merge pull request #63 from SOF3/monitor-blocking

fix(generator): MonitorWorkload should be called in the background
kubewharf · Jan 7, 2025 · 3003b28 · 3003b28
2 parents f836099 + 5fae79a
commit 3003b28
Showing 1 changed file with 11 additions and 5 deletions.
diff --git a/generator/monitor/monitor.go b/generator/monitor/monitor.go
@@ -86,12 +86,18 @@ var New = component.Declare[Args, Options, Deps, State, util.Empty](
 	component.Lifecycle[Args, Options, Deps, State]{
 		Start: func(ctx context.Context, _ *Args, options *Options, deps *Deps, state *State) error {
 			if *options.Enable {
-				deps.observer.Get().MonitorWorkloads(ctx, util.Empty{}, func() observer.MonitorWorkloads {
-					state.statusMu.Lock()
-					defer state.statusMu.Unlock()
+				go func() {
+					deps.podseidonInformers.Get().WaitForCacheSync(ctx.Done())
 
-					return state.status
-				})
+					if ctx.Err() == nil {
+						deps.observer.Get().MonitorWorkloads(ctx, util.Empty{}, func() observer.MonitorWorkloads {
+							state.statusMu.Lock()
+							defer state.statusMu.Unlock()
+
+							return state.status
+						})
+					}
+				}()
 			}
 
 			return nil