Skip to content

Commit

Permalink
[Event Hubs Client] Test Configuration Adjustments (#20888)
Browse files Browse the repository at this point in the history
The focus of these changes is to adjust the clouds used for nightly and
manual test runs, reverting to the Public and Canary clouds.  Weekly runs
will include the UsGov and China sovereign clouds as well.   After discussion,
this aligns with the configuration that the engineering team intended.

Due to the China cloud performing slowly during runs, we've seen some tests
start to exceed the previous execution time cap; this cap has been increased
to reduce flakiness in weekly runs.  I'll continue to monitor and adjust as we
learn more about test stability and performance in the sovereign clouds.
  • Loading branch information
jsquire authored May 6, 2021
1 parent fa061ce commit 4d8ec84
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 3 deletions.
2 changes: 1 addition & 1 deletion sdk/eventhub/test-resources.json
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,7 @@
},
"perTestExecutionLimitMinutes": {
"type": "string",
"defaultValue": "10",
"defaultValue": "15",
"metadata": {
"description": "The maximum duration, in minutes, that a single test is permitted to run before it is considered at-risk for being hung."
}
Expand Down
3 changes: 2 additions & 1 deletion sdk/eventhub/tests.data.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,4 +7,5 @@ extends:
ServiceDirectory: eventhub
SDKType: data
TimeoutInMinutes: 240
Clouds: 'Public,UsGov,China,Canary'
Clouds: 'Public,Canary'
SupportedClouds: 'Public,UsGov,China,Canary'
3 changes: 2 additions & 1 deletion sdk/eventhub/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,4 +7,5 @@ extends:
ServiceDirectory: eventhub
SDKType: client
TimeoutInMinutes: 240
Clouds: 'Public,UsGov,China,Canary'
Clouds: 'Public,Canary'
SupportedClouds: 'Public,UsGov,China,Canary'

0 comments on commit 4d8ec84

Please sign in to comment.