Skip to content

Smoke Tests: enable using other clouds #10200

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

Merged
merged 8 commits into from
Mar 14, 2020

Conversation

danieljurek
Copy link
Member

@danieljurek danieljurek commented Mar 9, 2020

Adds ability to specify an authority host via environment variables. This can be used to authenticate with other clouds.

…requires hostname without protocol wrapping)
…ic value because python uses just the host and doesn't use the URL like other languages)
@danieljurek danieljurek added the EngSys This issue is impacting the engineering system. label Mar 9, 2020
@adxsdk6
Copy link

adxsdk6 commented Mar 9, 2020

Can one of the admins verify this patch?

@danieljurek danieljurek changed the title Smoke tests authority host Smoke Tests: enable using other clouds Mar 9, 2020
@danieljurek
Copy link
Member Author

/azp run python - smoke-tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@danieljurek
Copy link
Member Author

/azp run python - smoke-tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@danieljurek
Copy link
Member Author

/azp run python - smoke-tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@danieljurek
Copy link
Member Author

/azp run java - smoke-tests

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@danieljurek
Copy link
Member Author

/azp run python- smoke-tests

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@danieljurek
Copy link
Member Author

/azp run python - smoke-tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@danieljurek danieljurek requested a review from jongio March 13, 2020 07:43
@danieljurek danieljurek marked this pull request as ready for review March 13, 2020 07:44
@danieljurek danieljurek merged commit d320a5a into Azure:master Mar 14, 2020
chenjianfei2017 pushed a commit that referenced this pull request Mar 17, 2020
* fix async tests and install async requirements

* fix Identity for KV to support AZURE_AUTHORITY_HOST with a hostname (requires hostname without protocol wrapping)

* add AZURE_AUTHORITY_HOST to tests.yml (this one uses a python-specific value because python uses just the host and doesn't use the URL like other languages)

* update naming for authority host variable

* add ability to use gov cloud

* public -> gov for gov cloud instances, tweaks to async event hubs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
EngSys This issue is impacting the engineering system.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants