Skip to content

inlet/bmp: avoid long lock times when flushing peers#254

Merged
vincentbernat merged 1 commit into
mainfrom
fix/reduce-bmp-lock-time
Nov 9, 2022
Merged

inlet/bmp: avoid long lock times when flushing peers#254
vincentbernat merged 1 commit into
mainfrom
fix/reduce-bmp-lock-time

Conversation

@vincentbernat
Copy link
Copy Markdown
Member

When the RIB is locked for too long, inlet is hung. Try to ensure give a bit of time for the inlet to move forward between two flush of the RIB. There are various knobs not documnted yet until we get better defaults:

  • inlet.bmp.peer-removal-max-time: how long to keep the lock
  • inlet.bmp.peer-removal-sleep-interval: how long to sleep between two runs if we were unable to flush the whole peer
  • inlet.bmp.peer-removal-max-queue: maximum number of flush requests
  • inlet.bmp.peer-removal-min-routes: minimum number of routes to flush before yielding

May fix #253

@vincentbernat vincentbernat force-pushed the fix/reduce-bmp-lock-time branch from 2bc0284 to 47c72d1 Compare November 9, 2022 13:12
When the RIB is locked for too long, inlet is hung. Try to ensure give a
bit of time for the inlet to move forward between two flush of the RIB.
There are various knobs not documnted yet until we get better defaults:

- `inlet.bmp.peer-removal-max-time`: how long to keep the lock
- `inlet.bmp.peer-removal-sleep-interval`: how long to sleep between two
  runs if we were unable to flush the whole peer
- `inlet.bmp.peer-removal-max-queue`: maximum number of flush requests
- `inlet.bmp.peer-removal-min-routes`: minimum number of routes to flush
  before yielding

May fix #253
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Optimize BMP module, notably when removing a peer

1 participant