Skip to content

Feature/update aks 135#33

Open
thomast1906 wants to merge 142 commits intoAdminTurnedDevOps:mainfrom
thomast1906:feature/update-aks-135
Open

Feature/update aks 135#33
thomast1906 wants to merge 142 commits intoAdminTurnedDevOps:mainfrom
thomast1906:feature/update-aks-135

Conversation

@thomast1906
Copy link
Copy Markdown

No description provided.

thomast1906 and others added 30 commits May 7, 2025 09:58
Incorporates all open Renovate PRs (#33, #37, #38, #39, #40, #41, #42, #43, #44, #45)
plus Terraform/provider version bumps and deploy script fixes.

Terraform:
- required_version: >= 1.9.8 -> >= 1.14.0, < 2.0.0 (all modules)
- Pinned to 1.14.8 in CI workflows

Azure provider (azurerm):
- >= 4.28.0 -> >= 4.68.0, < 5.0.0 (all modules)

GitHub Actions (closes PRs #39 #43 #45 #44 #42 #41 #38):
- actions/checkout: v4 -> v6
- azure/login: v2 -> v3
- azure/setup-kubectl: v4 -> v5
- azure/setup-helm: v4 -> v5
- docker/setup-buildx-action: v3 -> v4
- hashicorp/setup-terraform: v3 -> v4
- stefanzweifel/git-auto-commit-action: v5 -> v7

Python dependencies (closes PRs #33 #40):
- Flask: 3.0.3 -> 3.1.3
- Werkzeug: 3.0.4 -> 3.1.8

Note: python:3.14-slim (PR #37) deferred - 3.14 is still in beta

Fix deploy-all.sh:
- Compute REPO_ROOT from script location; use absolute paths throughout
- Pass -backend-config flags to terraform init (matches CI workflow)
- Fix deprecated --query objectId -> --query id for AD group creation

Docs:
- Test-lab-only.md: rewritten with version table, AKS check, env vars
- Update Terraform version prereq refs in README and 1-Create-ACR.md
- CHANGELOG.md: document all changes

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- scripts/deploy-all.sh: fix backend names (devopshardway-rg/devopshardwaysa),
  REPO_ROOT absolute paths, tf_init() helper, AD group capture, no tfvars overwrite
- scripts/cleanup-all.sh: fix TERRAFORM_RG to devopshardway-rg
- .gitignore: exclude tfplan binaries
- 2-vnet/nsg.tf: add inbound Allow rules for HTTP (80) and HTTPS (443) from Internet
  required for Application Gateway for Containers traffic to reach the app
- 2-vnet/terraform.tfvars: align appgw subnet CIDR to live infra (192.168.4.0/24)
- 4-kubernetes_manifest/scripts/1-alb-controller-install-k8s.sh: upgrade ALB
  controller from 1.7.9 to 1.9.16 (fixes schema validation error); make idempotent
  with helm upgrade fallback and kubectl apply for namespace
- .github/workflows/deploy-full.yml: align AKS subnet CIDR to 192.168.0.0/24

End-to-end validated: all Terraform modules deploy cleanly, Docker image built
and pushed to ACR, app accessible at http://bye7fxhjesf7enf7.fz32.alb.azure.com/

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Application Gateway for Containers requires the AzureLoadBalancer service
tag to be allowed inbound for Azure infrastructure health probes to succeed.
Without this rule, AGC data plane health checks can be silently dropped.

NSG rules for AGC delegated subnet:
- Internet -> 80   : HTTP frontend traffic from clients (data plane)
- Internet -> 443  : HTTPS frontend traffic from clients (data plane)
- AzureLoadBalancer -> * : Azure health probes (infrastructure requirement)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Modernise repo: Terraform/provider/AKS upgrades, script fixes, end-to-end validated
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.

4 participants