From eca5e1ebc623756ad4951b6b4aa567ae6d7b167b Mon Sep 17 00:00:00 2001 From: Gabriel Adrian Samfira Date: Mon, 10 Feb 2025 13:14:22 +0000 Subject: [PATCH] Update event logging Ping events are no longer logged. Unknonw events are now logged as Debug. Signed-off-by: Gabriel Adrian Samfira --- apiserver/controllers/controllers.go | 4 +++- params/github.go | 1 + 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/apiserver/controllers/controllers.go b/apiserver/controllers/controllers.go index 4f7d18c8..d8750a50 100644 --- a/apiserver/controllers/controllers.go +++ b/apiserver/controllers/controllers.go @@ -159,8 +159,10 @@ func (a *APIController) WebhookHandler(w http.ResponseWriter, r *http.Request) { switch event { case runnerParams.WorkflowJobEvent: a.handleWorkflowJobEvent(ctx, w, r) + case runnerParams.PingEvent: + // Ignore ping event. We may want to save the ping in the github entity table in the future. default: - slog.InfoContext(ctx, "ignoring unknown event", "gh_event", util.SanitizeLogEntry(string(event))) + slog.DebugContext(ctx, "ignoring unknown event", "gh_event", util.SanitizeLogEntry(string(event))) } } diff --git a/params/github.go b/params/github.go index fc4b1c59..cb9cecf0 100644 --- a/params/github.go +++ b/params/github.go @@ -22,6 +22,7 @@ const ( // WorkflowJobEvent is the event set in the webhook payload from github // when a workflow_job hook is sent. WorkflowJobEvent Event = "workflow_job" + PingEvent Event = "ping" ) // WorkflowJob holds the payload sent by github when a workload_job is sent.