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

Inherit Azure CLI/PS credentials in E2E testing framework #2956

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

adreed-msft
Copy link
Member

@adreed-msft adreed-msft commented Feb 25, 2025

MERGE ORDER

#2958 -> #2956 (this PR) -> #2959

Description

Allows Tenant ID and Resource to be specified in PS cred inheritance, fixes an upcoming breaking change with Az Powershell (image posted later), and allows the new E2E framework to inherit these credentials.

image

How Has This Been Tested?

E2E framework changes tested manually. Persistence changes validated in TestNewE2E/TokenPersistenceSuite/Scenario_InheritCred_Persist

@vibhansa-msft
Copy link
Member

This PR has changes from other PR as well. May be getting first one approved and merging it will make review of this one easier.

@adreed-msft
Copy link
Member Author

This PR has changes from other PR as well. May be getting first one approved and merging it will make review of this one easier.

That is the purpose of the merge order at the top.

@adreed-msft adreed-msft force-pushed the adreed/e2e-test-inherit-auth branch from ce525f8 to c427506 Compare March 2, 2025 02:55
@dphulkar-msft
Copy link
Member

Tests pipeline is failing for this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants