Skip to content

Feature/router - #11

Merged
embtom merged 5 commits into
mainfrom
feature/router
Oct 23, 2026
Merged

embtom merged 5 commits into
mainfrom
feature/router

Conversation

@embtom

@embtom embtom commented Oct 16, 2026

Copy link
Copy Markdown
Owner

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟡 Changes recommended

Unresolved DNS defaults, DHCPv6 filtering, and LAN interface ownership conflicts can disrupt connectivity.

3 open issues
What changed in this PR

Adds a default workstation firewall and an optional IPv4 router role.

Changes:

  • Adds nftables input filtering and service management.
  • Adds opt-in LAN DHCP, forwarding, and NAT.
  • Documents configuration and exposes role tags in VS Code.
File Description
README.md Documents firewall and router configuration.
ansible/​roles/​router/​templates/​20-router.nft.j2 Adds LAN access, forwarding, and NAT rules.
ansible/​roles/​router/​templates/​10-router-lan.network.j2 Configures LAN addressing and DHCP.
ansible/​roles/​router/​tasks/​main.yml Provisions optional routing services and configuration.
ansible/​roles/​router/​handlers/​main.yml Applies network and forwarding changes.
ansible/​roles/​router/​defaults/​main.yml Defines router and DHCP defaults.
ansible/​roles/​nftables/​templates/​10-filter.nft.j2 Defines default-drop input filtering.
ansible/​roles/​nftables/​tasks/​main.yml Installs and configures the firewall.
ansible/​roles/​nftables/​handlers/​main.yml Restarts nftables after changes.
ansible/​playbooks/​workstation.yml Integrates firewall and router roles.
.vscode/​tasks.json Adds firewall and router tag choices.

Review effort: Balanced

Comment thread ansible/roles/router/defaults/main.yml Resolved
Comment thread ansible/roles/router/tasks/main.yml Resolved
@embtom
embtom force-pushed the feature/router branch 2 times, most recently from d0423da to a6f00f4 Compare October 18, 2026 09:17
@embtom
embtom marked this pull request as ready for review October 21, 2026 04:44
@embtom
embtom force-pushed the feature/router branch 3 times, most recently from 2ee21d8 to 2901ace Compare October 22, 2026 21:27
@embtom
embtom merged commit ad17ac2 into main Oct 23, 2026
2 checks passed
@embtom
embtom deleted the feature/router branch October 23, 2026 04:08
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.

3 participants