Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

weidenbaum: init location #844

Merged
merged 1 commit into from
May 31, 2024
Merged

weidenbaum: init location #844

merged 1 commit into from
May 31, 2024

Conversation

Noki
Copy link
Member

@Noki Noki commented May 25, 2024

weidenbaum-r0 is connected via LAN but not setup as a standalone router that meshes via LAN. The reason for this is that we had issues with flashing Ubiquiti UniFi AC Mesh devices and bricked them in the process. We did not want to risk this today. This might be changed in the future. We also keep the mesh on the core router active as a backup, but make the metrics worse so routing via weidenbaum-r0 is preferred. There might also be an opportunity to connect to chris-core, but we had no antenna at hand to test this out, but already prepared the configuration to test this out.

@Noki Noki force-pushed the weidenbaum branch 2 times, most recently from 1b7b229 to c8ba474 Compare May 25, 2024 14:47
@Noki Noki marked this pull request as ready for review May 25, 2024 15:04
@Noki Noki marked this pull request as draft May 25, 2024 15:19
@Noki Noki marked this pull request as ready for review May 25, 2024 15:26
@Noki Noki force-pushed the weidenbaum branch 2 times, most recently from ae929eb to 4348286 Compare May 29, 2024 06:00
@Noki Noki requested a review from FFHener May 29, 2024 06:06
Copy link
Contributor

@FFHener FFHener left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not completly sure if i understood everything regarding the -r0 thing, but the code lgtm.You could add a comment about the reason tin the config file in case someone else will have a look at this location.

@Noki
Copy link
Member Author

Noki commented May 31, 2024

comment added

@Noki Noki merged commit 3cd13ba into master May 31, 2024
5 checks passed
@Noki Noki deleted the weidenbaum branch May 31, 2024 19:13
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