Skip to content
This repository has been archived by the owner on Mar 9, 2022. It is now read-only.

Enables unit tests for Travis build; address nits in Makefile; increase lint timeout to 5min #18

Merged
merged 3 commits into from
Apr 25, 2017
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -17,3 +17,4 @@ script:
- make .gitvalidation
- make verify
- make binaries
- make test
17 changes: 10 additions & 7 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,9 @@ EPOCH_TEST_COMMIT ?= f9e02affccd51702191e5312665a16045ffef8ab
PROJECT := github.com/kubernetes-incubator/cri-containerd
BINDIR ?= ${DESTDIR}/usr/local/bin
BUILD_DIR ?= _output
# VERSION is the version of the binary.
# VERSION is derived from the current tag for HEAD plus amends. Version is used
# to set/overide the criContainerdVersion variable in the verison package for
# cri-containerd.
VERSION:=$(shell git describe --tags --dirty)
BUILD_TAGS:= -ldflags '-X $(PROJECT)/pkg/version.criContainerdVersion=$(VERSION)'

Expand All @@ -30,7 +32,7 @@ help:
@echo
@echo " * 'install' - Install binaries to system locations"
@echo " * 'binaries' - Build cri-containerd"
@echo " * 'test' - Test cri-containerd"
@echo " * 'test' - Test cri-containerd"
@echo " * 'clean' - Clean artifacts"
@echo " * 'verify' - Execute the source code verification tools"
@echo " * 'install.tools' - Installs tools used by verify"
Expand Down Expand Up @@ -63,10 +65,10 @@ boiler:

cri-containerd: check-gopath
$(GO) build -o $(BUILD_DIR)/$@ \
$(BUILD_TAGS) \
$(PROJECT)/cmd/cri-containerd
$(BUILD_TAGS) \
$(PROJECT)/cmd/cri-containerd

test:
test:
go test -timeout=1m -v -race ./pkg/... $(BUILD_TAGS)

clean:
Expand All @@ -81,7 +83,8 @@ uninstall:
rm -f $(BINDIR)/cri-containerd

.PHONY: .gitvalidation
# When this is running in travis, it will only check the travis commit range
# When this is running in travis, it will only check the travis commit range.
# When running outside travis, it will check from $(EPOCH_TEST_COMMIT)..HEAD.
.gitvalidation: check-gopath
ifeq ($(TRAVIS),true)
git-validation -q -run DCO,short-subject
Expand All @@ -103,12 +106,12 @@ install.tools: .install.gitvalidation .install.gometalinter
.PHONY: \
binaries \
boiler \
test \
clean \
default \
gofmt \
help \
install \
lint \
test \
uninstall \
version
2 changes: 1 addition & 1 deletion hack/lint.sh
Original file line number Diff line number Diff line change
Expand Up @@ -29,5 +29,5 @@ for d in $(find . -type d -a \( -iwholename './pkg*' -o -iwholename './cmd*' \))
--cyclo-over=60 \
--dupl-threshold=100 \
--tests \
--deadline=30s "${d}"
--deadline=300s "${d}"
done