apiextensions: replace panic with error handling in DiscoveryController

omerap12 · omerap12 · commit bc46e3d9b3f5 · 2024-12-19T09:23:24.000+02:00
Signed-off-by: Omer Aplatony &lt;omerap12@gmail.com&gt;
diff --git a/staging/src/k8s.io/apiextensions-apiserver/pkg/apiserver/customresource_discovery_controller.go b/staging/src/k8s.io/apiextensions-apiserver/pkg/apiserver/customresource_discovery_controller.go
@@ -320,7 +320,8 @@ func (c *DiscoveryController) Run(stopCh <-chan struct{}, synchedCh chan<- struc
 			utilruntime.HandleError(fmt.Errorf("timed out waiting for initial discovery sync"))
 			return
 		}
-		panic(fmt.Errorf("unexpected error: %v", err))
+		utilruntime.HandleError(fmt.Errorf("unexpected error: %v", err))
+		return
 	}
 	close(synchedCh)
 

Original file line number	Diff line number	Diff line change
`@@ -320,7 +320,8 @@ func (c *DiscoveryController) Run(stopCh <-chan struct{}, synchedCh chan<- struc`
`320`	`320`	`utilruntime.HandleError(fmt.Errorf("timed out waiting for initial discovery sync"))`
`321`	`321`	`return`
`322`	`322`	`}`
`323`		`- panic(fmt.Errorf("unexpected error: %v", err))`
	`323`	`+ utilruntime.HandleError(fmt.Errorf("unexpected error: %v", err))`
	`324`	`+ return`
`324`	`325`	`}`
`325`	`326`	`close(synchedCh)`
`326`	`327`