Skip to content

Update docs/example for ECR token generator#4773

Merged
Skarlso merged 4 commits intoexternal-secrets:mainfrom
Moulick:patch-1
May 14, 2025
Merged

Update docs/example for ECR token generator#4773
Skarlso merged 4 commits intoexternal-secrets:mainfrom
Moulick:patch-1

Conversation

@Moulick
Copy link
Copy Markdown
Contributor

@Moulick Moulick commented May 12, 2025

With this templating, it will generate an actually usable secret that is of type dockerconfig and in the proper expected format

Problem Statement

The example for ECR token generator is incomplete. The current example creates a secret that is unusable as imagePullSecrets

Related Issue

Fixes #... None

Proposed Changes

Update the docs/example to make it easier for people to copy paste

Checklist

@Moulick Moulick changed the title update example for ECR token generator Update docs/example for ECR token generator May 12, 2025
@Moulick Moulick marked this pull request as ready for review May 12, 2025 17:13
@Moulick Moulick requested a review from a team as a code owner May 12, 2025 17:13
@Moulick Moulick requested a review from moolen May 12, 2025 17:13
@Moulick Moulick marked this pull request as draft May 13, 2025 09:50
With this templating, it will generate an actually usable secret that is of type dockerconfig and in the proper expected format

Signed-off-by: Moulick Aggarwal <15780903+Moulick@users.noreply.github.com>
@Moulick Moulick marked this pull request as ready for review May 13, 2025 12:18
@Moulick
Copy link
Copy Markdown
Contributor Author

Moulick commented May 13, 2025

@gusfcarvalho Can you please approve running the workflows as well?

@gusfcarvalho
Copy link
Copy Markdown
Member

@Moulick sure! but our workflows do not do docs checks, so this will likely skip most of the checks

@Skarlso Skarlso merged commit 5130dcb into external-secrets:main May 14, 2025
2 checks passed
@sonarqubecloud
Copy link
Copy Markdown

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.

3 participants