set image

Author: deitch

.gitignore

@@ -39,3 +39,6 @@ _artifacts
 
 # Goreleaser
 dist
+
+# test ephemeral input
+test/metadata.yaml

Makefile

@@ -37,7 +37,9 @@ E2E_FOCUS := "functional tests"
 TOOLS_DIR := hack/tools
 TOOLS_BIN_DIR := $(TOOLS_DIR)/bin
 BIN_DIR := bin
-TEST_E2E_DIR := test/e2e
+TEST_DIR := test
+TEST_E2E_DIR := $(TEST_DIR)/e2e
+TEST_METADATA_YAML := $(TEST_DIR)/metadata.yaml
 
 # Binaries.
 KUSTOMIZE := $(TOOLS_BIN_DIR)/kustomize
@@ -197,10 +199,14 @@ test: generate fmt vet crds
 	go test ./... -coverprofile cover.out
 
 # Run e2e tests
-e2e:
+.PHONY: e2e $(TEST_METADATA_YAML)
+$(TEST_METADATA_YAML):
+	cat $(METADATA_TEMPLATE) | sed 's/MAJOR/$(VERSION_MAJOR)/g' | sed 's/MINOR/$(VERSION_MINOR)/g' | sed 's/CONTRACT/$(VERSION_CONTRACT)/g' > $@
+
+e2e: $(TEST_METADATA_YAML)
 	# This is the name used inside the component.yaml for the container that runs the manager
 	# The image gets loaded inside kind from ./test/e2e/config/packet-dev.yaml
-	$(E2E_FLAGS) $(MAKE) -C test/e2e run
+	$(E2E_FLAGS) $(MAKE) -C $(TEST_E2E_DIR) run
 
 # Build manager binary
 manager: $(MANAGER)
@@ -222,11 +228,11 @@ uninstall: crds
 # Deploy controller in the configured Kubernetes cluster in ~/.kube/config
 deploy: crds
 	cd config/manager && $(KUSTOMIZE) edit set image controller=${IMG}
-	$(KUSTOMIZE) build config/default | kubectl apply -f -
+	$(KUSTOMIZE) build config/release | kubectl apply -f -
 
 # Generate manifests e.g. CRD, RBAC etc.
 crds: controller-gen
-	$(CONTROLLER_GEN) $(CRD_OPTIONS) rbac:roleName=manager-role webhook paths="./..." output:crd:artifacts:config=config/crd/bases
+	$(CONTROLLER_GEN) $(CRD_OPTIONS) rbac:roleName=manager-role webhook paths="./..." output:crd:artifacts:config=config/resources/crd/bases
 
 # Run go fmt against code
 fmt:
@@ -305,15 +311,12 @@ endif
 
 manifest: kustomize semver release-manifests release-clusterctl release-cluster-template
 
-release:
-	goreleaser release --rm-dist --snapshot --skip-publish --debug
-
-release/publish:
-	goreleaser release --rm-dist
-
 release-manifests: semver $(RELEASE_MANIFEST) $(RELEASE_METADATA) $(RELEASE_CLUSTER_TEMPLATE)
-$(RELEASE_MANIFEST): $(RELEASE_DIR) ## Builds the manifests to publish with a release
-	$(KUSTOMIZE) build config/default > $@
+release-version:
+	KUSTOMIZE_ENABLE_ALPHA_COMMANDS=true $(KUSTOMIZE) config set config/release image-tag $(VERSION)
+
+$(RELEASE_MANIFEST): $(RELEASE_DIR) release-version ## Builds the manifests to publish with a release
+	$(KUSTOMIZE) build config/release > $@
 
 $(RELEASE_METADATA): semver $(RELEASE_DIR) $(METADATA_TEMPLATE)
 	cat $(METADATA_TEMPLATE) | sed 's/MAJOR/$(VERSION_MAJOR)/g' | sed 's/MINOR/$(VERSION_MINOR)/g' | sed 's/CONTRACT/$(VERSION_CONTRACT)/g' > $@

README.md

@@ -9,10 +9,10 @@ namespace: cluster-api-provider-packet-system
 namePrefix: cluster-api-provider-packet-
 
 # Labels to add to all resources and selectors.
-#commonLabels:
-#  someName: someValue
+commonLabels:
+  cluster.x-k8s.io/provider: infrastructure-packet
 
-bases:
-- ../crd
-- ../rbac
-- ../namespace
+resources:
+- ../resources/crd
+- ../resources/rbac
+- ../resources/namespace

config/default/manager_image_patch.yaml

@@ -1,33 +1,32 @@
 # Adds namespace to all resources.
 namespace: cluster-api-provider-packet-system
-
 # Value of this field is prepended to the
 # names of all resources, e.g. a deployment named
 # "wordpress" becomes "alices-wordpress".
 # Note that it should also match with the prefix (text before '-') of the namespace
 # field above.
 namePrefix: cluster-api-provider-packet-
-
 # Labels to add to all resources and selectors.
 commonLabels:
   cluster.x-k8s.io/provider: infrastructure-packet
-
-bases:
-- ../crd
-- ../rbac
-- ../namespace
-- ../manager
+resources:
+- ../resources/crd
+- ../resources/rbac
+- ../resources/namespace
+- ../resources/manager
 # [WEBHOOK] To enable webhook, uncomment all the sections with [WEBHOOK] prefix including the one in
 # crd/kustomization.yaml
 #- ../webhook
 # [CERTMANAGER] To enable cert-manager, uncomment all sections with 'CERTMANAGER'. 'WEBHOOK' components are required.
 #- ../certmanager
 # [PROMETHEUS] To enable prometheus monitor, uncomment all sections with 'PROMETHEUS'.
 #- ../prometheus
-
+images:
+- name: packet-controller # images with this name
+  newTag: v0.3.0 # {"type":"string","x-kustomize":{"setter":{"name":"image-tag","value":"v0.3.0"}}}
+  newName: packethost/cluster-api-provider-packet # and this name
 patchesStrategicMerge:
-  # Protect the /metrics endpoint by putting it behind auth.
-  # If you want your controller-manager to expose the /metrics
-  # endpoint w/o any authn/z, please comment the following line.
+# Protect the /metrics endpoint by putting it behind auth.
+# If you want your controller-manager to expose the /metrics
+# endpoint w/o any authn/z, please comment the following line.
 - manager_auth_proxy_patch.yaml
-- manager_image_patch.yaml