[improvement] Migrate to new kubebuilder layout for webhooks and cont…

…roller code (#574)

* Migrate to new kubebuilder layout for webhooks and controller code.

* Update dockerfile

* Update envtest suite tests

* Update the tiltfile config for docker image builds

* Update golangci config to exclude webhook test suite

.golangci.yml

@@ -237,7 +237,10 @@ issues:
     - path: cmd/main.go
       linters:
         - gci
-    - path: controller/suite_test.go
+    - path: internal/controller/suite_test.go
+      linters:
+        - gci
+    - path: internal/webhook/v1alpha2/webhook_suite_test.go
       linters:
         - gci
     # Exclude goimports check for controller tests that import both mocktest and ginkgo/gomega as dot imports.

Dockerfile

@@ -16,7 +16,7 @@ RUN go mod download
 COPY cmd/main.go cmd/main.go
 COPY api/ api/
 COPY clients/ clients/
-COPY controller/ controller/
+COPY internal/ internal/
 COPY cloud/ cloud/
 COPY observability/ observability/
 COPY util/ util/

Makefile

@@ -320,7 +320,7 @@ GOWRAP         ?= $(CACHE_BIN)/gowrap
 S5CMD 		   ?= $(CACHE_BIN)/s5cmd
 
 ## Tool Versions
-KUSTOMIZE_VERSION        ?= v5.4.1
+KUSTOMIZE_VERSION        ?= v5.4.3
 CTLPTL_VERSION           ?= v0.8.29
 CLUSTERCTL_VERSION       ?= v1.7.2
 KUBECTL_VERSION          ?= v1.28.0

Tiltfile

@@ -142,7 +142,7 @@ if debug == "true":
     local_resource(
         "capl-compile",
         'GOOS=linux CGO_ENABLED=0 go build -gcflags "-N -l" -ldflags="-X github.com/linode/cluster-api-provider-linode/version.version=$VERSION" -a -o bin/manager ./cmd/main.go',
-        deps=["./main.go", "./start.go", "vendor", "go.mod", "go.sum", "./api",  "./cloud", "./cmd", "./controller",
+        deps=["./main.go", "./start.go", "vendor", "go.mod", "go.sum", "./api",  "./cloud", "./cmd", "./internal",
             "./observability", "./util", "./version",],
         labels=["CAPL"],
     )
@@ -210,7 +210,7 @@ if os.getenv("SKIP_DOCKER_BUILD", "false") != "true" and debug != "true":
         "docker.io/linode/cluster-api-provider-linode",
         context=".",
         only=("Dockerfile", "Makefile", "vendor", "go.mod", "go.sum",
-        "./api", "./clients", "./cloud", "./cmd", "./controller", "./observability", "./util", "./version"),
+        "./api", "./clients", "./cloud", "./cmd", "./internal", "./observability", "./util", "./version"),
         build_args={"VERSION": os.getenv("VERSION", "")},
     )