mirror of
https://github.com/logos-messaging/logos-delivery.git
synced 2026-07-30 08:23:32 +00:00
ci: run docker e2e subset on fork PRs against a pinned public image
Drop the gate that skipped the docker subset when no PR-built node image is available. Fork PRs lack the quay secrets, so build-docker-image succeeds with an empty output and the subset now falls back to DEFAULT_NWAKU instead of being skipped. Pin DEFAULT_NWAKU to wakuorg/nwaku:v0.38.1. The rolling :latest tag is a daily upstream master build and currently fails to boot with "updateAddressInENR: Public key does not correspond with given private key", which would have made the subset permanently red on fork PRs. s31 passes against v0.38.1 and against the PR-built image. Echo the resolved peer image at the start of the test step. The job name is left static so the check can still be made required.
This commit is contained in:
parent
80550e8429
commit
6105239506
3
.github/workflows/ci.yml
vendored
3
.github/workflows/ci.yml
vendored
@ -217,10 +217,9 @@ jobs:
|
||||
subset: non-docker
|
||||
secrets: inherit
|
||||
|
||||
# Docker subset; skipped when there's no node image (fork PRs lack the quay secrets).
|
||||
# Docker subset; an empty node_image falls back to DEFAULT_NWAKU (fork PRs lack the quay secrets).
|
||||
send-api-e2e-docker-tests:
|
||||
needs: [build, build-docker-image]
|
||||
if: ${{ needs.build-docker-image.outputs.image != '' }}
|
||||
uses: ./.github/workflows/e2e-api-tests.yml
|
||||
with:
|
||||
subset: docker
|
||||
|
||||
4
.github/workflows/e2e-api-tests.yml
vendored
4
.github/workflows/e2e-api-tests.yml
vendored
@ -44,4 +44,6 @@ jobs:
|
||||
|
||||
- name: Run API/e2e tests (${{ inputs.subset }} subset)
|
||||
working-directory: tests-e2e
|
||||
run: pytest tests/wrappers_tests -m "${{ inputs.subset == 'docker' && 'docker_required' || 'not docker_required' }}" --reruns 2
|
||||
run: |
|
||||
echo "docker peer image: ${NODE_2:-public default (DEFAULT_NWAKU)}"
|
||||
pytest tests/wrappers_tests -m "${{ inputs.subset == 'docker' && 'docker_required' || 'not docker_required' }}" --reruns 2
|
||||
|
||||
@ -14,7 +14,7 @@ def get_env_var(var_name, default=None):
|
||||
|
||||
|
||||
# Configuration constants. Need to be upercase to appear in reports
|
||||
DEFAULT_NWAKU = "wakuorg/nwaku:latest"
|
||||
DEFAULT_NWAKU = "wakuorg/nwaku:v0.38.1"
|
||||
STRESS_ENABLED = False
|
||||
USE_WRAPPERS = True
|
||||
NODE_1 = get_env_var("NODE_1", DEFAULT_NWAKU)
|
||||
|
||||
Loading…
x
Reference in New Issue
Block a user