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

[ContainerRegistry][Test] Add weekly tests for USGov Cloud and China Cloud #16292

Merged
merged 8 commits into from
Jul 13, 2021

Conversation

jeremymeng
Copy link
Member

@jeremymeng jeremymeng commented Jul 8, 2021

  • Update test resource deployment
  • Update test utils to pass authorityHost and authenticationScope
  • Update recorder to allow handling different tenant id than default AZURE_TENANT_ID

- Update test resource deployment
@jeremymeng jeremymeng force-pushed the acr-cloud-configuration branch from 0716ef1 to 13aa355 Compare July 8, 2021 23:40
@jeremymeng jeremymeng changed the title [ContainerRegistry] Add cloud configuration support [ContainerRegistry][Test] Add weekly tests for USGov cloud and China Cloud Jul 8, 2021
@jeremymeng jeremymeng changed the title [ContainerRegistry][Test] Add weekly tests for USGov cloud and China Cloud [ContainerRegistry][Test] Add weekly tests for USGov Cloud and China Cloud Jul 8, 2021
@jeremymeng jeremymeng force-pushed the acr-cloud-configuration branch 2 times, most recently from cbfdbd7 to 0387864 Compare July 9, 2021 19:30
as the environment is already the correct one
…t id parameter

with a default value of `env.AZURE_TENANT_ID`.

As found in ACR weekly-tests pipeline with multi-clouds enabled, AZURE_TENANT_ID
and service-specific CONTAINERREGISTRY_TENANT_ID are different so we need to be
able to specify which one to handle (while investigating whether it's by-design
to have different tenant ids.)
@jeremymeng jeremymeng force-pushed the acr-cloud-configuration branch 3 times, most recently from a8b838f to 1bcb455 Compare July 12, 2021 22:51
@jeremymeng jeremymeng marked this pull request as ready for review July 13, 2021 00:29
@jeremymeng
Copy link
Member Author

/azp run js - containerregistry - tests-weekly

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jeremymeng jeremymeng requested a review from maorleger July 13, 2021 19:31
@jeremymeng jeremymeng merged commit baef757 into Azure:main Jul 13, 2021
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.

3 participants