azure-sdk-for-python: [Service Bus]"Preparer failure when creating resource ServiceBusNamespacePreparer, Resource group could not be found" on UsGov and China cloud
We are running live Tests against other clouds like US Gov and Azure China Cloud. The goal is to check whether new azure sdk package work with other clouds or not.
Error Description:
When running the test tests\test_queues.py::ServiceBusQueueTests::test_github_issue_6178 on UsGov and China cloud, it runs failed and the error message is shown as following, for more details please check here:
Error Track:
The code that failed is here：
Note: We guessed that the environment variable did not set the value of RESOURCE_REGION, so the default value ‘westus’ was used, so the resource group was not created successfully in UsGov and China Cloud.
Expected Behavior: The resource group was successfully created and it can be found.
@benbp , @jameszliao-msft , @lmazuel , @lilyjma , @ramya-rao-a and @annatisch for notification.
About this issue
- Original URL
- State: closed
- Created 3 years ago
- Comments: 35 (35 by maintainers)
Commits related to this issue
- CodeGen from PR 20227 in Azure/azure-rest-api-specs [Microsoft.StorageSync] New API version 2022-06-01 (#20227) * Adds base for updating Microsoft.StorageSync from version stable/2020-09-01 to versio... — committed to azure-sdk/azure-sdk-for-python by deleted user 2 years ago
- [ServiceBus] enable sovereign cloud testing (#29202) following PR #21715 fixes: #20517 — committed to Azure/azure-sdk-for-python by swathipil a year ago
Hi @v-xuto - I have a PR to fix this and all the tests are passing: https://github.com/Azure/azure-sdk-for-python/pull/29202
However, CI is failing b/c of a dependency conflict. I have a PR out to fix that dependency conflict. Hoping we can merge all the PRs and close this issue by today. Thanks for your patience!
@v-xuto I have a few more tests to fix before merging the PR. Meanwhile, feel free to test it on my branch. You’d be seeing some test failures, but no set up failures.
cc: @kashifkhan
@v-xuto I have a PR out - i’m hoping to get it merged by next week https://github.com/Azure/azure-sdk-for-python/pull/23566
@v-xuto im currently working on getting rid of it completely and using the arm template. That will fix this issue. It should be done in a couple of weeks. Sorry for the late update.
@yunhaoling let’s just move to ARM template?
@v-xuto I think we should change places in the test framework that are causing problems, and we should not be deploying via the test code, only the ARM template. I think this is mostly done across the python repository but there are still a few places that use the legacy deployment method. @yunhaoling do you know if/where this work is tracked, or am I wrong?
@scbedd Yeah, currently I have tested the formrecognizer, eventhubs, textanalytics and appconfig, none of them have this problem. @benbp Is there anything to add?