Skip to content

[data-prepper] Support configurable secretAnnotations and extraDeploy#681

Open
al-cheb wants to merge 1 commit intoopensearch-project:mainfrom
al-cheb:add-extra-params
Open

[data-prepper] Support configurable secretAnnotations and extraDeploy#681
al-cheb wants to merge 1 commit intoopensearch-project:mainfrom
al-cheb:add-extra-params

Conversation

@al-cheb
Copy link
Copy Markdown

@al-cheb al-cheb commented Jun 3, 2025

  • Added configurable secretAnnotations

This allow to inject external secrets inside pipelines,e.g - https://docs.opensearch.org/docs/latest/data-prepper/common-use-cases/log-analytics/ .

To pass credentials:

 username: "${vault:secret/data/path#property#version}"
 password: "${vault:secret/data/data-prepper#password#1}"
  • Added configurable extraDeploy

This is a convenient way to define extra resources, like existingSecret.

Description

[Describe what this change achieves.]

Issues Resolved

[List any issues this PR will resolve. You should likely open an issue if one does not already exist.]

Check List

  • Commits are signed per the DCO using --signoff

For any changes to files within Helm chart directories:

  • Helm chart version bumped
  • Helm chart CHANGELOG.md updated to reflect change

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

- Added configurable `secretAnnotations`

This allow to inject external secrets inside pipelines,e.g - https://docs.opensearch.org/docs/latest/data-prepper/common-use-cases/log-analytics/ .

To pass credentials:

```
 username: "${vault:secret/data/path#property#version}"
 ${vault:secret/data/data-prepper#password#1}"
```

- Added configurable `extraDeploy`

This is convient way to define extra resources, like `existingSecret`.

Signed-off-by: Alexander Chebotov <a.chebotov@arenadata.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

1 participant