cilium / cilium / 40211

39%

push

travis-ci-com

cilium, lbmap: Use silent delete in deleteBackendLocked for now

This should hopefully relieve the large amount of CI flakes we've been seeing
recently, given the NAT46/64 work attempts to delete backends from both maps:

  [...]
  controller bpf-map-sync-cilium_lb6_backends_v2 failure '6 map sync errors'
  controller bpf-map-sync-cilium_lb4_backends_v2 failure '2 map sync errors'
  controller bpf-map-sync-cilium_lb6_backends_v2 failure '5 map sync errors'
  controller bpf-map-sync-cilium_lb4_backends_v2 failure '2 map sync errors'
  [...]

Use silent delete for now given we ignore these errors, so there is also zero
point in attempting to retry a delete if it failed previously anyway given the
element was not present.

Later, we'll refactor the deletion handling, so that we only delete in both
maps when we know NAT46/64 was involved. The patch here doesn't make things
better or worse, it just tells the controller to uselessly retry on missing
element.

Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>

2 of 2 new or added lines in 1 file covered. (100.0%)

46128 of 135281 relevant lines covered (34.1%)

6517.76 hits per line