-
Notifications
You must be signed in to change notification settings - Fork 531
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
feat(ci): Add e2e tests against latest docker images #22576
base: main
Are you sure you want to change the base?
Changes from 30 commits
6634641
37f3bc3
f637360
3e14cc4
30b1293
e512c35
4445af3
63a489c
6e2cd6a
1ed3e81
2bc9332
89ff3bb
0a6a9b4
801bf93
9eea008
9d6ecc0
4ea8aa2
ff9a22a
24f4164
470d825
65084b4
36d5ff6
ccf8c42
aeef294
a00697b
2f91a31
698dbbe
67eacd7
a5c84f2
04acba6
7411c26
af8b197
71bc40b
a44d8b9
18a1ae0
21fcbc7
16d7a1a
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,19 +1,35 @@ | ||
version: "3.4" | ||
services: | ||
proxy: | ||
platform: linux/amd64/v3 | ||
image: nginx:latest | ||
depends_on: | ||
- "alfred" | ||
- "historian" | ||
- "nexus" | ||
volumes: | ||
- ./routerlicious/nginx.conf:/etc/nginx/nginx.conf | ||
ports: | ||
- "3003:3003" | ||
- "3002:3002" | ||
- "3001:3001" | ||
alfred: | ||
image: ${REGISTRY_URL:-mcr.microsoft.com}/fluidframework/routerlicious/server:${ALFRED_IMAGE_TAG:-latest} | ||
ports: | ||
- "3003:3000" | ||
expose: | ||
- "3000" | ||
command: node packages/routerlicious/dist/alfred/www.js | ||
environment: | ||
- DEBUG=fluid:* | ||
- NODE_ENV=development | ||
- IS_FLUID_SERVER=true | ||
restart: always | ||
# THIS VOLUMES IS FOR LOCAL TESTING, IT SHOULD NOT GET MERGED | ||
volumes: | ||
- ./routerlicious/packages/routerlicious/config/config.json:/usr/src/server/packages/routerlicious/config/config.json | ||
nexus: | ||
image: ${REGISTRY_URL:-mcr.microsoft.com}/fluidframework/routerlicious/server:${ALFRED_IMAGE_TAG:-latest} | ||
ports: | ||
- "3002:3000" | ||
expose: | ||
- "3000" | ||
command: node packages/routerlicious/dist/nexus/www.js | ||
environment: | ||
- DEBUG=fluid:* | ||
|
@@ -64,15 +80,17 @@ services: | |
restart: always | ||
historian: | ||
image: ${REGISTRY_URL:-mcr.microsoft.com}/fluidframework/routerlicious/historian:${HISTORIAN_IMAGE_TAG:-latest} | ||
ports: | ||
- "3001:3000" | ||
expose: | ||
- "3000" | ||
environment: | ||
- DEBUG=fluid:* | ||
- NODE_ENV=development | ||
- IS_FLUID_SERVER=true | ||
# volumes: | ||
# - ./routerlicious/packages/routerlicious/config/config.json:/home/node/server/packages/routerlicious/config.json | ||
Comment on lines
+88
to
+89
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Will remove too, this was also for local testing. |
||
restart: always | ||
gitrest: | ||
image: ${REGISTRY_URL:-mcr.microsoft.com}/fluidframework/routerlicious/gitrest:${HISTORIAN_IMAGE_TAG:-latest} | ||
image: ${REGISTRY_URL:-mcr.microsoft.com}/fluidframework/routerlicious/gitrest:${GITREST_IMAGE_TAG:-latest} | ||
environment: | ||
- DEBUG=fluid:* | ||
- NODE_ENV=development | ||
|
@@ -81,7 +99,7 @@ services: | |
- git:/home/node/documents | ||
restart: always | ||
git: | ||
image: ${REGISTRY_URL:-mcr.microsoft.com}/fluidframework/routerlicious/gitssh:${HISTORIAN_IMAGE_TAG:-latest} | ||
image: ${REGISTRY_URL:-mcr.microsoft.com}/fluidframework/routerlicious/gitssh:${GITSSH_IMAGE_TAG:-latest} | ||
ports: | ||
- "3022:22" | ||
volumes: | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -77,6 +77,10 @@ parameters: | |
type: boolean | ||
default: false | ||
|
||
- name: preSteps | ||
type: stepList | ||
default: [] | ||
|
||
jobs: | ||
- ${{ each variant in parameters.splitTestVariants }}: | ||
- job: | ||
|
@@ -174,6 +178,8 @@ jobs: | |
|
||
- template: include-use-node-version.yml | ||
|
||
- ${{ parameters.preSteps }} | ||
|
||
# Download artifact | ||
- task: DownloadPipelineArtifact@2 | ||
displayName: Download test package | ||
|
@@ -341,8 +347,9 @@ jobs: | |
- task: Npm@1 | ||
displayName: '[test] ${{ parameters.testCommand }} ${{ variant.flags }}' | ||
continueOnError: ${{ parameters.continueOnError }} | ||
env: | ||
${{ parameters.env }} | ||
${{ if parameters.env }}: | ||
env: | ||
${{ parameters.env }} | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The new stage doesn't use it right now, and if nothing is passed this ends up being a syntax error with the current code. |
||
inputs: | ||
command: 'custom' | ||
workingDir: ${{ parameters.testWorkspace }}/node_modules/${{ parameters.testPackage }} | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Used this locally to validate running e2e tests against the latest server image, but with whole summary upload enabled (the image itself doesn't do it). This will go away before merging.