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

Add support for existing credentials #98

Open
otherguy opened this issue May 20, 2022 · 4 comments
Open

Add support for existing credentials #98

otherguy opened this issue May 20, 2022 · 4 comments
Labels
tracked Issue is tracked in 1Password's internal ticketing system as well.

Comments

@otherguy
Copy link

Summary

It would be great if we could specify e.g. operator.existingTokenSecret and connect.existingCredentialsSecret when installing the Helm chart

Use cases

I use this with ArgoCD and don't want the credentials/token in my Git repository.

Proposed solution

Specifying the name of a Secret with the existing token should negate the need to specify token/credentials on every install.

@edif2008 edif2008 added the tracked Issue is tracked in 1Password's internal ticketing system as well. label May 30, 2022
@onedr0p
Copy link

onedr0p commented Jun 17, 2022

This would be awesome, but it seems like the maintenance of this repo is lacking :/

@nklmilojevic
Copy link

Same problem with our usage at MailerLite. We imported it into Flux, but as soon as we updated the version, it broke cause it requires files each time. Please provide a mechanism for using an existing secret.

@Apollorion
Copy link

I know this is old, so it may just be outdated. But I installed this no problem with a existing credentials (also using flux). If you just dont set connect.credentials or operator.token.value neither secret will be created and you can use an existing secret.

@jillianwilson
Copy link
Contributor

jillianwilson commented May 26, 2023

Apologies for the slow movement on this issue, we have tracked this internally to look into adding in the future. We will update you when this has been implemented. In the meantime, is the solution that @Apollorion provided satisfactory?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tracked Issue is tracked in 1Password's internal ticketing system as well.
Projects
None yet
Development

No branches or pull requests

6 participants