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

Provide Azure Key Vault support in TriggerAuthentication #900

Closed
tomkerkhove opened this issue Jun 25, 2020 · 15 comments · Fixed by #2727
Closed

Provide Azure Key Vault support in TriggerAuthentication #900

tomkerkhove opened this issue Jun 25, 2020 · 15 comments · Fixed by #2727
Assignees
Labels
auth feature All issues for new features that have been committed to help wanted Looking for support from community

Comments

@tomkerkhove
Copy link
Member

Provide Azure Key Vault support in TriggerAuthentication to pull secrets from.

@tomkerkhove tomkerkhove added feature-request All issues for new features that have not been committed to needs-discussion labels Jun 25, 2020
@alexandery
Copy link

+1 on this.

@tomkerkhove tomkerkhove added this to Proposal in Roadmap Oct 13, 2020
@tomkerkhove tomkerkhove moved this from Proposal to Planned (Committed) in Roadmap Jan 20, 2021
@jeffhollan jeffhollan added the help wanted Looking for support from community label Feb 20, 2021
@SebastianSchuetze
Copy link

How would the general approach be? I am a bit confused on how I would need to start on it if I wanted to contribute.

@tomkerkhove
Copy link
Member Author

We provide a contribution guide on https://github.com/kedacore/keda/blob/main/CONTRIBUTING.md but I'd ask @zroubalik @ahmelsayed @JorTurFer for more in-depth guidance.

We should, however, document how to add a new authentication source.

@tomkerkhove
Copy link
Member Author

This should directly talk to Azure Key Vault as Azure Key Vault Provider for Secrets Store CSI Driver is another scenario to cover that should already work today.

@stale
Copy link

stale bot commented Jan 31, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale All issues that are marked as stale due to inactivity label Jan 31, 2022
@tomkerkhove tomkerkhove added the feature All issues for new features that have been committed to label Jan 31, 2022
@stale stale bot removed the stale All issues that are marked as stale due to inactivity label Jan 31, 2022
@tomkerkhove tomkerkhove removed the feature-request All issues for new features that have not been committed to label Jan 31, 2022
@SebastianSchuetze
Copy link

still want this feature and I think it should come!

@tomkerkhove
Copy link
Member Author

We are happy to review PRs ;)

@SebastianSchuetze
Copy link

I know. just wanted to keep this open for now 😜

@tomkerkhove tomkerkhove removed this from Planned (Committed) in Roadmap Feb 10, 2022
@v-shenoy
Copy link
Contributor

I plan on taking this up. @tomkerkhove

@tomkerkhove
Copy link
Member Author

Thank you!

@v-shenoy
Copy link
Contributor

v-shenoy commented Mar 4, 2022

Got a bit busy with some other work, working on this. Commenting so that the issue doesn't get marked as stale.

@tomkerkhove
Copy link
Member Author

No worries, issue won't get stale and take your time - Thanks for the update!

@v-shenoy
Copy link
Contributor

v-shenoy commented Mar 7, 2022

Could you provide me some more resources I could read up on for adding authentication sources? @tomkerkhove

@tomkerkhove
Copy link
Member Author

We don't have any docs on adding auth providers I think, but we should add this.

Or am I mistaken @JorTurFer @zroubalik ?

@zroubalik
Copy link
Member

Nope, it's in the code though 😅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auth feature All issues for new features that have been committed to help wanted Looking for support from community
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

6 participants