Project

General

Profile

Bug #507

system_routing_configure should not delete and re-add all routes

Added by Chris Buechler over 9 years ago. Updated over 9 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Category:
Routing
Target version:
Start date:
04/13/2010
Due date:
% Done:

0%

Estimated time:
Affected Version:
All
Affected Architecture:

Description

Currently system_routing_configure will blow away all routes and then re-add them. In environments with a lot of static routes, it can take a little time to re-add them. For example with about 50 routes, there will be about 2 seconds where routes are missing between them being deleted and added back. This causes a brief outage for anything that requires static routes when making changes in the web interface, and wiping out all the routes is unnecessary.

Possible solution, rather than deleting all routes, first check to see if the same route still exists, and if it does, don't delete it. i.e. only delete routes that are going to get deleted anyway.

Associated revisions

Revision a529aced (diff)
Added by Ermal Luçi over 9 years ago

Ticket #507. Do not remove all static routes to readd them back. The change will never delete a rule but define the action to be taken on it, either add or change a present route. Be consinstent on gateways allowed to be selected as destinations for static routes, including dynamic gateways.

Revision 07b54e8c (diff)
Added by Seth Mos over 9 years ago

Do not unconditionaly remove the default route and readd it. This should make system_routing_configure more
reliably. Ticket #507
stability

Revision b61154fb (diff)
Added by Seth Mos over 9 years ago

Reduce the amount of logic in system_routing_configure to check for existing routes. This code is in the same style as used
by Ermal. Ticket #507

History

#1 Updated by Ermal Luçi over 9 years ago

  • Status changed from New to Feedback

#2 Updated by Seth Mos over 9 years ago

  • Status changed from Feedback to Resolved

Confirmed fixed in snapshot 2010-05-16. Also reduced logic to be in line with existing code.

Also available in: Atom PDF