Project

General

Profile

Actions
Copy link

Todo #4858

closed
CB LS

Finish/fix CARP uniqid changes

Todo #4858: Finish/fix CARP uniqid changes

Added by Chris Buechler about 11 years ago. Updated over 10 years ago.

Status:
Resolved
Priority:
High
Assignee:
Luiz Souza
Category:
CARP
Target version:
2.3
Start date:
07/21/2015
Due date:
% Done:

100%

Estimated time:
Plus Target Version:
Release Notes:

Description

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.

JT Updated by Jim Thompson about 11 years ago Actions
Copy link
 #1

  • Assignee set to Chris Buechler

CT Updated by Cullen Trey almost 11 years ago Actions
Copy link
 #2

See Bug #4642 where i uploaded the rc.carpmaster and rc.carpbackup files, which should start or stop OpenVPN only on the necessary CARP IP events.

CB Updated by Chris Buechler almost 11 years ago Actions
Copy link
 #3

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).
https://github.com/pfsense/pfsense/commit/89f171b052fbe72aed654d2a1c3d5a24e9bf9902

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.

JT Updated by Jim Thompson almost 11 years ago Actions
Copy link
 #4

  • Assignee changed from Chris Buechler to Luiz Souza

I'd rather have Luiz do this, and drive things back as close as we can to stock FreeBSD

LS Updated by Luiz Souza almost 11 years ago Actions
Copy link
 #5

  • % Done changed from 0 to 30

JT Updated by Jim Thompson over 10 years ago Actions
Copy link
 #6

  • Status changed from New to Assigned

CB Updated by Chris Buechler over 10 years ago Actions
Copy link
 #7

  • Subject changed from Review CARP uniqid changes to Finish/fix CARP uniqid changes
  • Status changed from Assigned to Feedback
  • % Done changed from 30 to 100

that should be the last of it.

LS Updated by Luiz Souza over 10 years ago Actions
Copy link
 #8

Unfortunately this is not yet fixed, there are numerous issues when dealing with IP aliases and CARP in pfSense services.

I've a more complete fix which I'm going to commit in the next hours.

Thanks!

LS Updated by Luiz Souza over 10 years ago Actions
Copy link
 #9

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!

CB Updated by Chris Buechler over 10 years ago Actions
Copy link
 #10

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.

outside of the regressions in #5884 and #5885, the remainder works now.

Leaving to feedback until I can thoroughly test dynamic DNS on gateway groups with VIPs.

CB Updated by Chris Buechler over 10 years ago Actions
Copy link
 #11

  • Status changed from Feedback to Resolved

Dynamic DNS on gateway groups with VIPs was broken. It's working now including IP alias and CARP VIPs. That should be the last of this, outside things covered in other tickets.

Actions
Copy link

Also available in: Atom