Skip to content

Commit

Permalink
fix(cli): Copy static files into cli container for health smoke test. (
Browse files Browse the repository at this point in the history
…#3257)

#### Motivation

Running smoke test inside basemaps container missing static files for
health endpoint test.

#### Modification

Copy the static files into basemaps container.

#### Checklist

_If not applicable, provide explanation of why._

- [ ] Tests updated
- [ ] Docs updated
- [ ] Issue linked in Title
  • Loading branch information
Wentao-Kuang authored May 6, 2024
1 parent f14d0ba commit 9396f60
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 0 deletions.
1 change: 1 addition & 0 deletions .github/workflows/containers.yml
Original file line number Diff line number Diff line change
Expand Up @@ -50,6 +50,7 @@ jobs:
# Files are packed into the base directory
cp *.tgz packages/server/
cp *.tgz packages/cli/
cp -r packages/lambda-tiler/static/ packages/cli/
- name: Log in to registry
run: echo "${{ secrets.GITHUB_TOKEN }}" | docker login ghcr.io -u $ --password-stdin
Expand Down
3 changes: 3 additions & 0 deletions packages/cli/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,9 @@ COPY ./basemaps-landing*.tgz /app/
COPY ./basemaps-cogify*.tgz /app/
COPY ./basemaps-smoke*.tgz /app/

# Copy the static files for v1/health check
COPY ./static/ /app/static/

RUN npm install ./basemaps-landing*.tgz ./basemaps-cogify*.tgz ./basemaps-smoke*.tgz

COPY dist/index.cjs /app/
Expand Down

0 comments on commit 9396f60

Please sign in to comment.