Skip to content

Commit 4de0366

Browse files
ci: fix hybrid system test for prod-script changes and k3d install
Three fixes to hybrid_system_test: 1. Exclude glob 'scripts/prod/**/*' did not match top-level files like scripts/prod/common_lib.py under fnmatch (needs a sub-dir), so prod-script-only PRs wrongly triggered this infra-heavy test. Use 'scripts/prod/**'. 2. The trigger args were interpolated unquoted into the run step, so a glob like 'scripts/prod/**' was shell-expanded before reaching check_test_trigger.py. Quote the interpolations. 3. AbsaOSS/k3d-action defaults to k3d v5.4.6, whose release predates the checksums.txt asset that the current k3d install.sh fetches (with --fail), causing a 404 / 'Failed to install k3d'. Pin K3D_VERSION=v5.5.0 (earliest release shipping checksums.txt; smallest bump from the old default). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
1 parent 4cd57f7 commit 4de0366

1 file changed

Lines changed: 10 additions & 5 deletions

File tree

.github/workflows/hybrid_system_test.yaml

Lines changed: 10 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ env:
1616
cluster_name: hybrid-system-test
1717
crate_triggers: "apollo_node,apollo_deployments,apollo_integration_tests"
1818
path_triggers: ".github/workflows/hybrid_system_test.yaml,scripts/**,deployments/sequencer/**,deployments/images/**"
19-
path_triggers_exclude: "scripts/prod/**/*"
19+
path_triggers_exclude: "scripts/prod/**"
2020
pvc_storage_class_name: "premium-rwo"
2121
anvil_port: "8545"
2222
dockerfile_base_path: ./deployments/images/sequencer
@@ -74,10 +74,10 @@ jobs:
7474
OUTPUT_FILE=$(mktemp)
7575
7676
python ./scripts/check_test_trigger.py --output_file $OUTPUT_FILE \
77-
--commit_id ${{ github.event.pull_request.base.sha }} \
78-
--crate_triggers ${{ env.crate_triggers }} \
79-
--path_triggers ${{ env.path_triggers }} \
80-
--path_triggers_exclude ${{ env.path_triggers_exclude }}
77+
--commit_id "${{ github.event.pull_request.base.sha }}" \
78+
--crate_triggers "${{ env.crate_triggers }}" \
79+
--path_triggers "${{ env.path_triggers }}" \
80+
--path_triggers_exclude "${{ env.path_triggers_exclude }}"
8181
8282
should_run=$(cat "$OUTPUT_FILE")
8383
echo "Captured output: $should_run"
@@ -146,6 +146,11 @@ jobs:
146146

147147
- name: Create k3d cluster (Local k8s)
148148
uses: AbsaOSS/k3d-action@v2.4.0
149+
env:
150+
# Pin k3d explicitly: the action's default (v5.4.6) predates the checksums.txt release
151+
# asset that the current k3d install.sh fetches, so its install fails with a 404. v5.5.0
152+
# is the earliest release that ships checksums.txt (smallest bump from the old default).
153+
K3D_VERSION: v5.5.0
149154
with:
150155
# Assumption: only one PR can run per machine at a time.
151156
cluster-name: ${{ env.cluster_name }}

0 commit comments

Comments
 (0)