1
39%
master: 39%

Ran 14 Aug 2019 09:29PM UTC

Files 435

Run time 1min

Badge

Embed ▾

Committed 14 Aug 2019 09:09PM UTC coverage: 44.229% (+0.03%) from 44.201%

Job # 10055.1

Build Type

push

travis-ci-com

Committed by

tgraf

Commit Message

cilium: route mtu not set unless route.Spec set MTU

Calling route.Upsert() with an MTU parameter will only set the MTU on
the route if the MTU of the route spec (first argument to Upsert)
also sets the MTU to some non-zero value. However, this condition is
not met by any route.Upsert() calls. It seems calling code expects
the route to use the MTU param.

At least Encryption code expected this and is causing routes to
violate MTU. And when using vxlan I expected routes to use a lower
route MTU to account for vxlan header but this is not the case.

After this patch the MTU param will be used to set the route MTU if
it is !nil. If users of the API want to explicitly set the MTU then
they need to set the route spec MTU value and nil the MTU param into
route.Upsert().

Fixes: #8883
Signed-off-by: John Fastabend <john.fastabend@gmail.com>

Coverage Stats

25430 of 57496 relevant lines covered (44.23%)

1187.31 hits per line

cilium / cilium / 10055 / 1
39%
master: 39%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 10055.1

cilium / cilium / 10055 / 1 39% master: 39%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 10055.1

cilium / cilium / 10055 / 1
39%
master: 39%

README BADGES
x