Skip to content

Commit 0cbc25a

Browse files
committed
Build e2e controller image once
Signed-off-by: Derek Nola <derek.nola@suse.com>
1 parent f7f55ab commit 0cbc25a

1 file changed

Lines changed: 36 additions & 10 deletions

File tree

.github/workflows/release.yml

Lines changed: 36 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ jobs:
4040
- name: Run Unit Tests
4141
run: ./scripts/test
4242

43-
e2e-build:
43+
e2e-build-base:
4444
permissions:
4545
contents: read
4646
runs-on: ubuntu-latest
@@ -52,17 +52,38 @@ jobs:
5252
run: ./scripts/e2e-build
5353

5454
- name: Save Docker image
55-
run: docker save -o nginx-ingress-e2e.tar $(docker images --format "{{.Repository}}:{{.Tag}}" | grep "rancher/nginx")
55+
run: docker save -o nginx-ingress-e2e-base.tar $(docker images --format "{{.Repository}}:{{.Tag}}" | grep "rancher/nginx")
5656

57-
- name: Upload Docker image
57+
- name: Upload Docker images
5858
uses: actions/upload-artifact@v4
5959
with:
60-
name: nginx-ingress-e2e
61-
path: nginx-ingress-e2e.tar
60+
name: nginx-ingress-e2e-base
61+
path: nginx-ingress-e2e-base.tar
6262
retention-days: 1
63+
64+
e2e-build-ctr:
65+
permissions:
66+
contents: read
67+
runs-on: ubuntu-latest
68+
steps:
69+
- name: Check out code
70+
uses: actions/checkout@v4
71+
72+
- name: Build E2E Test Image
73+
run: make -C test/e2e-image image
74+
75+
- name: Save Docker image
76+
run: docker save -o nginx-ingress-e2e-ctr.tar nginx-ingress-controller:e2e
6377

78+
- name: Upload Docker image
79+
uses: actions/upload-artifact@v4
80+
with:
81+
name: nginx-ingress-e2e-ctr
82+
path: nginx-ingress-e2e-ctr.tar
83+
retention-days: 1
84+
6485
e2e:
65-
needs: e2e-build
86+
needs: [e2e-build-base, e2e-build-ctr]
6687
permissions:
6788
contents: read
6889
runs-on: ubuntu-latest
@@ -83,12 +104,17 @@ jobs:
83104
- name: Download Docker image
84105
uses: actions/download-artifact@v4
85106
with:
86-
name: nginx-ingress-e2e
87-
88-
- name: Load Docker image
89-
run: docker load -i nginx-ingress-e2e.tar
107+
pattern: nginx-ingress-e2e-*
108+
merge-multiple: true
109+
110+
- name: Load Docker images
111+
run: |
112+
docker load -i nginx-ingress-e2e-base.tar
113+
docker load -i nginx-ingress-e2e-ctr.tar
90114
91115
- name: Run E2E Tests with combined focus
116+
env:
117+
SKIP_E2E_IMAGE_CREATION: "true"
92118
run: |
93119
# Add brackets around each focus area in the OR pattern
94120
FOCUS_PATTERN=$(echo "${{ matrix.focus }}" | sed -E 's/([^|]+)/\\[\1\\]/g')

0 commit comments

Comments
 (0)