feat: STACKIT hub and spoke architecture#230
Conversation
Scorecard Check
📊 meshstack-hub Module Scorecard
📋 Per-Module Category SummaryScore per category per building block.
Core Structure — ✅ all passingBasic module file structure and documentation — applies to 3 modules
Core Structure — Summary
Integration — some checks failingmeshstack_integration.tf conventions — applies to 3 modules
Integration — Summary
Azure Backplane — not applicableAzure UAMI-based automation principal conventions — applies to 0 modules No applicable modules. STACKIT Backplane — ✅ all passingSTACKIT WIF-based automation principal conventions — applies to 3 modules
STACKIT Backplane — Summary
Testing — some checks failingEnd-to-end test coverage — applies to 3 modules
Testing — Summary
📈 Overall SummaryOverall Average Score: 88%Score Distribution
|
|
This pull request is automatically being deployed by Amplify Hosting (learn more). |
28fd029 to
5e6bfec
Compare
There was a problem hiding this comment.
Pull request overview
Adds a STACKIT “hub-and-spoke” reference architecture that bootstraps a sandbox platform plus centralized IPAM (network area hub) and a self-service routed-network building block (spokes), and extends the existing STACKIT sandbox landing zone to optionally propagate a network-area tag into tenant project labels.
Changes:
- Add new
reference-architectures/stackit-hub-spokerunnable reference architecture (meshStack integration + composed building block implementation + docs). - Extend
stackit-sandbox-landingzonebuilding block to optionally forward a meshStack landing zone tag as the STACKITnetworkAreaproject label input. - Fix
modules/stackit/network-areato avoid empty-list vs null inconsistencies fordefault_nameservers.
Reviewed changes
Copilot reviewed 13 out of 14 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| reference-architectures/stackit-sandbox-landingzone/meshstack_integration.tf | Updates the BBD symbol URL to the reference-architecture location. |
| reference-architectures/stackit-sandbox-landingzone/buildingblock/variables.tf | Adds network_area_tag_name variable (optional forwarding to nested integration). |
| reference-architectures/stackit-sandbox-landingzone/buildingblock/README.md | Updates generated terraform-docs outputs (inputs table includes new variable). |
| reference-architectures/stackit-sandbox-landingzone/buildingblock/main.tf | Forwards network_area_tag_name into the nested STACKIT integration. |
| reference-architectures/stackit-hub-spoke/README.md | Adds reference-architecture documentation (overview, diagram, responsibilities). |
| reference-architectures/stackit-hub-spoke/meshstack_integration.tf | Registers the “Hub and Spoke Network” BBD (inputs/outputs, inline readme). |
| reference-architectures/stackit-hub-spoke/buildingblock/versions.tf | Declares Terraform/provider constraints for the composed building block. |
| reference-architectures/stackit-hub-spoke/buildingblock/variables.tf | Defines inputs for foundation bootstrap + hub/spoke network parameters. |
| reference-architectures/stackit-hub-spoke/buildingblock/README.md | Adds generated terraform-docs for the new composed building block. |
| reference-architectures/stackit-hub-spoke/buildingblock/provider.tf | Configures the STACKIT provider for the composed building block. |
| reference-architectures/stackit-hub-spoke/buildingblock/outputs.tf | Exposes key outputs from the foundation module. |
| reference-architectures/stackit-hub-spoke/buildingblock/main.tf | Composes foundation + network-area + network modules; provisions hub NA instance; creates networked landing zone tagged with NA id. |
| modules/stackit/network-area/buildingblock/main.tf | Uses a length-check to set default_nameservers to null only when empty. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
grubmeshi
left a comment
There was a problem hiding this comment.
LGTM from technical point of view (will actually try it out later, once you've addressed little remarks)
5e6bfec to
134c862
Compare
650a205 to
8e52801
Compare
No description provided.