iceberg icon indicating copy to clipboard operation
iceberg copied to clipboard

Azure: KeyManagementClient implementation for Azure Key Vault

Open nandorKollar opened this issue 7 months ago • 15 comments

KeyManagement client implementation for connecting to Azure Key Vault. Unlike other key managers (like KMS on AWS), it seems that Key Vault doesn't support key generation, therefore supportsKeyGeneration returns false.

nandorKollar avatar May 30 '25 07:05 nandorKollar

@szlta you might be interested in this change, as you did similar for AWS and GCP.

nandorKollar avatar Jun 17 '25 15:06 nandorKollar

@nastra @amogh-jahagirdar would you mind have a look at this PR?

nandorKollar avatar Jul 04 '25 14:07 nandorKollar

This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 1 week if no further activity occurs. If you think that’s incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the [email protected] list. Thank you for your contributions.

github-actions[bot] avatar Aug 04 '25 00:08 github-actions[bot]

@amogh-jahagirdar / @nastra could you please take a look on this PR?

szlta avatar Aug 08 '25 11:08 szlta

This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 1 week if no further activity occurs. If you think that’s incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the [email protected] list. Thank you for your contributions.

github-actions[bot] avatar Sep 22 '25 00:09 github-actions[bot]

not stale

nandorKollar avatar Sep 26 '25 13:09 nandorKollar

@amogh-jahagirdar @nastra any change for this to get reviewed/merged? Azure is the last one which doesn't have a KeyManagementClient implementation, both AWS and GCP already has it. It would be great to make key management implementation work on Azure too.

nandorKollar avatar Oct 27 '25 17:10 nandorKollar

Thanks @nastra for the review, could you please also merge it, or is there anything else I should do?

nandorKollar avatar Nov 17 '25 19:11 nandorKollar

Thanks @nastra for the review, could you please also merge it, or is there anything else I should do?

I'd like to have some additional eyes on this before getting this in. @amogh-jahagirdar or @danielcweeks could any of you please review this one?

nastra avatar Nov 18 '25 10:11 nastra