Skip to content

feat: add support for configmaps as source#241

Open
pepordev wants to merge 2 commits intoexternal-secrets:mainfrom
pepordev:feat/support-configmaps-as-source
Open

feat: add support for configmaps as source#241
pepordev wants to merge 2 commits intoexternal-secrets:mainfrom
pepordev:feat/support-configmaps-as-source

Conversation

@pepordev
Copy link

Fixes: #240

This would allow configmaps to be set as source and also make the external-secrets to validate spec.target.template.templateFrom[].configMap.name paths

@pepordev pepordev force-pushed the feat/support-configmaps-as-source branch 3 times, most recently from 9ddb6d5 to e829bc0 Compare February 24, 2026 12:10
Signed-off-by: Pedro Parra Ortega <parraortega.pedro@gmail.com>
@pepordev pepordev force-pushed the feat/support-configmaps-as-source branch from e829bc0 to 36b7b52 Compare February 24, 2026 12:11
DEVELOPING.md Outdated
@@ -0,0 +1,220 @@
# Developing the Reloader Controller
Copy link
Contributor

Choose a reason for hiding this comment

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

We don't really need this. I mean, this is something I'm guessing your llm came up with? I much rather read something that a human wrote. The quick section and the project layout are particularly useless since it's ephemeral. We might be moving stuff around soon and I don't want something that I would have to keep updated every time I rename a package.

@Skarlso
Copy link
Contributor

Skarlso commented Feb 28, 2026

The doc can be removed, the rest looks okay.

Signed-off-by: Pedro Parra Ortega <parraortega.pedro@gmail.com>
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.

Kubernetes configmaps can not be configured as source

2 participants