From 4eac550c57c33550a7b5326c38ff47ce5abb542d Mon Sep 17 00:00:00 2001 From: vsoch Date: Sat, 7 Dec 2024 18:45:27 -0700 Subject: [PATCH] bug: context missing from EventsToRegister Problem: the custom scheduler plugin interface now requires a context variable (which is empty/not used) Solution: add context.Context for it. Signed-off-by: vsoch --- sig-scheduler-plugins/pkg/fluence/fluence.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sig-scheduler-plugins/pkg/fluence/fluence.go b/sig-scheduler-plugins/pkg/fluence/fluence.go index 44f0349..1ff1ee1 100644 --- a/sig-scheduler-plugins/pkg/fluence/fluence.go +++ b/sig-scheduler-plugins/pkg/fluence/fluence.go @@ -140,7 +140,7 @@ func (fluence *Fluence) Name() string { // Fluence has added delete, although I wonder if update includes that signal // and it's redundant? -func (fluence *Fluence) EventsToRegister() []framework.ClusterEventWithHint { +func (fluence *Fluence) EventsToRegister(_ context.Context) []framework.ClusterEventWithHint { // To register a custom event, follow the naming convention at: // https://git.k8s.io/kubernetes/pkg/scheduler/eventhandlers.go#L403-L410 podGroupGVK := fmt.Sprintf("podgroups.v1alpha1.%v", scheduling.GroupName)