Skip to content

Commit

Permalink
workflows: ignore "No egress gateway found" drops
Browse files Browse the repository at this point in the history
it's fine to ignore the "No egress gateway found" drop reason as this may be
caused by the kind=echo pods sending traffic while the egressgw policy map is
still being populated.

The actual connectivity test will ensure that the map is in sync with the policy
and that egressgw traffic always go through the correct gateway

Signed-off-by: Gilberto Bertin <jibi@cilium.io>
  • Loading branch information
jibi authored and aanm committed May 16, 2024
1 parent 83e5a9e commit d15410d
Show file tree
Hide file tree
Showing 2 changed files with 21 additions and 0 deletions.
7 changes: 7 additions & 0 deletions .github/workflows/conformance-e2e.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -387,6 +387,13 @@ jobs:
# EXTRA+=("--expected-drop-reasons=+Host datapath not ready")
# it's fine to ignore the "No egress gateway found" drop reason as this may be caused by the kind=echo pods
# sending traffic while the egressgw policy map is still being populated.
#
# The actual connectivity test will ensure that the map is in sync with the policy and that egressgw traffic
# always go through the correct gateway
EXTRA+=("--expected-drop-reasons=+No egress gateway found")
./cilium-cli connectivity test --include-unsafe-tests --collect-sysdump-on-failure \
"${EXTRA[@]}" \
--sysdump-hubble-flows-count=1000000 --sysdump-hubble-flows-timeout=5m \
Expand Down
14 changes: 14 additions & 0 deletions .github/workflows/tests-e2e-upgrade.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -444,6 +444,13 @@ jobs:
EXTRA="--secondary-network-iface=eth1"
fi
# it's fine to ignore the "No egress gateway found" drop reason as this may be caused by the kind=echo pods
# sending traffic while the egressgw policy map is still being populated.
#
# The actual connectivity test will ensure that the map is in sync with the policy and that egressgw traffic
# always go through the correct gateway
EXTRA+=("--expected-drop-reasons=+No egress gateway found")
./cilium-cli connectivity test --include-unsafe-tests --collect-sysdump-on-failure \
--include-conn-disrupt-test \
--flush-ct \
Expand Down Expand Up @@ -475,6 +482,13 @@ jobs:
EXTRA="--secondary-network-iface=eth1"
fi
# it's fine to ignore the "No egress gateway found" drop reason as this may be caused by the kind=echo pods
# sending traffic while the egressgw policy map is still being populated.
#
# The actual connectivity test will ensure that the map is in sync with the policy and that egressgw traffic
# always go through the correct gateway
EXTRA+=("--expected-drop-reasons=+No egress gateway found")
./cilium-cli connectivity test --include-unsafe-tests --collect-sysdump-on-failure \
--include-conn-disrupt-test \
--flush-ct \
Expand Down

0 comments on commit d15410d

Please sign in to comment.