Skip to content

Commit

Permalink
Merge pull request #294 from kajinamit/webhook-checker
Browse files Browse the repository at this point in the history
Ensure webhook is started
  • Loading branch information
openshift-merge-robot authored Aug 17, 2023
2 parents 836fb65 + 697f9f2 commit 36ff067
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions main.go
Original file line number Diff line number Diff line change
Expand Up @@ -126,19 +126,21 @@ func main() {
glancev1.SetupDefaults()

// Setup webhooks if requested
checker := healthz.Ping
if strings.ToLower(os.Getenv("ENABLE_WEBHOOKS")) != "false" {
if err = (&glancev1.Glance{}).SetupWebhookWithManager(mgr); err != nil {
setupLog.Error(err, "unable to create webhook", "webhook", "Glance")
os.Exit(1)
}
checker = mgr.GetWebhookServer().StartedChecker()
}
//+kubebuilder:scaffold:builder

if err := mgr.AddHealthzCheck("healthz", healthz.Ping); err != nil {
if err := mgr.AddHealthzCheck("healthz", checker); err != nil {
setupLog.Error(err, "unable to set up health check")
os.Exit(1)
}
if err := mgr.AddReadyzCheck("readyz", healthz.Ping); err != nil {
if err := mgr.AddReadyzCheck("readyz", checker); err != nil {
setupLog.Error(err, "unable to set up ready check")
os.Exit(1)
}
Expand Down

0 comments on commit 36ff067

Please sign in to comment.