Skip to content

Commit 1961ebf

Browse files
authored
refactor: move handler directories (#1321)
**What problem does this PR solve?**: Moving the handlers and docs around to make it clear when adding new providers and new handlers if they apply to all providers or only those that use kubeadm too bootstrap the Nodes. This better aligns with the changes in #1297 **Which issue(s) this PR fixes**: Fixes # **How Has This Been Tested?**: <!-- Please describe the tests that you ran to verify your changes. Provide output from the tests and any manual steps needed to replicate the tests. --> **Special notes for your reviewer**: <!-- Use this to provide any additional information to the reviewers. This may include: - Best way to review the PR. - Where the author wants the most review attention on. - etc. -->
1 parent 64cab94 commit 1961ebf

File tree

204 files changed

+129
-110
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

204 files changed

+129
-110
lines changed

cmd/main.go

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ import (
4242
"github.com/nutanix-cloud-native/cluster-api-runtime-extensions-nutanix/pkg/handlers/docker"
4343
"github.com/nutanix-cloud-native/cluster-api-runtime-extensions-nutanix/pkg/handlers/eks"
4444
"github.com/nutanix-cloud-native/cluster-api-runtime-extensions-nutanix/pkg/handlers/generic"
45-
"github.com/nutanix-cloud-native/cluster-api-runtime-extensions-nutanix/pkg/handlers/generic/lifecycle"
45+
"github.com/nutanix-cloud-native/cluster-api-runtime-extensions-nutanix/pkg/handlers/lifecycle"
4646
"github.com/nutanix-cloud-native/cluster-api-runtime-extensions-nutanix/pkg/handlers/nutanix"
4747
"github.com/nutanix-cloud-native/cluster-api-runtime-extensions-nutanix/pkg/handlers/options"
4848
"github.com/nutanix-cloud-native/cluster-api-runtime-extensions-nutanix/pkg/webhook/addons"
@@ -109,7 +109,7 @@ func main() {
109109

110110
globalOptions := options.NewGlobalOptions()
111111

112-
genericLifecycleHandlers := lifecycle.New(globalOptions)
112+
lifecycleHandlers := lifecycle.New(globalOptions)
113113

114114
// awsMetaHandlers combines all AWS patch and variable handlers under a single handler.
115115
// It allows to specify configuration under a single variable.
@@ -140,7 +140,7 @@ func main() {
140140
logsv1.AddFlags(logOptions, pflag.CommandLine)
141141
globalOptions.AddFlags(pflag.CommandLine)
142142
runtimeWebhookServerOpts.AddFlags(pflag.CommandLine)
143-
genericLifecycleHandlers.AddFlags(pflag.CommandLine)
143+
lifecycleHandlers.AddFlags(pflag.CommandLine)
144144
awsMetaHandlers.AddFlags(pflag.CommandLine)
145145
dockerMetaHandlers.AddFlags(pflag.CommandLine)
146146
nutanixMetaHandlers.AddFlags(pflag.CommandLine)
@@ -175,7 +175,7 @@ func main() {
175175
}
176176

177177
var allHandlers []handlers.Named
178-
allHandlers = append(allHandlers, genericLifecycleHandlers.AllHandlers(mgr)...)
178+
allHandlers = append(allHandlers, lifecycleHandlers.AllHandlers(mgr)...)
179179
allHandlers = append(allHandlers, awsMetaHandlers.AllHandlers(mgr)...)
180180
allHandlers = append(allHandlers, dockerMetaHandlers.AllHandlers(mgr)...)
181181
allHandlers = append(allHandlers, nutanixMetaHandlers.AllHandlers(mgr)...)
Lines changed: 8 additions & 0 deletions

0 commit comments

Comments
 (0)