Skip to content

Conversation

@rmvangun
Copy link
Contributor

@rmvangun rmvangun commented May 20, 2025

A few light enhancements to improve things when performing windsor up --wait + windsor down against a production grade cluster. These changes make sure the defaults are sufficient for a standard blueprint deployment.

IP range change:

  • Ensure multiple subnets don't overlap
  • Ensure there are enough available IPs in the private subnet for azure networking

Other changes:

  • Update documentation for Azure AKS and VNet modules
  • Add Azure example context
  • Update default subnet input values
  • Enhance input validation
  • Create azure-example context
  • Update to the latest windsor init generated assets
  • Make nat gateway optional

rmvangun added 2 commits May 20, 2025 10:46
- Update AKS service_cidr to 10.0.0.0/16 and dns_service_ip to 10.0.0.10

- Simplify network config structure in AKS module

- Fix test assertions for VNet address_space and subnet structure

- Add context_id to network configuration tests

- Update subnet object structure in VNet tests
- Update documentation for Azure AKS and VNet modules

- Add Azure example context

- Update Talos and Nginx configurations

- Add AKS module README
@rmvangun rmvangun self-assigned this May 20, 2025
@rmvangun rmvangun added the enhancement New feature or request label May 20, 2025
@rmvangun rmvangun changed the title feat(azure): update docs and examples for network defaults feat(azure): update network defaults May 20, 2025
@rmvangun rmvangun changed the title feat(azure): update network defaults feat(azure): Set defaults for production AKS May 20, 2025
@rmvangun rmvangun requested a review from hernan82arg May 20, 2025 18:38
Copy link
Contributor

@hernan82arg hernan82arg left a comment

Choose a reason for hiding this comment

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

LGTM

@rmvangun rmvangun merged commit e428c4e into main May 20, 2025
5 checks passed
@rmvangun rmvangun deleted the feat/azure-blueprint-network-defaults branch May 20, 2025 20:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants