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

CAPZ creates a vnet, subnet while creating an AKS cluster. #3051

Closed
LochanRn opened this issue Jan 18, 2023 · 6 comments
Closed

CAPZ creates a vnet, subnet while creating an AKS cluster. #3051

LochanRn opened this issue Jan 18, 2023 · 6 comments
Labels
area/managedclusters Issues related to managed AKS clusters created through the CAPZ ManagedCluster Type kind/bug Categorizes issue or PR as related to a bug.

Comments

@LochanRn
Copy link
Member

LochanRn commented Jan 18, 2023

/kind bug

What steps did you take and what happened:
[A clear and concise description of what the bug is.]
Create an AKS cluster from capz, without specifying vnet and subnet details

What did you expect to happen:
An aks cluster is created along with a vnet and subnet.

By default when a user creates an AKS cluster from az cli or the azure portal without specifying vnet and subnet details, AKS generates a vnet and subnet and manages it internally. The below image clearly shows VNET and SUBNET are Managed and user, has no access to the vnet and subnet created by AKS.
Screenshot 2023-01-18 at 3 29 47 PM

When an AKS cluster is created by CAPZ without specifying BYO vnet and subnet, CAPZ creates a VNET and a SUBNET.
As shown in the image below and user can access the subnet, vnet.
Screenshot 2023-01-18 at 3 33 34 PM

Anything else you would like to add:
[Miscellaneous information that will assist in solving the issue.]
The defaulting webhooks defined for vnet and subnet creates a default subnet and default vnet.

Environment:

  • cluster-api-provider-azure version:
  • Kubernetes version: (use kubectl version):
  • OS (e.g. from /etc/os-release):
@k8s-ci-robot k8s-ci-robot added the kind/bug Categorizes issue or PR as related to a bug. label Jan 18, 2023
@LochanRn
Copy link
Member Author

@CecileRobertMichon @jackfrancis thoughts ??

@CecileRobertMichon
Copy link
Contributor

@alexeldeib do you recall why the AKS implementation was done this way?

@alexeldeib
Copy link
Contributor

alexeldeib commented Jan 18, 2023 via email

@CecileRobertMichon
Copy link
Contributor

#929 (comment) :D

@CecileRobertMichon CecileRobertMichon added this to the next milestone Jan 26, 2023
@dtzar dtzar added the area/managedclusters Issues related to managed AKS clusters created through the CAPZ ManagedCluster Type label Apr 18, 2023
@jackfrancis jackfrancis removed this from the next milestone May 11, 2023
@CecileRobertMichon
Copy link
Contributor

/close

this is by design, no plans to fix currently

@k8s-ci-robot
Copy link
Contributor

@CecileRobertMichon: Closing this issue.

In response to this:

/close

this is by design, no plans to fix currently

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/managedclusters Issues related to managed AKS clusters created through the CAPZ ManagedCluster Type kind/bug Categorizes issue or PR as related to a bug.
Projects
Archived in project
Development

No branches or pull requests

6 participants