12599
39%

Ran 13 Jan 2020 06:39PM UTC

Jobs 1

Files 447

Run time 32s

Badge

Embed ▾

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

Build # 12599

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

Jobs

ID	Job ID	Ran	Files	Coverage
1	12599.1	13 Jan 2020 06:39PM UTC	0	45.9	Travis Job 12599.1

Source Files on build 12599

Detailed source file information is not available for this build.

cilium / cilium / 12599
39%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 12599

cilium / cilium / 12599 39%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 12599

cilium / cilium / 12599
39%

README BADGES
x