Merge pull request #859 from Santosh1176/fix-2952/sc

Paulo Gomes · web-flow · commit 3e2de43b548c · 2022-08-11T14:31:30.000+01:00
Enable panic recovery
diff --git a/controllers/bucket_controller.go b/controllers/bucket_controller.go
@@ -241,6 +241,7 @@ func (r *BucketReconciler) SetupWithManagerAndOptions(mgr ctrl.Manager, opts Buc
 		WithOptions(controller.Options{
 			MaxConcurrentReconciles: opts.MaxConcurrentReconciles,
 			RateLimiter:             opts.RateLimiter,
+			RecoverPanic:            true,
 		}).
 		Complete(r)
 }
diff --git a/controllers/gitrepository_controller.go b/controllers/gitrepository_controller.go
@@ -156,6 +156,7 @@ func (r *GitRepositoryReconciler) SetupWithManagerAndOptions(mgr ctrl.Manager, o
 		WithOptions(controller.Options{
 			MaxConcurrentReconciles: opts.MaxConcurrentReconciles,
 			RateLimiter:             opts.RateLimiter,
+			RecoverPanic:            true,
 		}).
 		Complete(r)
 }
diff --git a/controllers/helmchart_controller.go b/controllers/helmchart_controller.go
@@ -176,6 +176,7 @@ func (r *HelmChartReconciler) SetupWithManagerAndOptions(mgr ctrl.Manager, opts
 		WithOptions(controller.Options{
 			MaxConcurrentReconciles: opts.MaxConcurrentReconciles,
 			RateLimiter:             opts.RateLimiter,
+			RecoverPanic:            true,
 		}).
 		Complete(r)
 }
diff --git a/controllers/helmrepository_controller.go b/controllers/helmrepository_controller.go
@@ -142,6 +142,7 @@ func (r *HelmRepositoryReconciler) SetupWithManagerAndOptions(mgr ctrl.Manager,
 		WithOptions(controller.Options{
 			MaxConcurrentReconciles: opts.MaxConcurrentReconciles,
 			RateLimiter:             opts.RateLimiter,
+			RecoverPanic:            true,
 		}).
 		Complete(r)
 }
diff --git a/controllers/helmrepository_controller_oci.go b/controllers/helmrepository_controller_oci.go
@@ -102,6 +102,7 @@ func (r *HelmRepositoryOCIReconciler) SetupWithManagerAndOptions(mgr ctrl.Manage
 		WithOptions(controller.Options{
 			MaxConcurrentReconciles: opts.MaxConcurrentReconciles,
 			RateLimiter:             opts.RateLimiter,
+			RecoverPanic:            true,
 		}).
 		Complete(r)
 }
diff --git a/controllers/ocirepository_controller.go b/controllers/ocirepository_controller.go
@@ -147,6 +147,7 @@ func (r *OCIRepositoryReconciler) SetupWithManagerAndOptions(mgr ctrl.Manager, o
 		WithOptions(controller.Options{
 			MaxConcurrentReconciles: opts.MaxConcurrentReconciles,
 			RateLimiter:             opts.RateLimiter,
+			RecoverPanic:            true,
 		}).
 		Complete(r)
 }

Original file line number	Diff line number	Diff line change
`@@ -241,6 +241,7 @@ func (r *BucketReconciler) SetupWithManagerAndOptions(mgr ctrl.Manager, opts Buc`
`241`	`241`	`WithOptions(controller.Options{`
`242`	`242`	`MaxConcurrentReconciles: opts.MaxConcurrentReconciles,`
`243`	`243`	`RateLimiter: opts.RateLimiter,`
	`244`	`+ RecoverPanic: true,`
`244`	`245`	`}).`
`245`	`246`	`Complete(r)`
`246`	`247`	`}`
Original file line number	Diff line number	Diff line change
`@@ -156,6 +156,7 @@ func (r *GitRepositoryReconciler) SetupWithManagerAndOptions(mgr ctrl.Manager, o`
`156`	`156`	`WithOptions(controller.Options{`
`157`	`157`	`MaxConcurrentReconciles: opts.MaxConcurrentReconciles,`
`158`	`158`	`RateLimiter: opts.RateLimiter,`
	`159`	`+ RecoverPanic: true,`
`159`	`160`	`}).`
`160`	`161`	`Complete(r)`
`161`	`162`	`}`
Original file line number	Diff line number	Diff line change
`@@ -176,6 +176,7 @@ func (r *HelmChartReconciler) SetupWithManagerAndOptions(mgr ctrl.Manager, opts`
`176`	`176`	`WithOptions(controller.Options{`
`177`	`177`	`MaxConcurrentReconciles: opts.MaxConcurrentReconciles,`
`178`	`178`	`RateLimiter: opts.RateLimiter,`
	`179`	`+ RecoverPanic: true,`
`179`	`180`	`}).`
`180`	`181`	`Complete(r)`
`181`	`182`	`}`
Original file line number	Diff line number	Diff line change
`@@ -142,6 +142,7 @@ func (r *HelmRepositoryReconciler) SetupWithManagerAndOptions(mgr ctrl.Manager,`
`142`	`142`	`WithOptions(controller.Options{`
`143`	`143`	`MaxConcurrentReconciles: opts.MaxConcurrentReconciles,`
`144`	`144`	`RateLimiter: opts.RateLimiter,`
	`145`	`+ RecoverPanic: true,`
`145`	`146`	`}).`
`146`	`147`	`Complete(r)`
`147`	`148`	`}`
Original file line number	Diff line number	Diff line change
`@@ -102,6 +102,7 @@ func (r *HelmRepositoryOCIReconciler) SetupWithManagerAndOptions(mgr ctrl.Manage`
`102`	`102`	`WithOptions(controller.Options{`
`103`	`103`	`MaxConcurrentReconciles: opts.MaxConcurrentReconciles,`
`104`	`104`	`RateLimiter: opts.RateLimiter,`
	`105`	`+ RecoverPanic: true,`
`105`	`106`	`}).`
`106`	`107`	`Complete(r)`
`107`	`108`	`}`
Original file line number	Diff line number	Diff line change
`@@ -147,6 +147,7 @@ func (r *OCIRepositoryReconciler) SetupWithManagerAndOptions(mgr ctrl.Manager, o`
`147`	`147`	`WithOptions(controller.Options{`
`148`	`148`	`MaxConcurrentReconciles: opts.MaxConcurrentReconciles,`
`149`	`149`	`RateLimiter: opts.RateLimiter,`
	`150`	`+ RecoverPanic: true,`
`150`	`151`	`}).`
`151`	`152`	`Complete(r)`
`152`	`153`	`}`