Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enable secrets to be referenced in the install namespace so this can be installed with tools like argo #171

Merged
merged 1 commit into from
Jun 30, 2023

Conversation

jmcshane
Copy link
Contributor

@jmcshane jmcshane commented Jun 7, 2023

This came up in a conversation with a team using Buildkite right now and felt like a good pattern to support.

If this is installed via argo, you would likely have another mechanism that would install the secret in the namespace, such as external secrets operator. We should enable this secret to be pre-provisioned and referenced in the envFrom block by name.

@jmcshane jmcshane force-pushed the feature/external-secret-name branch from 80a7695 to 6587599 Compare June 7, 2023 14:47
Copy link
Contributor

@triarius triarius left a comment

Choose a reason for hiding this comment

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

Thanks, @jmcshane, and apologies for taking so long to review this, everyone.

I suggested a different approach in #167, but this is delivering value from today (or 3 weeks ago 😅) so I think we can merge it now and tweak things later.

@triarius triarius merged commit 6f1cc7a into buildkite:main Jun 30, 2023
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