Skip to content

Commit

Permalink
Fix that CNI plugin itself was only being included if make ut was run (
Browse files Browse the repository at this point in the history
…#939)

Co-authored-by: Shaun Crampton <[email protected]>
  • Loading branch information
lmm and fasaxc authored Aug 27, 2020
1 parent cf3e2e7 commit 5389d4a
Showing 1 changed file with 5 additions and 4 deletions.
9 changes: 5 additions & 4 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down Expand Up @@ -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 \
Expand Down

0 comments on commit 5389d4a

Please sign in to comment.