Bug #3967
closedNeed to restore IP aliases on CARP IPs in 2.2
100%
Description
Ticket for what Ermal and I discussed earlier re: IP aliases on CARP. That functionality is effectively gone from 2.2, which will break a ton of things on upgrade. Need to bring back CARP VIPs as an option for the parent interface of IP aliases.
Updated by Chris Buechler about 10 years ago
- Status changed from New to Confirmed
Updated by Ermal Luçi almost 10 years ago
- Status changed from Confirmed to Feedback
Getting back to this.
Apparently i just made conversion code to convert any such aliases to carp on same vhid.
So i think there should be no issue here.
Upgrade code is on function upgrade_105_to_106!
Updated by Chris Buechler almost 10 years ago
- Status changed from Feedback to Confirmed
that does work, but there are issues with that approach. One, you have to remember what VHID you're using on that, and pick the same, which isn't user-friendly at all. Two, the password has to be set the same on both VIPs or it'll break and there are a variety of circumstances with advskew and advbase that are similarly problematic. Three, people are accustomed to the old way in general.
Let's bring it back to where it was, and remove the upgrade code.
Updated by Ermal Luçi almost 10 years ago
- Status changed from Confirmed to Feedback
It should work same as before.
Updated by Ermal Luçi almost 10 years ago
- % Done changed from 0 to 100
Applied in changeset b0d054ca3b314d0ac7dcfd6a5ba30170a71fe63b.
Updated by Chris Buechler almost 10 years ago
- Assignee changed from Ermal Luçi to Chris Buechler
to me for testing
Updated by Chris Buechler almost 10 years ago
- Status changed from Feedback to Confirmed
- Assignee changed from Chris Buechler to Ermal Luçi
this is mostly fixed after my commits earlier, I think just one last piece. ifconfig is missing the interface. You'll end up with:
php-fpm[244]: /firewall_virtual_ip.php: The command '/sbin/ifconfig '' inet '192.168.128.254'/'32' alias vhid 201' returned exit code '1', the output was 'ifconfig: interface does not exist'
The interface is blank it seems because get_real_interface() doesn't return correctly for this circumstance.
Updated by Ermal Luçi almost 10 years ago
Applied in changeset 94115b931349c4c2dbaff080842bcdbe60ed94b7.
Updated by Chris Buechler almost 10 years ago
- Status changed from Feedback to Confirmed
that fixed that issue, one remaining. Config sync no longer syncs those aliases to the secondary.
Updated by Chris Buechler almost 10 years ago
- Status changed from Confirmed to Resolved
that one's fixed now as well. Everything here works now.