FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Added support for azure keyvault provider by jore731 · Pull Request #319 · tellerops/teller · GitHub

/ teller Public

Added support for azure keyvault provider - #319

Open
jore731 wants to merge 9 commits into
tellerops:masterfrom
jore731:master
ConversationCommits (9)ChecksFiles changed
Open

Added support for azure keyvault provider#319
jore731 wants to merge 9 commits into
tellerops:masterfrom
jore731:master

Conversation

jore731 commented Apr 25, 2025

Copy link
Copy Markdown

No description provided.

jore731 commented Apr 25, 2025

Copy link
Copy Markdown
Author

Devbox support was also added since used for local development, but not manadatory to be merged. If needed I can remove it

Also nightly branch was conflicting with my computer (arm64 macOS) so moved to stable branch.

ManuInNZ commented Oct 4, 2025

Copy link
Copy Markdown

not sure if there is something blocking this PR but it would be great to have Azure Key Vault support.

Any chance to get this merged ?

ManuInNZ commented Oct 5, 2025

Copy link
Copy Markdown

@jondot / @kaplanelad ?
Please

ManuInNZ reviewed Oct 5, 2025
View reviewed changes
Comment thread teller-providers/src/registry.rs

#[cfg(feature = "azure_keyvault")]
ProviderKind::AzureKeyVault => Box::new(
crate::providers::etcd::Etcd::new(

ManuInNZ Oct 5, 2025

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

I guess the new provider to be used should not be Etcd here. right ?

jore731 commented Oct 5, 2025

Copy link
Copy Markdown
Author

@ManuInNZ as far as I know, the CNCF colleagues marked the project as archived since no support has been given from the developers. I would like to keep it up with the fork if I have the time at some point. Help is welcome!

ManuInNZ commented Oct 5, 2025

Copy link
Copy Markdown

To be fair I am not a pro Rustacean and even qualified beginner but I can read code and make sense.
I have been able to create my own working Azure Key Vault support for teller using Cline but that does not make me Maintainer level.
So not sure how to help from there.

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer
ManuInNZ ManuInNZ left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

Back | FazBrowse Home | New Git URL