Skip to content

Commit 61c65a9

Browse files
e2e: derive composefs image from BINK_NODE_DISK_IMAGE
Instead of hardcoding the composefs image in CI, derive it from BINK_NODE_DISK_IMAGE by appending -composefs. This keeps both variables in sync and requires no CI-specific overrides. Signed-off-by: HarshwardhanPatil07 <harshpat@redhat.com>
1 parent 24b3e57 commit 61c65a9

2 files changed

Lines changed: 3 additions & 3 deletions

File tree

.github/workflows/ci.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -152,7 +152,7 @@ jobs:
152152
run: make gather-bink
153153

154154
- name: Run e2e tests
155-
run: make e2e V=1 BINK_NODE_DISK_IMAGE_COMPOSEFS=ghcr.io/bootc-dev/bink/node:v1.35-fedora-44-disk-composefs
155+
run: make e2e V=1
156156

157157
- name: Upload logs
158158
if: always()

Makefile

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ BINK_CLUSTER_NAME ?= e2e
77
KUBECONFIG_BINK ?= ./kubeconfig-$(BINK_CLUSTER_NAME)
88
ARTIFACTS ?= $(abspath _output/logs)
99
BINK_NODE_DISK_IMAGE ?= ghcr.io/bootc-dev/bink/node:v1.35-fedora-44-disk
10-
BINK_NODE_DISK_IMAGE_COMPOSEFS ?=
10+
BINK_NODE_DISK_IMAGE_COMPOSEFS ?= $(BINK_NODE_DISK_IMAGE)-composefs
1111
BINK_LOCAL_REGISTRY_NODE_IMAGE ?= registry.cluster.local:5000/node
1212
# YEAR defines the year value used for substituting the YEAR placeholder in the boilerplate header.
1313
YEAR ?= $(shell date +%Y)
@@ -66,7 +66,7 @@ e2e: ## Run e2e tests (requires: make deploy-bink). V=1 for verbose. RUN=<regex>
6666
cd test/e2e && KUBECONFIG=$(abspath $(KUBECONFIG_BINK)) BINK_CLUSTER_NAME=$(BINK_CLUSTER_NAME) \
6767
$(if $(BINK_NODE_IMAGE),BINK_NODE_IMAGE=$(BINK_NODE_IMAGE)) \
6868
BINK_NODE_DISK_IMAGE=$(BINK_NODE_DISK_IMAGE) \
69-
$(if $(BINK_NODE_DISK_IMAGE_COMPOSEFS),BINK_NODE_DISK_IMAGE_COMPOSEFS=$(BINK_NODE_DISK_IMAGE_COMPOSEFS)) \
69+
BINK_NODE_DISK_IMAGE_COMPOSEFS=$(BINK_NODE_DISK_IMAGE_COMPOSEFS) \
7070
BINK_LOCAL_REGISTRY_NODE_IMAGE=$(BINK_LOCAL_REGISTRY_NODE_IMAGE) \
7171
ARTIFACTS=$(ARTIFACTS) \
7272
BINK_NODE_IMAGE_DIGEST=$$(skopeo inspect --tls-verify=false --format '{{.Digest}}' docker://localhost:5000/node:latest) \

0 commit comments

Comments
 (0)