42598
39%

Ran 03 Jun 2022 09:07PM UTC

Jobs 2

Files 686

Run time 4min

Badge

Embed ▾

Committed 03 Jun 2022 08:30PM UTC coverage: 39.454% (+5.7%) from 33.727%

Build # 42598

Build Type

push

travis-ci-com

Committed by

aanm

Commit Message

fix aws-cni conformance test

Commit 2d7af3a (".github: add support for cilium-cli in aws-cni
conformance tests") changed the AWS-CNI workflow to install Cilium using
the CLI instead of Helm. All of the Helm flags were passed through the
CLI using --helm-set.

However, the CLI also does it's own magic and isn't aware that we want
to install Cilium in chaining mode. Therefore, it detects EKS and
incorrectly sets IPAM mode to ENI.

As a result, Cilium attempts to setup the IP rules and routes for new
pods. It seems to fail in most cases—maybe because AWS-CNI already setup
the state—but sometimes succeeds. When it succeeds, pods end up with an
egress rule pointing to a non-existing ip route table.

This usually surfaces in the workflow runs as a DNS failure: the client
pod fails to egress to the DNS backend on a remote node. In rarer cases,
it surfaces as a failing connectivity test for some of the pods.

This commit fixes it by overwriting some of the helm flags set
automatically by Cilium-cli namely "eni.enabled=false" and
"ipam=cluster-pool".

The longer-term fix would be to support chaining mode in the CLI.

Reported-by: Paul Chaignon <paul@cilium.io>
Signed-off-by: André Martins <andre@cilium.io>

Coverage Stats

37692 of 95534 relevant lines covered (39.45%)

4612.38 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	42598.1	03 Jun 2022 09:10PM UTC	0	39.48	Travis Job 42598.1
2	42598.2	03 Jun 2022 09:07PM UTC	0	39.45	Travis Job 42598.2

Source Files on build 42598

Detailed source file information is not available for this build.

cilium / cilium / 42598
39%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 42598

cilium / cilium / 42598 39%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 42598

cilium / cilium / 42598
39%

README BADGES
x