2
39%
master: 39%

Ran 30 Mar 2022 09:56AM UTC

Files 908

Run time 1min

Badge

Embed ▾

Committed 30 Mar 2022 09:07AM UTC coverage: 35.084% (-0.007%) from 35.091%

Job # 39873.2

Build Type

push

travis-ci-com

Committed by

borkmann

Commit Message

routing: Consistent conditions for ip rule creation/deletion

In addition to the inconsistency fixed in the previous commit, there is
another we need to fix. On the deletion of ip rules with the form "to
x.x.x.x/x", we check that we are running in ENI mode. We don't perform
the same check on the creation of those rules. Thus, on AKS and
AlibabaCloud, we could theoretically create rules of one form and
attempt to delete the other form.

In practice, this isn't causing a bug because:
- The CIDR used in the "to x.x.x.x/x" form appears to always be
  0.0.0.0/0 and is therefore equivalent to not having a to CIDR
  specified.
- We don't support equivalent flags to --aws-release-excess-ips on AKS
  and AlibabaCloud and, as detailed in the previous commit, that flag is
  necessary to trigger the bug.

Thus, this commit is fixing an inconsistency in the code that doesn't
have any consequence today.

Signed-off-by: Paul Chaignon <paul@cilium.io>

Coverage Stats

45049 of 128403 relevant lines covered (35.08%)

2138.02 hits per line

cilium / cilium / 39873 / 2
39%
master: 39%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 39873.2

cilium / cilium / 39873 / 2 39% master: 39%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 39873.2

cilium / cilium / 39873 / 2
39%
master: 39%

README BADGES
x