Skip to content

Conversation

@kgal-akl
Copy link

@kgal-akl kgal-akl commented Oct 30, 2025

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

Part of dapr/components-contrib#4036. Opened instead of #4910 as PR 4910 was targeted towards release v1.16.

Issue reference

@kgal-akl kgal-akl requested review from a team as code owners October 30, 2025 18:43
@kgal-akl kgal-akl mentioned this pull request Oct 30, 2025
7 tasks
@kgal-akl
Copy link
Author

@msfussell FYI.

Copy link
Member

@msfussell msfussell left a comment

Choose a reason for hiding this comment

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

@kgal-akl Great PR and looking forward to getting this into the release. Left a few comments


## Authentication Methods

We currently support the following authentication methods:
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
We currently support the following authentication methods:
The following authentication methods are supported:


## Related links

- [Secrets building block]({{% ref secrets %}})
Copy link
Member

Choose a reason for hiding this comment

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

Suggest a reference to Akeyless account creation

```

{{% alert title="Failing Retrieval in Bulk" color="info" %}}
When retrieving secrets using the bulk API, all successfully-retrieved secrets will be returned and failed ones will be logged in the component log.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
When retrieving secrets using the bulk API, all successfully-retrieved secrets will be returned and failed ones will be logged in the component log.
When retrieving secrets using the bulk API, all successfully-retrieved secrets are returned and failed ones are logged in the component log.


Or using the Dapr SDK. The example below retrieves all static secrets from path `/path/to/department`:

```go
Copy link
Member

Choose a reason for hiding this comment

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

Can you add a "Language Tab" section around this, so it is then clear which language and other could be added in the future.

## Component Format

```yaml
# yaml-language-server: $schema=../../component-metadata-schema.json
Copy link
Member

Choose a reason for hiding this comment

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

This component format is wrong. It should be like this, yes?

apiVersion: dapr.io/v1alpha1
kind: Component
metadata:
  name: akeyless
spec:
  type: secretstores.akeyless
  version: v1
  metadata:
  - name: gatewayUrl
```

@msfussell msfussell added the waiting-on-code-pr The code PR needs to be merged before the docs are updated label Dec 4, 2025
@msfussell msfussell added this to the 1.17 milestone Dec 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

waiting-on-code-pr The code PR needs to be merged before the docs are updated

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants