Skip to content

Commit

Permalink
Log setup of integrations (#1457)
Browse files Browse the repository at this point in the history
Change-Id: I36b6f54f8e26f70209f353b84d30d6d3f20c4062
  • Loading branch information
alculquicondor authored Dec 15, 2023
1 parent ee627c1 commit a68c958
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions cmd/kueue/main.go
Original file line number Diff line number Diff line change
Expand Up @@ -257,7 +257,6 @@ func setupControllers(mgr ctrl.Manager, cCache *cache.Cache, queues *queue.Manag
}
err := jobframework.ForEachIntegration(func(name string, cb jobframework.IntegrationCallbacks) error {
log := setupLog.WithValues("jobFrameworkName", name)

if isFrameworkEnabled(cfg, name) {
gvk, err := apiutil.GVKForObject(cb.JobType, mgr.GetScheme())
if err != nil {
Expand All @@ -267,7 +266,7 @@ func setupControllers(mgr ctrl.Manager, cCache *cache.Cache, queues *queue.Manag
if !meta.IsNoMatchError(err) {
return err
}
log.Info("No matching API server for job framework, skip to create controller and webhook")
log.Info("No matching API in the server for job framework, skipped setup of controller and webhook")
} else {
if err = cb.NewReconciler(
mgr.GetClient(),
Expand Down Expand Up @@ -296,6 +295,7 @@ func setupControllers(mgr ctrl.Manager, cCache *cache.Cache, queues *queue.Manag
log.Error(err, "Unable to create webhook")
return err
}
log.Info("Set up controller and webhook for job framework")
return nil
}
}
Expand Down

0 comments on commit a68c958

Please sign in to comment.