Skip to content

Commit

Permalink
Bug fix to GitHub CI build. Set compose project name via env variable…
Browse files Browse the repository at this point in the history
…. Also add default name if unspecified
  • Loading branch information
tdonohue committed Feb 5, 2024
1 parent 2c2f59c commit 9043f63
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 1 deletion.
2 changes: 2 additions & 0 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,8 @@ jobs:
#CHROME_VERSION: "90.0.4430.212-1"
# Bump Node heap size (OOM in CI after upgrading to Angular 15)
NODE_OPTIONS: '--max-old-space-size=4096'
# Project name to use when running docker-compose prior to e2e tests
COMPOSE_PROJECT_NAME: 'ci'
strategy:
# Create a matrix of Node versions to test against (in parallel)
matrix:
Expand Down
3 changes: 2 additions & 1 deletion docker/cli.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,8 +16,9 @@ version: "3.7"
networks:
# Default to using network named 'dspacenet' from docker-compose-rest.yml.
# Its full name will be prepended with the project name (e.g. "-p d7" means it will be named "d7_dspacenet")
# If COMPOSITE_PROJECT_NAME is missing, default value will be "docker" (name of folder this file is in)
default:
name: ${COMPOSE_PROJECT_NAME}_dspacenet
name: ${COMPOSE_PROJECT_NAME:-docker}_dspacenet
external: true
services:
dspace-cli:
Expand Down

0 comments on commit 9043f63

Please sign in to comment.