Open stormshield-rlibaert opened 2 years ago
Cool, I say we try it out and see what breaks! :sunglasses:
Please submit a pull request and I'll try to find some time to get back to my backlog (of yet unpushed fixes) in pimd. It's easier for me to test as well.
Update: nice catch there with the bugfix! :+1: Much facepalming over here right now .... :man_facepalming:
(I still haven't gotten around to debugging the BSR + flood-to-RP-when-RP-changes issue, but this might trigger me to get back to it)
There, tested and merged! Sorry it took me a while to get around to, been busy getting up to speed at a new job.
Hello Joachim,
We had some more issues when testing some configurations with spt-threshold enabled. We tried to understand the mechanisms used to do the switch to SPT and realised a few things. I have a branch with a few modifications:
https://github.com/stormshield-rlibaert/pimd/commit/b8c5442bf4175e26e8f49360ce7b1eb11918c852
To sum up:
try_switch_to_spt
which was done regardless of the provided configurationinit_route
initialization functionThe last one is the most risky to me as I don't have a wide understanding of pimd, so maybe it brings breaking changes. But the reason for this is that the
spt_threshold
timer was set to the default value (defined bySPT_THRESHOLD_DEFAULT_INTERVAL
), as the configuration of this is done throughinit_vifs
.