Skip to content

Commit 2d09664

Browse files
authored
codeflare operator: drop unused ctx parameter to setup methods (#107)
1 parent c4eeb84 commit 2d09664

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

cmd/main.go

+2-2
Original file line numberDiff line numberDiff line change
@@ -146,9 +146,9 @@ func main() {
146146
<-certsReady
147147
setupLog.Info("Certs ready")
148148
if ptr.Deref(cfg.WebhooksEnabled, false) {
149-
exitOnError(controller.SetupWebhooks(ctx, mgr, cfg.AppWrapper), "unable to configure webhook")
149+
exitOnError(controller.SetupWebhooks(mgr, cfg.AppWrapper), "unable to configure webhook")
150150
}
151-
exitOnError(controller.SetupControllers(ctx, mgr, cfg.AppWrapper), "unable to start controllers")
151+
exitOnError(controller.SetupControllers(mgr, cfg.AppWrapper), "unable to start controllers")
152152
}()
153153

154154
exitOnError(controller.SetupIndexers(ctx, mgr, cfg.AppWrapper), "unable to setup indexers")

pkg/controller/setup.go

+2-2
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ import (
3737
)
3838

3939
// SetupControllers creates and configures all components of the AppWrapper controller
40-
func SetupControllers(ctx context.Context, mgr ctrl.Manager, awConfig *config.AppWrapperConfig) error {
40+
func SetupControllers(mgr ctrl.Manager, awConfig *config.AppWrapperConfig) error {
4141
if awConfig.EnableKueueIntegrations {
4242
if err := workload.WorkloadReconciler(
4343
mgr.GetClient(),
@@ -67,7 +67,7 @@ func SetupControllers(ctx context.Context, mgr ctrl.Manager, awConfig *config.Ap
6767
}
6868

6969
// SetupWebhooks creates and configures the AppWrapper controller's Webhooks
70-
func SetupWebhooks(ctx context.Context, mgr ctrl.Manager, awConfig *config.AppWrapperConfig) error {
70+
func SetupWebhooks(mgr ctrl.Manager, awConfig *config.AppWrapperConfig) error {
7171
if err := (&webhook.AppWrapperWebhook{
7272
Config: awConfig,
7373
}).SetupWebhookWithManager(mgr); err != nil {

0 commit comments

Comments
 (0)