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

[Identity] Update live test config #35019

Merged
merged 1 commit into from
Apr 1, 2024
Merged

Conversation

pvaneck
Copy link
Member

@pvaneck pvaneck commented Mar 30, 2024

This should hopefully use the shared identity keyvault and fix our failing identity live tests pipeline. I refactored the CertificateCredential live tests to only conditionally try using the password protected certs as there aren't any in the shared identity sub config yet.

@pvaneck
Copy link
Member Author

pvaneck commented Mar 30, 2024

/azp run python - identity - tests

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@azure-sdk
Copy link
Collaborator

API change check

API changes are not detected in this pull request.

@pvaneck pvaneck force-pushed the identity-live-certs branch from 96c7d54 to d68aef6 Compare April 1, 2024 06:45
@pvaneck
Copy link
Member Author

pvaneck commented Apr 1, 2024

/azp run python - identity - tests

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@pvaneck pvaneck force-pushed the identity-live-certs branch 3 times, most recently from 41c97a3 to 0c941f5 Compare April 1, 2024 08:08
Signed-off-by: Paul Van Eck <paulvaneck@microsoft.com>
@pvaneck pvaneck force-pushed the identity-live-certs branch from 0c941f5 to 96cdd1a Compare April 1, 2024 08:28
@pvaneck pvaneck marked this pull request as ready for review April 1, 2024 18:01
@pvaneck pvaneck requested review from xiangyan99 and a team as code owners April 1, 2024 18:01
@pvaneck pvaneck merged commit 1ade699 into Azure:main Apr 1, 2024
43 checks passed
@pvaneck pvaneck deleted the identity-live-certs branch April 1, 2024 18:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

3 participants