Nail/fission-src
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

executor: update gpm.podLister in AddNamespace to fix IsValid nil panic

Browse Source
This commit is contained in:
Naeel
2026-04-27 14:43:10 +03:00
parent 775845b55c
commit 0cae275dbd
1 changed files with 5 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
pkg/executor/executortype/poolmgr/gpm.go
+5
View File
@@ -833,6 +833,11 @@ func (gpm *GenericPoolManager) AddNamespace(ctx context.Context, ns string, mgr
return fmt.Errorf("AddNamespace %s: register informers: %w", ns, err) return fmt.Errorf("AddNamespace %s: register informers: %w", ns, err)
} }
// Also update gpm.podLister so IsValid can resolve pods in this namespace.
// Without this line gpm.podLister[ns] is nil for dynamically-added user namespaces,
// causing a nil-pointer panic in IsValid, which leaves activeRequests permanently stuck at 1.
gpm.podLister[ns] = gpmInformer.Core().V1().Pods().Lister()
// Start the factories — they will begin syncing immediately. // Start the factories — they will begin syncing immediately.
finformer.Start(ctx.Done()) finformer.Start(ctx.Done())
gpmInformer.Start(ctx.Done()) gpmInformer.Start(ctx.Done())