1
39%
master: 39%

Ran 13 Jan 2020 06:39PM UTC

Files 447

Run time 29s

Badge

Embed ▾

Committed 13 Jan 2020 06:16PM UTC coverage: 45.903% (-0.02%) from 45.919%

Job # 12599.1

Build Type

push

travis-ci-com

Committed by

aanm

Commit Message

eni: Fix releases of excess IPs

The release of excess IPs has been incorrect due to not taking into account the
max-above-watermark limit in combination with min-allocate. This bug was hidden
in the unit test as min-allocate was set to a value equal to the max IP limit
of the interface which rendered the value of max-above-watermark (4) to never
be taken into account as min-allocate had already maxed out the interface limit.

Fix the calculation of excess IPs to never fall below min-allocate +
max-above-watermark and change the unit tests to cover this scenario.

This fixes a bug where IPs would always be immediately released again if
min-allocate was greater than pre-allocate and the number of used IPs did not
make up for that gap.

Signed-off-by: Thomas Graf <thomas@cilium.io>

Coverage Stats

26640 of 58036 relevant lines covered (45.9%)

1667.28 hits per line

cilium / cilium / 12599 / 1
39%
master: 39%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 12599.1

cilium / cilium / 12599 / 1 39% master: 39%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 12599.1

cilium / cilium / 12599 / 1
39%
master: 39%

README BADGES
x