Merge pull request #8408 from atwamahmoud/master
Fix CA's `make test-in-docker` command
This commit is contained in:
		
						commit
						50c3d3e7a2
					
				| 
						 | 
				
			
			@ -1,6 +1,10 @@
 | 
			
		|||
ALL_ARCH = amd64 arm64 s390x
 | 
			
		||||
all: $(addprefix build-arch-,$(ALL_ARCH))
 | 
			
		||||
 | 
			
		||||
# TODO: #8127 - Use default analyzers set by `go test` to include `printf` analyzer.
 | 
			
		||||
# Default analyzers that go test runs according to https://github.com/golang/go/blob/52624e533fe52329da5ba6ebb9c37712048168e0/src/cmd/go/internal/test/test.go#L649
 | 
			
		||||
# This doesn't include the `printf` analyzer until cluster-autoscaler libraries are updated.
 | 
			
		||||
GO_TEST_DEFAULT_ANALYZERS?=atomic,bool,buildtags,directive,errorsas,ifaceassert,nilfunc,slog,stringintconv,tests
 | 
			
		||||
TAG?=dev
 | 
			
		||||
FLAGS=
 | 
			
		||||
LDFLAGS?=-s
 | 
			
		||||
| 
						 | 
				
			
			@ -129,7 +133,7 @@ container-arch-%: build-in-docker-arch-% make-image-arch-%
 | 
			
		|||
	@echo "Full in-docker image ${TAG}${FOR_PROVIDER}-$* completed"
 | 
			
		||||
 | 
			
		||||
test-in-docker: clean docker-builder
 | 
			
		||||
	docker run ${RM_FLAG} -v `pwd`:/cluster-autoscaler/:Z autoscaling-builder:latest bash -c 'cd /cluster-autoscaler && go test -race ./... ${TAGS_FLAG}'
 | 
			
		||||
	docker run ${RM_FLAG} -v `pwd`:/cluster-autoscaler/:Z autoscaling-builder:latest bash -c 'cd /cluster-autoscaler && go test -race ./... -vet="${GO_TEST_DEFAULT_ANALYZERS}" ${TAGS_FLAG}'
 | 
			
		||||
 | 
			
		||||
.PHONY: all build test-unit clean format execute-release dev-release docker-builder build-in-docker release generate push-image push-manifest
 | 
			
		||||
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
		Loading…
	
		Reference in New Issue