Skip to content

Commit

Permalink
connectivity: echo-ingress-l7-via-hostport-with-encryption only runs …
Browse files Browse the repository at this point in the history
…with cilium 1.16.0 or newer.

Fixes: #2670

Suggested-by: Julian Wiedmann <[email protected]>
Signed-off-by: gray <[email protected]>
  • Loading branch information
jschwinger233 authored and sayboras committed Jul 15, 2024
1 parent 2ddeb2f commit 8d27118
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions connectivity/builder/echo_ingress_l7.go
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,8 @@ package builder
import (
_ "embed"

"github.com/cilium/cilium/pkg/versioncheck"

"github.com/cilium/cilium-cli/connectivity/check"
"github.com/cilium/cilium-cli/connectivity/tests"
"github.com/cilium/cilium-cli/utils/features"
Expand Down Expand Up @@ -43,6 +45,9 @@ func (t echoIngressL7) build(ct *check.ConnectivityTest, _ map[string]string) {

newTest("echo-ingress-l7-via-hostport-with-encryption", ct).
WithCondition(func() bool { return !ct.Params().SingleNode }).
WithCondition(func() bool {
return versioncheck.MustCompile(">=1.16.0")(ct.CiliumVersion)
}).
WithFeatureRequirements(
features.RequireEnabled(features.L7Proxy),
// Once https://github.com/cilium/cilium/issues/33168 is fixed, we
Expand Down

0 comments on commit 8d27118

Please sign in to comment.