Skip to content

Commit 57c79e7

Browse files
KaloyanTanevclaude
andcommitted
Skip http-tests when no beacon endpoint configured
The http-tests job runs live integration tests against a beacon node. It hard -failed on every PR because HTTP_ADDRESS/HTTP_TEST_CONCURRENCY are not set in this fork. Gate the job on HTTP_ADDRESS so it skips cleanly when unconfigured and runs automatically if the variable is ever set; drop the now-redundant validation step. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
1 parent b22b800 commit 57c79e7

1 file changed

Lines changed: 3 additions & 11 deletions

File tree

.github/workflows/http-tests.yml

Lines changed: 3 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,9 @@ on:
77
jobs:
88
test:
99
runs-on: ubuntu-24.04
10+
# These are live integration tests that need a beacon node HTTP endpoint.
11+
# Skip cleanly when it is not configured; runs automatically once it is.
12+
if: vars.HTTP_ADDRESS != ''
1013
steps:
1114
- name: Checkout code
1215
uses: actions/checkout@v4
@@ -16,17 +19,6 @@ jobs:
1619
with:
1720
go-version: '1.25.2'
1821

19-
- name: Validate required variables
20-
run: |
21-
if [ -z "${{ vars.HTTP_ADDRESS }}" ]; then
22-
echo "Error: HTTP_ADDRESS variable is not set in repository variables"
23-
exit 1
24-
fi
25-
if [ -z "${{ vars.HTTP_TEST_CONCURRENCY }}" ]; then
26-
echo "Error: HTTP_TEST_CONCURRENCY variable is not set in repository variables"
27-
exit 1
28-
fi
29-
3022
- name: Run HTTP tests
3123
env:
3224
HTTP_ADDRESS: ${{ vars.HTTP_ADDRESS }}

0 commit comments

Comments
 (0)