Skip to content

Commit

Permalink
Apply variants of air-gapped jobs
Browse files Browse the repository at this point in the history
The job definitions only consider globalnet, deploytool, and
extra-toggles matrix entries; ovn, lighthouse etc. are ignored. This
uses extra-toggles for all matrix entries apart from globalnet and
deploytool, and adjusts the "Reclaim space" test accordingly.

ovn jobs still expected to push the networkplugin-syncer image when
air-gapped; that is now obsolete, so this removes it too.

Signed-off-by: Stephen Kitt <[email protected]>
  • Loading branch information
skitt authored and sridhargaddam committed Oct 5, 2023
1 parent 4f4f5cb commit 0329346
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 8 deletions.
8 changes: 3 additions & 5 deletions .github/workflows/testing.yml
Original file line number Diff line number Diff line change
Expand Up @@ -118,10 +118,8 @@ jobs:
- extra-toggles: air-gap
- extra-toggles: air-gap
globalnet: globalnet
- extra-toggles: air-gap
lighthouse: lighthouse
- extra-toggles: air-gap
ovn: ovn
- extra-toggles: 'air-gap, lighthouse'
- extra-toggles: 'air-gap, ovn'
- extra-toggles: dual-stack
- extra-toggles: ovn
- deploytool: operator
Expand All @@ -132,7 +130,7 @@ jobs:
- extra-toggles: prometheus
steps:
- name: Reclaim space on GHA host (if the job needs it)
if: ${{ matrix.ovn != '' }}
if: ${{ contains('ovn', matrix.extra-toggles) }}
run: rm -rf /usr/share/dotnet

- name: Check out the repository
Expand Down
3 changes: 0 additions & 3 deletions Makefile.inc
Original file line number Diff line number Diff line change
Expand Up @@ -146,9 +146,6 @@ override PRELOAD_IMAGES = $(EXTRA_PRELOAD_IMAGES) nettest \
ifeq ($(GLOBALNET),true)
override PRELOAD_IMAGES += submariner-globalnet
endif
ifneq (,$(shell grep -w ovn $(SETTINGS)))
override PRELOAD_IMAGES += submariner-networkplugin-syncer
endif
ifeq ($(LIGHTHOUSE),true)
override PRELOAD_IMAGES += lighthouse-agent lighthouse-coredns
endif
Expand Down

0 comments on commit 0329346

Please sign in to comment.