Skip to content

Commit 2552034

Browse files
author
Ayaz Salikhov
committed
Some fixes
1 parent c8e7f24 commit 2552034

File tree

2 files changed

+6
-10
lines changed

2 files changed

+6
-10
lines changed

.github/workflows/docker.yml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -66,10 +66,12 @@ jobs:
6666
python -m pip install --upgrade pip
6767
make -C main dev-env
6868
- name: Build Docker Images
69-
run: make -C main build-test-all
69+
run: make -C main build-all
7070
env:
7171
# Full logs for CI build
7272
BUILDKIT_PROGRESS: plain
73+
- name: Test Docker Images
74+
run: make -C main test-all
7375
- name: Clone Wiki
7476
uses: actions/checkout@v2
7577
with:

Makefile

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -35,29 +35,23 @@ export DOCKER_BUILDKIT:=1
3535
help:
3636
@echo "jupyter/docker-stacks"
3737
@echo "====================="
38-
@echo "Replace % with a stack directory name (e.g., make build/minimal-notebook)"
38+
@echo "Replace % with a stack directory name (e.g., make build-cross/minimal-notebook)"
3939
@echo
4040
@grep -E '^[a-zA-Z0-9_%/-]+:.*?## .*$$' $(MAKEFILE_LIST) | sort | awk 'BEGIN {FS = ":.*?## "}; {printf "\033[36m%-30s\033[0m %s\n", $$1, $$2}'
4141

4242
build-x86/%: DARGS?=
43-
build-x86/%: ## build the latest image for a stack
43+
build-x86/%: ## build the latest image for a stack on x86 only
4444
docker buildx build $(DARGS) --rm --force-rm -t $(OWNER)/$(notdir $@):latest ./$(notdir $@) --build-arg OWNER=$(OWNER) --platform "linux/amd64" --push
4545
@echo -n "Built image size: "
4646
@docker images $(OWNER)/$(notdir $@):latest --format "{{.Size}}"
47-
build-test-x86-all: $(foreach I,$(X86_IMAGES), build-x86/$(I) test/$(I) ) ## build and test all stacks
4847

4948
build-cross/%: DARGS?=
5049
build-cross/%: ## build the latest image for a stack on x86 and ARM
5150
docker buildx build $(DARGS) --rm --force-rm -t $(OWNER)/$(notdir $@):latest ./$(notdir $@) --build-arg OWNER=$(OWNER) --platform "linux/amd64,linux/arm64" --push
5251
@echo -n "Built image size: "
5352
@docker images $(OWNER)/$(notdir $@):latest --format "{{.Size}}"
54-
build-test-cross-all: $(foreach I,$(X86_IMAGES), build-x86/$(I) test/$(I) ) ## build and test all stacks
5553

56-
build-all: $(foreach I,$(CROSS_IMAGES), build-cross/$(I) ) $(foreach I,$(X86_IMAGES), build/$(I) ) ## build all stacks
57-
build-test-all: build-all $(foreach I,$(ALL_IMAGES), test/$(I) ) ## build and test all stacks
58-
59-
build-all: $(foreach I,$(ALL_IMAGES), build/$(I) ) ## build all stacks
60-
build-test-all: $(foreach I,$(ALL_IMAGES), build/$(I) test/$(I) ) ## build and test all stacks
54+
build-all: $(foreach I,$(CROSS_IMAGES), build-cross/$(I) ) $(foreach I,$(X86_IMAGES), build-x86/$(I) ) ## build all stacks
6155

6256
check-outdated/%: ## check the outdated conda packages in a stack and produce a report (experimental)
6357
@TEST_IMAGE="$(OWNER)/$(notdir $@)" pytest test/test_outdated.py

0 commit comments

Comments
 (0)