From 5389d4a80233a01fa2507ab3bc058a7259bdad18 Mon Sep 17 00:00:00 2001 From: Laurence Man Date: Thu, 27 Aug 2020 13:23:39 -0700 Subject: [PATCH] Fix that CNI plugin itself was only being included if make ut was run (#939) Co-authored-by: Shaun Crampton --- Makefile | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/Makefile b/Makefile index 46fd7ddef..d490dead1 100644 --- a/Makefile +++ b/Makefile @@ -75,7 +75,7 @@ update-pins: update-libcalico-pin # Building the binary ############################################################################### BIN=bin/$(ARCH) -build: $(BIN)/install +build: $(BIN)/install $(BIN)/calico $(BIN)/calico-ipam ifeq ($(ARCH),amd64) # Go only supports amd64 for Windows builds. BIN_WIN=bin/windows @@ -207,12 +207,13 @@ sub-tag-images-%: # Unit Tests ############################################################################### ## Run the unit tests. -ut: run-k8s-controller build $(BIN)/host-local - cp $(BIN)/install $(BIN)/calico-ipam - cp $(BIN)/install $(BIN)/calico +ut: run-k8s-controller $(BIN)/install $(BIN)/host-local $(BIN)/calico-ipam $(BIN)/calico $(MAKE) ut-datastore DATASTORE_TYPE=etcdv3 $(MAKE) ut-datastore DATASTORE_TYPE=kubernetes +$(BIN)/calico-ipam $(BIN)/calico: $(BIN)/install + cp "$<" "$@" + ut-datastore: $(LOCAL_BUILD_DEP) # The tests need to run as root docker run --rm -t --privileged --net=host \