Skip to content
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.

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

[AKS] az aks create: Add --network-plugin=none support for BYO CNI #23339

Closed
wants to merge 1 commit into from

Conversation

mattstam
Copy link
Contributor

@mattstam mattstam commented Jul 26, 2022

Related command

az aks create --network-plugin none

Description

Previous valid values for networkPlugin: kubenet, azure
New valid values for networkPlugin: kubenet, azure, none

network-plugin=none enables Bring Your Own CNI (BYOCNI). With this option, no CNI is installed by default.

Testing Guide
az aks create --network-plugin none

History Notes

[AKS] az aks create: Add --network-plugin=none support for BYO CNI


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

@ghost ghost requested review from yonzhan and kairu-ms July 26, 2022 18:41
@ghost ghost added the Auto-Assign Auto assign by bot label Jul 26, 2022
@ghost ghost requested review from wangzelin007 and necusjz July 26, 2022 18:41
@ghost ghost assigned necusjz Jul 26, 2022
@ghost ghost added this to the Jul 2022 (2022-08-02) milestone Jul 26, 2022
@ghost ghost added the Network az network vnet/lb/nic/dns/etc... label Jul 26, 2022
@ghost ghost assigned zhoxing-ms Jul 26, 2022
@ghost ghost added the AKS az aks/acs/openshift label Jul 26, 2022
@yonzhan
Copy link
Collaborator

yonzhan commented Jul 27, 2022

AKS


@AllowLargeResponse()
@AKSCustomResourceGroupPreparer(random_name_length=17, name_prefix='clitest', location='westus2')
def test_aks_create_with_network_plugin_none(self, resource_group, resource_group_location):
Copy link
Member

Choose a reason for hiding this comment

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

Queued a pipeline to run the test in live mode. Passed.

Please upload the recording file from the artifact of the above pipeline.

@zhoxing-ms
Copy link
Contributor

@mattstam Could you please upload the recording file for test_aks_create_with_network_plugin_none test to pass the CI issue?

@zhoxing-ms
Copy link
Contributor

@mattstam Please note that if you do not resolve the CI issue before Thursday, this PR will not be able to catch up with the release of this sprint, we can only postpone it to the next sprint (09-06)

@@ -369,7 +369,7 @@
- name: --network-plugin
type: string
short-summary: The Kubernetes network plugin to use.
long-summary: Specify "azure" for advanced networking configurations. Defaults to "kubenet".
long-summary: Specify "azure" for routable pod IPs from VNET, "kubenet" for non-routable pod IPs with an overlay network, or "none" for no networking configured. Defaults to "kubenet".
Copy link
Member

Choose a reason for hiding this comment

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

Missing help message update several lines above. Opened another PR #23344 to catch the July release.

@zhoxing-ms
Copy link
Contributor

zhoxing-ms commented Jul 27, 2022

Replaced by #23344

@zhoxing-ms zhoxing-ms closed this Jul 27, 2022
@mattstam mattstam deleted the mattstam/network-plugin-none branch July 27, 2022 16:19
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 Network az network vnet/lb/nic/dns/etc...
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants