Skip to content

Commit 6b403d9

Browse files
committed
fix(applications): wire newArgocdClientFn in Application controller connector
Setup() built the connector without setting newArgocdClientFn (unlike every other controller, e.g. projects), leaving it nil. Connect() -> NewExternal() then invokes that nil function pointer, which panics. controller-runtime recovers the panic and requeues with backoff instead of crashing, so the Application managed resource silently never reconciles: no error is logged, no event is recorded, and .status is never populated, even though the underlying ArgoCD Application is healthy.
1 parent 001217a commit 6b403d9

1 file changed

Lines changed: 2 additions & 1 deletion

File tree

pkg/controller/cluster/applications/controller.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,8 @@ const (
5555
// Setup adds a controller that reconciles applications.
5656
func Setup(mgr ctrl.Manager, o xpcontroller.Options) error {
5757
return SetupWithExternalConnector(mgr, o, &connector{
58-
kube: mgr.GetClient(),
58+
kube: mgr.GetClient(),
59+
newArgocdClientFn: applications.NewApplicationServiceClient,
5960
})
6061
}
6162

0 commit comments

Comments
 (0)