Skip to content

Commit 5478bb3

Browse files
authored
Use updated tenant steps in stress pipeline (#25437)
Use the new tenants introduced in #23382 in the stress test pipeline for the ODSP prod stage. Largely follows the same steps as the E2E change and uses the infrastructure from that PR. This only deals with ODSP prod - the ODSPDF stage will be updated in a follow-up. [AB#48372](https://dev.azure.com/fluidframework/235294da-091d-4c29-84fc-cdfc3d90890b/_workitems/edit/48372)
1 parent 9c6b44b commit 5478bb3

File tree

2 files changed

+5
-7
lines changed

2 files changed

+5
-7
lines changed

tools/pipelines/templates/include-test-real-service.yml

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -111,7 +111,7 @@ parameters:
111111

112112
stages:
113113
- stage: ${{ parameters.stageId}}
114-
${{ if in(parameters.stageId, 'e2e_routerlicious', 'e2e_frs', 'stress_tests_odsp', 'stress_tests_odspdf', 'stress_tests_frs', 'stress_tests_frs_canary') }}:
114+
${{ if in(parameters.stageId, 'e2e_routerlicious', 'e2e_frs', 'stress_tests_odspdf', 'stress_tests_frs', 'stress_tests_frs_canary') }}:
115115
lockBehavior: sequential
116116
condition: ${{ parameters.condition }}
117117
displayName: ${{ parameters.stageDisplayName }}
@@ -347,7 +347,7 @@ stages:
347347
path: ${{ parameters.testWorkspace }}/node_modules/@fluid-private/test-version-utils/node_modules/.legacy/
348348

349349
# Only check out tenants from the tenant pool if we are running tests against ODSP
350-
- ${{ if eq(parameters.stageId, 'e2e_odsp') }}:
350+
- ${{ if or(eq(parameters.stageId, 'e2e_odsp'), eq(parameters.stageId, 'stress_tests_odsp')) }}:
351351
# Retrieve a tenant from the tenant pool
352352
- task: AzureCLI@2
353353
displayName: 'Log in to retrieve tenant credentials'
@@ -381,9 +381,9 @@ stages:
381381
displayName: '[test] ${{ parameters.testCommand }} ${{ variant.flags }}'
382382
continueOnError: ${{ parameters.continueOnError }}
383383
env:
384-
${{ if ne(parameters.stageId, 'e2e_odsp') }}:
384+
${{ if and(ne(parameters.stageId, 'e2e_odsp'), ne(parameters.stageId, 'stress_tests_odsp')) }}:
385385
${{ parameters.env }}
386-
${{ if eq(parameters.stageId, 'e2e_odsp') }}:
386+
${{ else }}:
387387
login__odsp__test__tenants: $(tenantCreds)
388388
login__microsoft__clientId: $(appClientId)
389389
inputs:
@@ -437,7 +437,7 @@ stages:
437437
continueOnError: true # Keep running subsequent tasks even if this one fails (e.g. the tinylicious log wasn't there)
438438

439439
# Only release tenants that were checked out from the tenant pool for ODSP tests
440-
- ${{ if eq(parameters.stageId, 'e2e_odsp') }}:
440+
- ${{ if or(eq(parameters.stageId, 'e2e_odsp'), eq(parameters.stageId, 'stress_tests_odsp')) }}:
441441
# Login to release tenant credentials
442442
# Currently, some of the compat tests run for longer than 60 minutes and exceed the average token lifetime:
443443
# https://learn.microsoft.com/en-us/entra/identity-platform/configurable-token-lifetimes#access-tokens

tools/pipelines/test-real-service-stress.yml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -50,8 +50,6 @@ stages:
5050
testCommand: start:odsp
5151
skipTestResultPublishing: true
5252
pipelineIdentifierForTelemetry: ${{ variables.pipelineIdentifierForTelemetry }}
53-
stageVariables:
54-
- group: stress-odsp-lock
5553
env:
5654
login__microsoft__clientId: $(login-microsoft-clientId)
5755
login__odsp__test__tenants: $(automation-stress-login-odsp-test-tenants)

0 commit comments

Comments
 (0)