Skip to content

Commit

Permalink
refactor: override controller
Browse files Browse the repository at this point in the history
  • Loading branch information
mrlihanbo committed Jul 26, 2023
1 parent ea75f7c commit 7286f96
Show file tree
Hide file tree
Showing 8 changed files with 316 additions and 297 deletions.
2 changes: 2 additions & 0 deletions cmd/controller-manager/app/controllermanager.go
Original file line number Diff line number Diff line change
Expand Up @@ -39,10 +39,12 @@ const (
FederateControllerName = "federate"
MonitorControllerName = "monitor"
FollowerControllerName = "follower"
OverrideControllerName = "overridepolicy"
)

var knownControllers = map[string]controllermanager.StartControllerFunc{
FederateControllerName: startFederateController,
OverrideControllerName: startOverridePolicyController,
}

var controllersDisabledByDefault = sets.New(MonitorControllerName)
Expand Down
20 changes: 20 additions & 0 deletions cmd/controller-manager/app/core.go
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,7 @@ import (
"github.com/kubewharf/kubeadmiral/pkg/controllermanager"
controllercontext "github.com/kubewharf/kubeadmiral/pkg/controllers/context"
"github.com/kubewharf/kubeadmiral/pkg/controllers/federate"
"github.com/kubewharf/kubeadmiral/pkg/controllers/override"
)

func startFederateController(
Expand All @@ -51,3 +52,22 @@ func startFederateController(

return federateController, nil
}

func startOverridePolicyController(ctx context.Context, controllerCtx *controllercontext.Context) (controllermanager.Controller, error) {
overrideController, err := override.NewOverridePolicyController(
controllerCtx.KubeClientset,
controllerCtx.FedClientset,
controllerCtx.FedInformerFactory,
controllerCtx.InformerManager,
controllerCtx.Metrics,
klog.Background(),
controllerCtx.WorkerCount,
)
if err != nil {
return nil, fmt.Errorf("error creating override controller: %w", err)
}

go overrideController.Run(ctx)

return overrideController, nil
}
Loading

0 comments on commit 7286f96

Please sign in to comment.