Skip to content

Commit

Permalink
Make namespace configurable for tink manifest (#624)
Browse files Browse the repository at this point in the history
Signed-off-by: Abhinav Pandey <[email protected]>

## Description
Make namespace configurable for tink manifest

## Why is this needed
This is needed to allow users to build and deploy the tink manifest in different namespaces

## How Has This Been Tested?
ran `make release-manifests` and verified the changes

## How are existing users impacted? What migration steps/scripts do we need?
No existing user impact


## Checklist:

I have:

- [ ] updated the documentation and/or roadmap (if required)
- [ ] added unit or e2e tests
- [ ] provided instructions on how to upgrade
  • Loading branch information
mergify[bot] authored May 23, 2022
2 parents 7b60369 + 660accd commit 9bb6d05
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion kube.mk
Original file line number Diff line number Diff line change
Expand Up @@ -44,14 +44,18 @@ TINK_SERVER_IMAGE ?= quay.io/tinkerbell/tink-server
TINK_CONTROLLER_IMAGE ?= quay.io/tinkerbell/tink-controller
TINK_SERVER_TAG ?= latest
TINK_CONTROLLER_TAG ?= latest
NAMESPACE ?= tink-system

out/release/default/kustomization.yaml: config/default/kustomization.yaml
rm -rf out/
mkdir -p out/
cp -a config/ out/release/

out/release/tink.yaml: bin/kustomize generate-manifests out/release/default/kustomization.yaml
(cd out/release/default && kustomize edit set image server=$(TINK_SERVER_IMAGE):$(TINK_CONTROLLER_TAG) controller=$(TINK_CONTROLLER_IMAGE):$(TINK_CONTROLLER_TAG))
(cd out/release/default && \
kustomize edit set image server=$(TINK_SERVER_IMAGE):$(TINK_CONTROLLER_TAG) controller=$(TINK_CONTROLLER_IMAGE):$(TINK_CONTROLLER_TAG) && \
kustomize edit set namespace $(NAMESPACE) \
)
kustomize build out/release/default -o $@
prettier --write $@

Expand Down

0 comments on commit 9bb6d05

Please sign in to comment.