Skip to content

Commit

Permalink
fix(Makefile):remove certmanager from target run
Browse files Browse the repository at this point in the history
- avoid cert-manager deploymemt conflicts
- add new target run-with-cm to support install cert-manager before calling run

fix #216

Signed-off-by: Steven Zou <szou@vmware.com>
  • Loading branch information
steven-zou committed Nov 27, 2020
1 parent 4fcf39a commit c66fa2a
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -60,9 +60,12 @@ test: go-test go-dependencies-test

# Run against the configured Kubernetes cluster in ~/.kube/config
.PHONY: run
run: go-generate vendor certmanager $(TMPDIR)k8s-webhook-server/serving-certs/tls.crt
run: go-generate vendor $(TMPDIR)k8s-webhook-server/serving-certs/tls.crt
go run *.go

# Install cert-manager before run
run-with-cm: certmanager run

# Run linters against all files
.PHONY: lint
lint: \
Expand Down

0 comments on commit c66fa2a

Please sign in to comment.