Skip to content

[KeyVault] Adding performance tests#17492

Merged
seankane-msft merged 11 commits intoAzure:mainfrom
seankane-msft:kv-perf
Apr 14, 2022
Merged

[KeyVault] Adding performance tests#17492
seankane-msft merged 11 commits intoAzure:mainfrom
seankane-msft:kv-perf

Conversation

@seankane-msft
Copy link
Copy Markdown
Contributor

closes #17362

@seankane-msft seankane-msft self-assigned this Apr 8, 2022
@ghost ghost added the KeyVault label Apr 8, 2022
@seankane-msft seankane-msft marked this pull request as draft April 8, 2022 19:58
@seankane-msft seankane-msft added the pillar-performance The issue is related to performance, one of our core engineering pillars. label Apr 8, 2022
@seankane-msft seankane-msft marked this pull request as ready for review April 8, 2022 21:09
Copy link
Copy Markdown
Member

@heaths heaths left a comment

Choose a reason for hiding this comment

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

A few suggestions, but otherwise LGTM.

Comment thread sdk/keyvault/azkeys/testdata/perf/decrypt.go Outdated
Comment thread sdk/keyvault/azkeys/testdata/perf/decrypt.go
Comment thread sdk/keyvault/azkeys/testdata/perf/get_key.go Outdated
Comment thread sdk/keyvault/azkeys/testdata/perf/unwrap.go
@seankane-msft seankane-msft merged commit 47b0fa8 into Azure:main Apr 14, 2022
@seankane-msft seankane-msft deleted the kv-perf branch April 14, 2022 17:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

KeyVault pillar-performance The issue is related to performance, one of our core engineering pillars.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Perf] Add performance tests for KeyVault

3 participants