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

Experiment: 2-networking-factory #2850

Draft
wants to merge 18 commits into
base: fast-dev
Choose a base branch
from

Conversation

sruffilli
Copy link
Collaborator

@sruffilli sruffilli commented Jan 29, 2025

This (very drafty) PR is meant as a playground for a FAST networking stage that heavily leverages factories to manage VPCs and connectivity.

Some architectural choices I would like to discuss:

  • Projects are defined in terraform, to avoid having a dedicated project factory within networking
  • Connectivity is an attribute of each VPC (vs. having both a VPC factory and a connectivity factory). This feels very natural to me, as each VPC manages its own "wires"
  • Managing NVA connectivity is Really Hard and Really Messy. A drop-in solution could be published together with a specific recipe, but I wouldn't want to add it to the factory

TODO:

  • "Factory"-ize net-vpc
  • "Factory"-ize project
  • "Factory"-ize DNS
    • Support public zones (?)
  • Implement VPN connectivity
    • Basic implementation
    • Handle cloud router management
  • Implement peering connectivity
  • Implement NCC connectivity
    • Basic implementation
    • Handle route filtering
    • Handle connectivity groups
    • Handle dynamic spokes
    • Handle PSA spokes
  • Implement VLAN Attachments (Interconnects are imho out of scope)
  • All sub-factories should be located in the recipe directory
  • Implement NVA connectivity (?)

@sruffilli sruffilli changed the title Sruffilli/fast networking factory Experiment: 2-networking-factory Jan 29, 2025
@sruffilli
Copy link
Collaborator Author

Per conversation with @ludoo and @juliocc, adding project and ncc management to the scope of the factory

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant