Skip to content

[Draft] Support API server VNet Changes #938

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
Draft
sumanthreddy29 wants to merge 20 commits into
base: main
Choose a base branch
from
Draft

[Draft] Support API server VNet Changes #938

sumanthreddy29 wants to merge 20 commits into from

Conversation

@sumanthreddy29
Copy link
Collaborator

@sumanthreddy29 sumanthreddy29 commented Nov 19, 2025

No description provided.

xinWeiWei24 and others added 20 commits November 12, 2025 14:23
@xinWeiWei24
feat: enable API server VNet integration
2a1f9ba 
Pass enable_apiserver_vnet_integration and api_server_subnet_name from pipeline to Terraform to configure VNet integration and specify API server subnet.
@xinWeiWei24
Add role assignment for subnet of apiserver vnet integration
d969b7d
@xinWeiWei24
Using updated_aks_cli_config_list to avoid type mismatch error
487c563
@xinWeiWei24
Add apiserver_vnet_integration test in new-pipeline-test.yml
fab8b34
@xinWeiWei24
Revert "Add apiserver_vnet_integration test in new-pipeline-test.yml"
bbbf345 
This reverts commit 97c3b34.
@xinWeiWei24
Make sure managed identity has been created
8bc733f
@xinWeiWei24
Handle trailing whitespace in aks_cli test regex patterns
9c03f5d
@xinWeiWei24
Use enable_apiserver_vnet_integration flag for consistency
ef513a1
@xinWeiWei24
Use enable_apiserver_vnet_integration flag for consistency
41ae6d2
@xinWeiWei24
Add new input to azure_input_schema
851fa13
@xinWeiWei24
Expose enable_apiserver_vnet_integration and define api_server_subnet…
70e32ca 
…_name in tfvars file
@xinWeiWei24
Add apiserver_vnet_integration test in new-pipeline-test.yml
1fa9b24
@xinWeiWei24
Remove excessive code
ea77982
@xinWeiWei24
Revert "Add apiserver_vnet_integration test in new-pipeline-test.yml"
5478220 
This reverts commit 1fa9b24.
@xinWeiWei24
Ensure aks_cli_config is consistent across both the Azure and aks-cli…
fade289 
… modules
@sumanthreddy29
update aks-cli module
90b005d
@sumanthreddy29
test changes
e450e10
@sumanthreddy29
update
1ff05a8
@sumanthreddy29
fix error
95959b4
@sumanthreddy29
update var
ea6249b
@xinWeiWei24 xinWeiWei24 force-pushed the xinwei/support_apiserver_vnet branch from fade289 to 630db52 Compare November 20, 2025 04:28
Base automatically changed from xinwei/support_apiserver_vnet to main November 20, 2025 23:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alyssa1303 alyssa1303 Awaiting requested review from alyssa1303 alyssa1303 will be requested when the pull request is marked ready for review alyssa1303 is a code owner

@anson627 anson627 Awaiting requested review from anson627 anson627 will be requested when the pull request is marked ready for review anson627 is a code owner

@xinWeiWei24 xinWeiWei24 Awaiting requested review from xinWeiWei24 xinWeiWei24 will be requested when the pull request is marked ready for review xinWeiWei24 is a code owner

@vittoriasalim vittoriasalim Awaiting requested review from vittoriasalim vittoriasalim will be requested when the pull request is marked ready for review vittoriasalim is a code owner

@wonderyl wonderyl Awaiting requested review from wonderyl wonderyl will be requested when the pull request is marked ready for review wonderyl is a code owner

Copilot code review Copilot Awaiting requested review from Copilot Copilot will automatically review once the pull request is marked ready for review

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@sumanthreddy29 @xinWeiWei24