Skip to content

Documented allowedServiceAccounts#5105

Open
acroca wants to merge 2 commits intodapr:v1.18from
acroca:allowed-service-accounts
Open

Documented allowedServiceAccounts#5105
acroca wants to merge 2 commits intodapr:v1.18from
acroca:allowed-service-accounts

Conversation

@acroca
Copy link
Copy Markdown
Member

@acroca acroca commented Apr 7, 2026

Thank you for helping make the Dapr documentation better!

Please follow this checklist before submitting:

  • Commits are signed with Developer Certificate of Origin (DCO - learn more)
  • Read the contribution guide
  • Commands include options for Linux, MacOS, and Windows within tabpane
  • New file and folder names are globally unique
  • Page references use shortcodes instead of markdown or URL links
  • Images use HTML style and have alternative text
  • Places where multiple code/command options are given have tabpane

In addition, please fill out the following to help reviewers understand this pull request:

Description

Added documentations for allowedServiceAccounts, specially highlighting the newly supported format.

Issue reference

dapr/dapr#9704

Signed-off-by: Albert Callarisa <albert@diagrid.io>
@acroca acroca requested review from a team as code owners April 7, 2026 13:10
@alicejgibbons
Copy link
Copy Markdown
Contributor

@acroca did you update the helm docs too? Ie. i would expect a change here:
image


### Examples

Configure via Helm:
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can we link directly to the Helm values here?

| `*:*` | All service accounts in all namespaces |

{{% alert title="Note" color="primary" %}}
The `dapr_sidecar_injector.allowedServiceAccountsPrefixNames` Helm value is deprecated. Migrate your entries to `dapr_sidecar_injector.allowedServiceAccounts` using glob patterns instead (for example, `my-ns:my-prefix*` replaces the previous prefix-matching behavior). The deprecated value still functions but logs a deprecation warning.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Deprecated as of what version? i assume 1.18 which is this pr

Copy link
Copy Markdown
Contributor

@alicejgibbons alicejgibbons left a comment

Choose a reason for hiding this comment

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

Minor things

Signed-off-by: Albert Callarisa <albert@diagrid.io>
@acroca
Copy link
Copy Markdown
Member Author

acroca commented Apr 14, 2026

@acroca did you update the helm docs too? Ie. i would expect a change here: image

I forgot that, here's the PR to fix those: dapr/dapr#9793

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