Skip to content

Commit

Permalink
.github: include more tests in cilium-integration-tests
Browse files Browse the repository at this point in the history
Include also sni test not having "l7" in it's name, and check the logs
for errors as well.

Test with embedded cilium-envoy so that Envoy error and warning logs will
also be checked for.

Signed-off-by: Jarno Rajahalme <[email protected]>
  • Loading branch information
jrajahalme committed Nov 26, 2024
1 parent a78649b commit 4879439
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion .github/workflows/cilium-integration-tests.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -146,6 +146,7 @@ jobs:
--helm-set operator.image.tag=${{ env.CILIUM_IMAGE_TAG }} \
--helm-set operator.image.useDigest=false \
--helm-set operator.image.pullPolicy=Never \
--helm-set envoy.enabled=false \
--helm-set debug.enabled=true \
--helm-set debug.verbose=envoy
Expand All @@ -155,7 +156,7 @@ jobs:
- name: Execute Cilium L7 Connectivity Tests
shell: bash
run: cilium connectivity test --test=l7
run: cilium connectivity test --test="l7|sni|check-log-errors"

- name: Gather Cilium system dump
if: failure()
Expand Down

0 comments on commit 4879439

Please sign in to comment.