Skip to content

Commit

Permalink
[Smoke Tests] Add Timeout, Update to March Releases (Azure#19321)
Browse files Browse the repository at this point in the history
The focus of these changes is to add the standard timeout parameter used
in other archtypes to the Smoke Tests template and configure the main
job to respect it.
  • Loading branch information
jsquire authored Mar 10, 2021
1 parent c54a824 commit f9b5676
Show file tree
Hide file tree
Showing 4 changed files with 11 additions and 6 deletions.
8 changes: 4 additions & 4 deletions common/SmokeTests/SmokeTest/SmokeTest.csproj
Original file line number Diff line number Diff line change
Expand Up @@ -11,10 +11,10 @@
</ItemGroup>
<ItemGroup>
<!-- Add an OverrideDailyVersion attribute to prevent the Update-Dependencies script from overwriting it with a daily build version -->
<PackageReference Include="Azure.Core" Version="1.9.0" />
<PackageReference Include="Azure.Identity" Version="1.4.0-beta.3" />
<PackageReference Include="Azure.Messaging.EventHubs" Version="5.3.0" />
<PackageReference Include="Azure.Messaging.EventHubs.Processor" Version="5.3.0" />
<PackageReference Include="Azure.Core" Version="1.10.0" />
<PackageReference Include="Azure.Identity" Version="1.4.0-beta.4" />
<PackageReference Include="Azure.Messaging.EventHubs" Version="5.3.1" />
<PackageReference Include="Azure.Messaging.EventHubs.Processor" Version="5.3.1" />
<PackageReference Include="Azure.Security.Keyvault.Secrets" Version="4.2.0-beta.4" />
<PackageReference Include="Azure.Storage.Blobs" Version="12.9.0-beta.1" />
<PackageReference Include="Microsoft.Azure.Amqp" Version="2.4.11" />
Expand Down
2 changes: 1 addition & 1 deletion common/SmokeTests/smoke-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,4 +5,4 @@ jobs:
- template: /eng/pipelines/templates/jobs/smoke-tests.yml
parameters:
Daily: true
TimeoutInMinutes: 240
TimeoutInMinutes: 90
5 changes: 5 additions & 0 deletions eng/pipelines/templates/jobs/smoke-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,9 @@ parameters:
- name: ArtifactName
type: string
default: ""
- name: TimeoutInMinutes
type: number
default: 60

jobs:
- ${{ if eq(parameters.Daily, false) }}:
Expand Down Expand Up @@ -110,6 +113,8 @@ jobs:
azureUSGovernmentArmParameters: "@{ storageEndpointSuffix = 'core.usgovcloudapi.net'; azureCloud = 'AzureUSGovernment'; }"
azureChinaCloudArmParameters: "@{ storageEndpointSuffix = 'core.chinacloudapi.cn'; azureCloud = 'AzureChinaCloud'; }"

timeoutInMinutes: ${{ parameters.TimeoutInMinutes }}

pool:
name: $(Pool)
vmImage: $(OSVmImage)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
</ItemGroup>

<ItemGroup>
<PackageReference Include="Azure.Messaging.EventHubs" Version="5.3.0" />
<PackageReference Include="Azure.Messaging.EventHubs" Version="5.3.1" />
<PackageReference Include="Microsoft.Azure.Amqp" Version="2.4.11" />
<PackageReference Include="Microsoft.Azure.Devices" Version="1.20.1" />
</ItemGroup>
Expand Down

0 comments on commit f9b5676

Please sign in to comment.