Adds AKS support #376

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.

Sign up for GitHub

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

Jump to bottom

Merged

hernan82arg merged 18 commits into main from feat/azure

May 7, 2025

Contributor

hernan82arg commented May 2, 2025 •

edited by rmvangun

Loading

Adds AKS support. This includes both the network/azure-vnet and cluster/azure-aks components.


          Adds AKS support

5133e7b

hernan82arg added the feature label

hernan82arg self-assigned this


          Fixes formatting

a90c2ab

rmvangun reviewed

View reviewed changes

terraform/cluster/azure-aks/main.tf Show resolved Hide resolved

rmvangun reviewed

View reviewed changes

terraform/cluster/azure-aks/main.tf Show resolved Hide resolved

rmvangun reviewed

View reviewed changes

terraform/cluster/azure-aks/main.tf Outdated Show resolved Hide resolved

rmvangun reviewed

View reviewed changes

terraform/network/azure-vpc/variables.tf Outdated Show resolved Hide resolved

rmvangun reviewed

View reviewed changes

terraform/cluster/azure-aks/variables.tf Outdated Show resolved Hide resolved

rmvangun reviewed

View reviewed changes

terraform/cluster/azure-aks/main.tf Outdated Show resolved Hide resolved

rmvangun reviewed

View reviewed changes

terraform/cluster/azure-aks/main.tf Show resolved Hide resolved

rmvangun reviewed

View reviewed changes

Contributor

rmvangun left a comment

Rename azure vpc to vnet or VNet

hernan82arg changed the title ~~Adds AKS support~~ WIP - Adds AKS support

hernan82arg added 6 commits

May 2, 2025 17:30


          Enables autoscaling

3834d10


          Fix formatting

0ccdc6b


          Fixes checkov findings

b384f5a


          Fix formatting

d37509e


          Fix checkov findings

89cce76


          Fix formatting

7e012c5

github-advanced-security bot found potential problems

View reviewed changes

terraform/cluster/azure-aks/main.tf Dismissed Show dismissed Hide dismissed

terraform/cluster/azure-aks/main.tf Fixed Show fixed Hide fixed

terraform/cluster/azure-aks/main.tf Fixed Show fixed Hide fixed

terraform/cluster/azure-aks/main.tf Fixed Show fixed Hide fixed

terraform/cluster/azure-aks/main.tf Fixed Show fixed Hide fixed

terraform/cluster/azure-aks/main.tf Fixed Show fixed Hide fixed

hernan82arg added 3 commits

May 5, 2025 16:44


          Fixes

45969cb


          Fix checkov finding

c7234de


          Fix checkov finding

67a9beb

github-advanced-security bot found potential problems

View reviewed changes

terraform/cluster/azure-aks/main.tf Dismissed Show dismissed Hide dismissed

terraform/cluster/azure-aks/main.tf Dismissed Show dismissed Hide dismissed

hernan82arg added 2 commits

May 5, 2025 18:32


          Fixes key expiricy issue

bddfd45


          Remove unnecessary variables

88c44e5

hernan82arg changed the title ~~WIP - Adds AKS support~~ Adds AKS support


          Addresses many of the points outlined in our call

9223f7a

github-advanced-security bot found potential problems

View reviewed changes

terraform/cluster/azure-aks/main.tf Fixed Show fixed Hide fixed

terraform/cluster/azure-aks/main.tf Fixed Show fixed Hide fixed

terraform/cluster/azure-aks/main.tf Fixed Show fixed Hide fixed

terraform/cluster/azure-aks/main.tf Fixed Show fixed Hide fixed

terraform/cluster/azure-aks/main.tf Fixed Show fixed Hide fixed


          Some more refactoring

f9619f7

hernan82arg added 3 commits

May 6, 2025 17:43


          Fix formatting


          more fixes

d945e38


          Fixes

cc6cd1e

github-advanced-security bot found potential problems

View reviewed changes

terraform/cluster/azure-aks/main.tf Dismissed Show dismissed Hide dismissed

terraform/cluster/azure-aks/main.tf Dismissed Show dismissed Hide dismissed

terraform/cluster/azure-aks/main.tf Dismissed Show dismissed Hide dismissed

terraform/cluster/azure-aks/main.tf Dismissed Show dismissed Hide dismissed

terraform/cluster/azure-aks/main.tf Dismissed Show dismissed Hide dismissed

terraform/cluster/azure-aks/main.tf Dismissed Show dismissed Hide dismissed

terraform/cluster/azure-aks/main.tf Dismissed Show dismissed Hide dismissed

rmvangun approved these changes

View reviewed changes

Contributor

rmvangun left a comment

looks good!

hernan82arg merged commit 266fc2e into main

8 checks passed

hernan82arg deleted the feat/azure branch

May 7, 2025 14:49

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels