Finish/fix CARP uniqid changes
The changes made in 89f171b052fbe72aed654d2a1c3d5a24e9bf9902 need review and completion. Need to verify OpenVPN bound to gateway groups with CARP IPs is working as well. Fixed in RELENG_2_2 in 6eb520938c518a958f09db67e5e9eba2dbdc02d2.
Updated by Chris Buechler almost 6 years ago
This is just a mess, for the reasons Phil Davis noted on the commit, and other things it's broken (#5441, maybe #5442, and definitely other issues that don't have bugs at the moment).
It was tagged with #3997 but has nothing to do with that ticket, that doesn't apply to CARP at all. It no longer git reverts after things have moved around, but will probably end up backing this out as I see no benefit to having it, it wasn't finished, and it doesn't work.
Updated by Luiz Souza over 5 years ago
The uniqid changes are now finished. All the serious and known bugs are fixed.
It took me a while to understand the big picture here and deliver a solution that wasn't too intrusive and at the same time provide all the fixes we need. This was indeed incomplete and broken at various places.
One of the most important failures here was the upgrade code that was never committed and so, all the existent setups would fail miserably after the upgrade.
Thanks everyone for the patience!
Updated by Chris Buechler over 5 years ago
I've re-tested every use of CARP and IP aliases including on gateway groups with the exception of dynamic DNS which I haven't had a chance to get to yet.
Leaving to feedback until I can thoroughly test dynamic DNS on gateway groups with VIPs.