Skip to content

More WAN Uptime Improvements#24

Merged
Martinski4GitHub merged 1 commit intoAMTM-OSR:developfrom
Martinski4GitHub:develop
Jul 26, 2025
Merged

More WAN Uptime Improvements#24
Martinski4GitHub merged 1 commit intoAMTM-OSR:developfrom
Martinski4GitHub:develop

Conversation

@Martinski4GitHub
Copy link
Member

Modified code to have a separate "SEED" file for each WAN interface so that each would have its own timestamp. In cases where users have 2 WAN interfaces connected (one as the "Active Primary" and the other in "Hot-Standby"), we can keep data for each interface separately.

Modify code to have a separate "SEED" file for each WAN interface so that each would have its own timestamp. In cases where users have 2 WAN interfaces connected (one as the "Active Primary" and the other in "Hot-Standby"), we can keep data for each interface separately.
@Martinski4GitHub Martinski4GitHub requested a review from a team as a code owner July 26, 2025 09:12
@Martinski4GitHub
Copy link
Member Author

@ExtremeFiretop,
I had another idea to improve the handling of the 2 WAN interfaces since both could be "connected" at the same time. This PR includes those changes for such edge cases.

@Martinski4GitHub Martinski4GitHub merged commit e6d06a7 into AMTM-OSR:develop Jul 26, 2025
1 check passed
@ExtremeFiretop
Copy link
Member

Yes that works well! :) Great idea!
I didn't really think or factor in dual wan when I original thought about the solution

@Martinski4GitHub
Copy link
Member Author

Yes that works well! :) Great idea! I didn't really think or factor in dual wan when I original thought about the solution

Yeah, the "Dual WAN with Failover" setup was completely off my radar as well. I just never had such a configuration at home, but fortunately, the SNB Forums user Stephen Harrington stepped forward to run tests and provide useful feedback, so we caught that in time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants