Skip to content

[Key Vault] Drop service principal authentication from test pipelines #36270

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.

Sign up for GitHub

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

Closed
mccoyp wants to merge 14 commits into from
Closed

[Key Vault] Drop service principal authentication from test pipelines #36270

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
6484fcf
Update pipelines for non-SP auth
mccoyp Jun 27, 2024
32b2140
Update Administration config
mccoyp Jun 27, 2024
e3a17ee
Update Keys config
mccoyp Jun 27, 2024
898a3bb
Refresh credentials in test-resources-post
mccoyp Jul 29, 2024
9326f0f
Refresh credentials earlier in script
mccoyp Jul 29, 2024
974a43d
Try getting credential with context
mccoyp Jul 30, 2024
f3add32
Sync test-resources with JS to fix deployment auth issues
mccoyp Jul 31, 2024
0e55932
Repair administration's tests.yml
mccoyp Jul 31, 2024
2a4acdd
Try same location as JS
mccoyp Jul 31, 2024
462c3b6
Try aligning Keys tests.yml with JS's
mccoyp Aug 1, 2024
1a722a1
Fix Admin; Target only Keys/Admin
mccoyp Aug 1, 2024
32cb72c
Make multitenant tests compatible with pipelines
mccoyp Aug 2, 2024
b1ed137
Correctly get credentials
mccoyp Aug 2, 2024
dcfb0d5
Fix async credential
mccoyp Aug 3, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
Update pipelines for non-SP auth
  • Loading branch information
@mccoyp
mccoyp committed Aug 1, 2024
commit 6484fcfacf914015781f4742e4185b497463af74
4 changes: 1 addition & 3 deletions sdk/keyvault/tests.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@ extends:
Packages: ${{ parameters.Services }}
SupportedClouds: 'Public,UsGov,China'
TestTimeoutInMinutes: 240
UseFederatedAuth: true
CloudConfig:
Public:
ServiceConnection: azure-sdk-tests
Expand All @@ -36,7 +37,4 @@ extends:
AZURE_TEST_RUN_LIVE: true
AZURE_SKIP_LIVE_RECORDING: 'True'
AZURE_SUBSCRIPTION_ID: $(azure-subscription-id)
AZURE_TENANT_ID: $(aad-azure-sdk-test-tenant-id)
AZURE_CLIENT_ID: $(aad-azure-sdk-test-client-id)
AZURE_CLIENT_SECRET: $(aad-azure-sdk-test-client-secret)
AZURE_CLIENT_OID: $(aad-azure-sdk-test-client-oid)