Skip to content

Conversation

@jonasz-lasut
Copy link
Contributor

Description of your changes

PR implements support for short-lived workload identity credentials that can be used in AKS clusters with workload identity enabled and properly created federated identity credentials. (https://azure.github.io/azure-workload-identity/docs/topics/federated-identity-credential.html)
Default behavior of the function which expects ServicePrincipal credentials have not been changed.
README was updated to show how to configure DeploymentRuntimeConfig so that function can use workload identity credentials.
Function was tested on Upbound Spaces 1.13.1 with Crossplane in version 1.20.0-up.1

I have:

Copy link
Member

@ytsarev ytsarev left a comment

Choose a reason for hiding this comment

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

I built and pushed rc build of xpkg.upbound.io/upbound/function-msgraph:v0.3.0-rc3 and performed e2e tests.

k apply -f example/secrets/azure-workload-identity-creds.yaml
k apply -f example/identity-type-workload-identity.yaml
k apply -f example/definition.yaml # comment status fields there
k get -f example/xr.yaml -o yaml
...
  validatedUsers:
  - displayName: Yury Tsarev
    id: <redacted>
    mail: null
    userPrincipalName: [email protected]

everything works perfectly end-to-end.

@jonasz-lasut thank you so much for this contribution!

@ytsarev ytsarev merged commit 150cd3e into upbound:main Aug 27, 2025
10 checks passed
@jonasz-lasut jonasz-lasut deleted the feat/workload-identity-support branch September 4, 2025 14:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants