Skip to content

feat: add kbs.extraSecrets for non-ESO secrets#15

Merged
beraldoleal merged 3 commits intovalidatedpatterns:mainfrom
beraldoleal:extra-secrets
Mar 6, 2026
Merged

feat: add kbs.extraSecrets for non-ESO secrets#15
beraldoleal merged 3 commits intovalidatedpatterns:mainfrom
beraldoleal:extra-secrets

Conversation

@beraldoleal
Copy link
Collaborator

Allow adding pre-existing K8s Secrets to kbsSecretResources without creating ExternalSecrets. Useful for secrets created by imperative jobs or other controllers outside the Vault flow.

Allow adding pre-existing K8s Secrets to kbsSecretResources
without creating ExternalSecrets. Useful for secrets created
by imperative jobs or other controllers outside the Vault flow.

Signed-off-by: Beraldo Leal <bleal@redhat.com>
@beraldoleal beraldoleal requested review from a team and butler54 March 5, 2026 15:03
Copy link
Collaborator

@butler54 butler54 left a comment

Choose a reason for hiding this comment

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

LGTM nice enhancement

@butler54 butler54 self-requested a review March 6, 2026 01:59
@butler54
Copy link
Collaborator

butler54 commented Mar 6, 2026

@beraldoleal. You should be able to merge now can you do the following (i've left it open so you can make changes w/o needing re-approval)

  1. Change the version in chart.yaml to 0.2.1
  2. run make helm-docs
  3. Push
  4. Merge
  5. Create a release / v0.2.1 version tag as well
    (e.g. to release properly the version in chart.yaml must reflect the new tag)

@butler54
Copy link
Collaborator

butler54 commented Mar 6, 2026

@sabre1041 ^^ we should have a discussion on how messy this release process is with the wider team

IIUC, the semantic ver is not triggered yet. And we need a 0.2.1

Signed-off-by: Beraldo Leal <bleal@redhat.com>
just a make helm-docs

Signed-off-by: Beraldo Leal <bleal@redhat.com>
@beraldoleal beraldoleal merged commit 17cc8f3 into validatedpatterns:main Mar 6, 2026
4 checks passed
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