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

ASO API - AKS Automatic Authentication Failure #5017

Open
ljtill opened this issue Jul 24, 2024 · 4 comments
Open

ASO API - AKS Automatic Authentication Failure #5017

ljtill opened this issue Jul 24, 2024 · 4 comments
Assignees
Labels
kind/bug Categorizes issue or PR as related to a bug. priority/important-longterm Important over the long term, but may not be staffed and/or may need multiple releases to complete. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. triage/accepted Indicates an issue or PR is ready to be actively worked on.
Milestone

Comments

@ljtill
Copy link

ljtill commented Jul 24, 2024

/kind bug

What steps did you take and what happened:
When integrating the new ASO API with an AKS Automatic resource, I discovered that the reconciler cannot authenticate with the newly provisioned cluster. By default, AKS Automatic uses Entra ID with Azure RBAC integration. This setup causes the reconciler to fail authentication, preventing reconciliation completion.

Cluster authentication and authorization

What did you expect to happen:
The reconciler successfully authenticates with the cluster using Azure RBAC. Maybe similar to how the AzureClusterIdentity object is used.

Anything else you would like to add:
Management cluster is using AKS Automatic with the Cluster API Operator and Workload Identity to authenticate.

Environment:

  • cluster-api-provider-azure version: v1.16.0
  • Kubernetes version: 1.29.6
@k8s-ci-robot k8s-ci-robot added the kind/bug Categorizes issue or PR as related to a bug. label Jul 24, 2024
@mboersma mboersma added this to the v1.17 milestone Aug 8, 2024
@mboersma mboersma added the needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. label Aug 8, 2024
@nojnhuh
Copy link
Contributor

nojnhuh commented Aug 14, 2024

I think I have a decent handle on this.

/triage accepted
/assign

@k8s-ci-robot k8s-ci-robot added the triage/accepted Indicates an issue or PR is ready to be actively worked on. label Aug 14, 2024
@nojnhuh nojnhuh removed the needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. label Aug 15, 2024
@willie-yao
Copy link
Contributor

/priority backlog

@k8s-ci-robot k8s-ci-robot added the priority/backlog Higher priority than priority/awaiting-more-evidence. label Aug 15, 2024
@nojnhuh
Copy link
Contributor

nojnhuh commented Sep 12, 2024

I've been working on this in the background and have it mostly working, but don't want to rush anything for the release next week.

/milesone next

@nojnhuh
Copy link
Contributor

nojnhuh commented Sep 12, 2024

/milestone next

@k8s-ci-robot k8s-ci-robot modified the milestones: v1.17, next Sep 12, 2024
@nojnhuh nojnhuh mentioned this issue Sep 23, 2024
4 tasks
@dtzar dtzar added priority/important-longterm Important over the long term, but may not be staffed and/or may need multiple releases to complete. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed priority/backlog Higher priority than priority/awaiting-more-evidence. labels Oct 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes issue or PR as related to a bug. priority/important-longterm Important over the long term, but may not be staffed and/or may need multiple releases to complete. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. triage/accepted Indicates an issue or PR is ready to be actively worked on.
Projects
Status: No status
Development

No branches or pull requests

6 participants