Skip to content

[AKS] Add feature support for apiserver vnet integration#31318

Merged
yanzhudd merged 9 commits intoAzure:devfrom
levimm:lima2/0416-avi
May 7, 2025
Merged

[AKS] Add feature support for apiserver vnet integration#31318
yanzhudd merged 9 commits intoAzure:devfrom
levimm:lima2/0416-avi

Conversation

@levimm
Copy link
Contributor

@levimm levimm commented Apr 23, 2025

Related command
az aks create --enable-apiserver-vnet-integration
az aks update --enable-apiserver-vnet-integration --apiserver-subnet-id
az aks update --enable-private-cluster
az aks update --disable-private-cluster

Description
GA the feature for apiserver vnet integration. Support create/update with --enable-apiserver-vnet-integration. Also support update cluster to enable or disable private cluster with apiserver vnet integration is used.

Most logic are copied from AKS cli extension.

Testing Guide
az aks create --enable-apiserver-vnet-integration
az aks update --enable-apiserver-vnet-integration --apiserver-subnet-id
az aks update --enable-private-cluster
az aks update --disable-private-cluster

History Notes

[AKS] az aks create/update: Add support for apiserver vnet integration feature


This checklist is used to make sure that common guidelines for a pull request are followed.

@azure-client-tools-bot-prd
Copy link

azure-client-tools-bot-prd bot commented Apr 23, 2025

️✔️AzureCLI-FullTest
️✔️acr
️✔️latest
️✔️3.12
️✔️3.9
️✔️acs
️✔️latest
️✔️3.12
️✔️3.9
️✔️advisor
️✔️latest
️✔️3.12
️✔️3.9
️✔️ams
️✔️latest
️✔️3.12
️✔️3.9
️✔️apim
️✔️latest
️✔️3.12
️✔️3.9
️✔️appconfig
️✔️latest
️✔️3.12
️✔️3.9
️✔️appservice
️✔️latest
️✔️3.12
️✔️3.9
️✔️aro
️✔️latest
️✔️3.12
️✔️3.9
️✔️backup
️✔️latest
️✔️3.12
️✔️3.9
️✔️batch
️✔️latest
️✔️3.12
️✔️3.9
️✔️batchai
️✔️latest
️✔️3.12
️✔️3.9
️✔️billing
️✔️latest
️✔️3.12
️✔️3.9
️✔️botservice
️✔️latest
️✔️3.12
️✔️3.9
️✔️cdn
️✔️latest
️✔️3.12
️✔️3.9
️✔️cloud
️✔️latest
️✔️3.12
️✔️3.9
️✔️cognitiveservices
️✔️latest
️✔️3.12
️✔️3.9
️✔️compute_recommender
️✔️latest
️✔️3.12
️✔️3.9
️✔️computefleet
️✔️latest
️✔️3.12
️✔️3.9
️✔️config
️✔️latest
️✔️3.12
️✔️3.9
️✔️configure
️✔️latest
️✔️3.12
️✔️3.9
️✔️consumption
️✔️latest
️✔️3.12
️✔️3.9
️✔️container
️✔️latest
️✔️3.12
️✔️3.9
️✔️containerapp
️✔️latest
️✔️3.12
️✔️3.9
️✔️core
️✔️latest
️✔️3.12
️✔️3.9
️✔️cosmosdb
️✔️latest
️✔️3.12
️✔️3.9
️✔️databoxedge
️✔️latest
️✔️3.12
️✔️3.9
️✔️dls
️✔️latest
️✔️3.12
️✔️3.9
️✔️dms
️✔️latest
️✔️3.12
️✔️3.9
️✔️eventgrid
️✔️latest
️✔️3.12
️✔️3.9
️✔️eventhubs
️✔️latest
️✔️3.12
️✔️3.9
️✔️feedback
️✔️latest
️✔️3.12
️✔️3.9
️✔️find
️✔️latest
️✔️3.12
️✔️3.9
️✔️hdinsight
️✔️latest
️✔️3.12
️✔️3.9
️✔️identity
️✔️latest
️✔️3.12
️✔️3.9
️✔️iot
️✔️latest
️✔️3.12
️✔️3.9
️✔️keyvault
️✔️latest
️✔️3.12
️✔️3.9
️✔️lab
️✔️latest
️✔️3.12
️✔️3.9
️✔️managedservices
️✔️latest
️✔️3.12
️✔️3.9
️✔️maps
️✔️latest
️✔️3.12
️✔️3.9
️✔️marketplaceordering
️✔️latest
️✔️3.12
️✔️3.9
️✔️monitor
️✔️latest
️✔️3.12
️✔️3.9
️✔️mysql
️✔️latest
️✔️3.12
️✔️3.9
️✔️netappfiles
️✔️latest
️✔️3.12
️✔️3.9
️✔️network
️✔️latest
️✔️3.12
️✔️3.9
️✔️policyinsights
️✔️latest
️✔️3.12
️✔️3.9
️✔️privatedns
️✔️latest
️✔️3.12
️✔️3.9
️✔️profile
️✔️latest
️✔️3.12
️✔️3.9
️✔️rdbms
️✔️latest
️✔️3.12
️✔️3.9
️✔️redis
️✔️latest
️✔️3.12
️✔️3.9
️✔️relay
️✔️latest
️✔️3.12
️✔️3.9
️✔️resource
️✔️latest
️✔️3.12
️✔️3.9
️✔️role
️✔️latest
️✔️3.12
️✔️3.9
️✔️search
️✔️latest
️✔️3.12
️✔️3.9
️✔️security
️✔️latest
️✔️3.12
️✔️3.9
️✔️servicebus
️✔️latest
️✔️3.12
️✔️3.9
️✔️serviceconnector
️✔️latest
️✔️3.12
️✔️3.9
️✔️servicefabric
️✔️latest
️✔️3.12
️✔️3.9
️✔️signalr
️✔️latest
️✔️3.12
️✔️3.9
️✔️sql
️✔️latest
️✔️3.12
️✔️3.9
️✔️sqlvm
️✔️latest
️✔️3.12
️✔️3.9
️✔️storage
️✔️latest
️✔️3.12
️✔️3.9
️✔️synapse
️✔️latest
️✔️3.12
️✔️3.9
️✔️telemetry
️✔️latest
️✔️3.12
️✔️3.9
️✔️util
️✔️latest
️✔️3.12
️✔️3.9
️✔️vm
️✔️latest
️✔️3.12
️✔️3.9

@azure-client-tools-bot-prd
Copy link

azure-client-tools-bot-prd bot commented Apr 23, 2025

⚠️AzureCLI-BreakingChangeTest
⚠️acs
rule cmd_name rule_message suggest_message
⚠️ 1006 - ParaAdd aks create cmd aks create added parameter apiserver_subnet_id
⚠️ 1006 - ParaAdd aks create cmd aks create added parameter enable_apiserver_vnet_integration
⚠️ 1006 - ParaAdd aks update cmd aks update added parameter apiserver_subnet_id
⚠️ 1006 - ParaAdd aks update cmd aks update added parameter disable_private_cluster
⚠️ 1006 - ParaAdd aks update cmd aks update added parameter enable_apiserver_vnet_integration
⚠️ 1006 - ParaAdd aks update cmd aks update added parameter enable_private_cluster

@yonzhan
Copy link
Collaborator

yonzhan commented Apr 23, 2025

Thank you for your contribution! We will review the pull request and get back to you soon.

@github-actions
Copy link

The git hooks are available for azure-cli and azure-cli-extensions repos. They could help you run required checks before creating the PR.

Please sync the latest code with latest dev branch (for azure-cli) or main branch (for azure-cli-extensions).
After that please run the following commands to enable git hooks:

pip install azdev --upgrade
azdev setup -c <your azure-cli repo path> -r <your azure-cli-extensions repo path>

@microsoft-github-policy-service microsoft-github-policy-service bot added Auto-Assign Auto assign by bot AKS az aks/acs/openshift labels Apr 23, 2025
@levimm levimm changed the title WIP: [AKS] Add feature support for apiserver vnet integration [AKS] Add feature support for apiserver vnet integration Apr 28, 2025
@levimm levimm marked this pull request as ready for review April 28, 2025 00:12
@levimm levimm requested a review from jsntcy as a code owner April 28, 2025 00:12
@levimm levimm closed this Apr 28, 2025
@levimm levimm reopened this Apr 28, 2025
@yonzhan
Copy link
Collaborator

yonzhan commented Apr 28, 2025

Please fix CI issues

@levimm
Copy link
Contributor Author

levimm commented Apr 28, 2025

all CI failure is

# Verify installation and build command index
az --version
/mnt/vss/_work/1/s/env/bin/python: No module named azure.cli.__main__; 'azure.cli' is a package and cannot be directly executed

I don't think it's related to my change.

@FumingZhang
Copy link
Member

Queued live test to validate the change, test passed!

  • test_aks_create_with_apiserver_vnet_integration
  • test_aks_create_with_apiserver_vnet_integration_public

@FumingZhang
Copy link
Member

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 3 pipeline(s).

Copy link
Member

@FumingZhang FumingZhang left a comment

Choose a reason for hiding this comment

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

lgtm

@levimm levimm force-pushed the lima2/0416-avi branch from a0cd813 to 025a1f9 Compare May 1, 2025 01:08
@levimm
Copy link
Contributor Author

levimm commented May 1, 2025

now the failure is related to appservice module, not acs.

@FumingZhang
Copy link
Member

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@levimm
Copy link
Contributor Author

levimm commented May 7, 2025

/azp run

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 31318 in repo Azure/azure-cli

@FumingZhang
Copy link
Member

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@yanzhudd
Copy link
Contributor

yanzhudd commented May 7, 2025

updated the history notes:
image

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

Labels

AKS az aks/acs/openshift Auto-Assign Auto assign by bot

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants