Bug #13228
closedRecovering interface gateway may not be added back into gateway groups and rules when expected
100%
Description
When an interface/gateway recovers and rc.newwanip runs, the gateway may not end up in the ruleset in groups or rules as expected. This doesn't affect every setup or event, but it's not clear why it isn't consistent. Users report that manually running a filter reload after the system settles, without changing any settings, returns the groups and rules to the expected state.
The following diff triggers a filter reload at the end of rc.newwanip unconditionally which appears to help based on user reports:
diff --git a/src/etc/rc.newwanip b/src/etc/rc.newwanip
index 34aa4c602d..ae2c68fa38 100755
--- a/src/etc/rc.newwanip
+++ b/src/etc/rc.newwanip
@@ -279,9 +279,7 @@ if (!is_ipaddr($oldip) || ($curwanip != $oldip) || file_exists("{$g['tmp_path']}
if (empty($config['interfaces'][$interface]['ipaddrv6'])) {
unlink_if_exists("{$g['tmp_path']}/{$interface}_upstart6");
}
-} else {
- /* signal filter reload */
- filter_configure();
}
+filter_configure();
?>
Without knowing more about why it helps I'm hesitant to commit that as-is at this point, but we can investigate and consider it for the next release.
Updated by Marcos M over 2 years ago
May be related to https://redmine.pfsense.org/issues/12920.
Updated by Jim Pingle over 2 years ago
- Plus Target Version changed from 22.09 to 22.11
Updated by Lee Brown over 2 years ago
I have this issue. Adding the filter_configure(); to the end, (while not removing the else block) does resolve this for me. The filter reload is called properly and new traffic is routed out the correct WAN2 gateway.
Updated by Jim Pingle over 2 years ago
- Status changed from New to Feedback
- % Done changed from 0 to 100
Applied in changeset ad20a68bae86fff5660b02789a49618a6e71ae22.
Updated by Jim Pingle about 2 years ago
- Plus Target Version changed from 22.11 to 23.01
Updated by Jim Pingle about 2 years ago
- Category changed from Interfaces to Gateways
Updated by Jim Pingle about 2 years ago
- Status changed from Feedback to Resolved
Seems to be doing OK here for now, and was previously found to help before committing.
If there is a regression discovered later, we can handle it separately.