Skip to content

Azure Workload Identity Support#361

Open
kgal-akl wants to merge 2 commits intomainfrom
ASM-17130-bump-injector-alpine
Open

Azure Workload Identity Support#361
kgal-akl wants to merge 2 commits intomainfrom
ASM-17130-bump-injector-alpine

Conversation

@kgal-akl
Copy link
Contributor

@kgal-akl kgal-akl commented Mar 5, 2026

add azure workload identity env vars to injector chart.

added azure workload identity env vars to injector values
@kgal-akl kgal-akl changed the title Update Injector Version Update Injector 0.35.0 + Azure Workload Identity Support Mar 5, 2026
@kgal-akl kgal-akl changed the title Update Injector 0.35.0 + Azure Workload Identity Support ASM-17130 | Update Injector 0.35.0 + Azure Workload Identity Support Mar 5, 2026
@kgal-akl kgal-akl changed the title ASM-17130 | Update Injector 0.35.0 + Azure Workload Identity Support Azure Workload Identity Support Mar 5, 2026
# the gw base64 certificate file e.g: 'cat my-gw.crt | base64'
# AKEYLESS_GW_CERTIFICATE:

# For Azure
Copy link
Contributor

Choose a reason for hiding this comment

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

For comments in values.yaml, please also get approval from Eldad

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@EldadH89 WDYT?

Copy link
Contributor

Choose a reason for hiding this comment

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

@kgal-akl why do we need to add this inside the chart? isn't this should be supported using SA with the relevant annotations? https://learn.microsoft.com/en-us/azure/aks/workload-identity-deploy-cluster?tabs=new-cluster

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@kgal-akl kgal-akl requested a review from EldadH89 March 5, 2026 22:22
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.

3 participants