Skip to content

Commit

Permalink
Clean up cloudprovider webhook after dropping the gardener version ch…
Browse files Browse the repository at this point in the history
…eck (#823)
  • Loading branch information
ialidzhikov committed Aug 26, 2024
1 parent 771df0f commit a08ed29
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 19 deletions.
2 changes: 1 addition & 1 deletion pkg/cmd/options.go
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ func WebhookSwitchOptions(gardenerVersion *string) *webhookcmd.SwitchOptions {
return webhookcmd.NewSwitchOptions(
webhookcmd.Switch(extensioncontrolplanewebhook.WebhookName, controlplanewebhook.AddToManager(gardenerVersion)),
webhookcmd.Switch(extensioncontrolplanewebhook.ExposureWebhookName, controlplaneexposurewebhook.AddToManager),
webhookcmd.Switch(extensionscloudproviderwebhook.WebhookName, cloudproviderwebhook.AddToManager(gardenerVersion)),
webhookcmd.Switch(extensionscloudproviderwebhook.WebhookName, cloudproviderwebhook.AddToManager),
webhookcmd.Switch(infrastructurewebhook.WebhookName, infrastructurewebhook.AddToManager),
)
}
25 changes: 7 additions & 18 deletions pkg/webhook/cloudprovider/add.go
Original file line number Diff line number Diff line change
Expand Up @@ -5,34 +5,23 @@
package cloudprovider

import (
"github.com/Masterminds/semver/v3"
extensionswebhook "github.com/gardener/gardener/extensions/pkg/webhook"
"github.com/gardener/gardener/extensions/pkg/webhook/cloudprovider"
utilruntime "k8s.io/apimachinery/pkg/util/runtime"
"sigs.k8s.io/controller-runtime/pkg/log"
"sigs.k8s.io/controller-runtime/pkg/manager"

"github.com/gardener/gardener-extension-provider-openstack/pkg/openstack"
)

var (
logger = log.Log.WithName("openstack-cloudprovider-webhook")
versionConstraintGreaterEqual142 *semver.Constraints
logger = log.Log.WithName("openstack-cloudprovider-webhook")
)

func init() {
var err error
versionConstraintGreaterEqual142, err = semver.NewConstraint(">= 1.42")
utilruntime.Must(err)
}

// AddToManager creates the cloudprovider webhook and adds it to the manager.
func AddToManager(_ *string) func(mgr manager.Manager) (*extensionswebhook.Webhook, error) {
return func(mgr manager.Manager) (*extensionswebhook.Webhook, error) {
logger.Info("adding webhook to manager")
return cloudprovider.New(mgr, cloudprovider.Args{
Provider: openstack.Type,
Mutator: cloudprovider.NewMutator(mgr, logger, NewEnsurer(mgr, logger)),
})
}
func AddToManager(mgr manager.Manager) (*extensionswebhook.Webhook, error) {
logger.Info("Adding webhook to manager")
return cloudprovider.New(mgr, cloudprovider.Args{
Provider: openstack.Type,
Mutator: cloudprovider.NewMutator(mgr, logger, NewEnsurer(mgr, logger)),
})
}

0 comments on commit a08ed29

Please sign in to comment.