-
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
Open
alexvy86
wants to merge
37
commits into
microsoft:main
Choose a base branch
from
alexvy86:ff-docker-e2e-tests
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
37 commits
Select commit
Hold shift + click to select a range
6634641
Enable whole summary upload
alexvy86 37f3bc3
Make all other settings the same with r11s in AKS
alexvy86 f637360
Remove use of docker-compose.dev.yml file
alexvy86 3e14cc4
Add missing redirect from alfred to nexus in local docker env.
alexvy86 30b1293
Trying out running e2e tests against docker in the pipeline
alexvy86 e512c35
Undo changes to config.json
alexvy86 4445af3
Restore use of docker-compose.dev.yml
alexvy86 63a489c
Conditional env
alexvy86 6e2cd6a
Add project name
alexvy86 1ed3e81
Checkout
alexvy86 2bc9332
Syntax
alexvy86 89ff3bb
Pass checkout true
alexvy86 0a6a9b4
Try using built images from ACR
alexvy86 801bf93
Re-arrange
alexvy86 9eea008
Use actual env: section
alexvy86 9d6ecc0
Auth to contaitner registry
alexvy86 4ea8aa2
Move variable group
alexvy86 ff9a22a
Try hardcoding service connection name
alexvy86 24f4164
Add display name
alexvy86 470d825
Use correct image tags for gitrest and gitssh
alexvy86 65084b4
Skip additionalDockerComposeFiles
alexvy86 36d5ff6
Restore use of docker-compose.dev.yml
alexvy86 ccf8c42
Fix reference to docker compose files
alexvy86 aeef294
Use nginx directly in the main docker compose file and stop using the…
alexvy86 a00697b
Whole summary upload by default in the server config
alexvy86 2f91a31
Make non-routerlicious images configurable
alexvy86 698dbbe
Use x64 architecture instead of arm
alexvy86 67eacd7
Use the in-repo routerlicious config in docker env
alexvy86 a5c84f2
Remove repo checkout for e2e tests
alexvy86 04acba6
Uncomment other stages
alexvy86 7411c26
Merge branch 'main' into ff-docker-e2e-tests
alexvy86 af8b197
Remove deleted parameter
alexvy86 71bc40b
Add dependency on the e2e_docker stage
alexvy86 a44d8b9
Revert "Remove repo checkout for e2e tests"
alexvy86 18a1ae0
Pass parameter again
alexvy86 21fcbc7
Merge branch 'main' into ff-docker-e2e-tests
alexvy86 16d7a1a
Cleanup
alexvy86 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,18 +1,34 @@ | ||
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:* | ||
|
@@ -63,15 +79,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 | ||
|
@@ -80,7 +98,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: | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.