Skip to content

Commit

Permalink
fix(server): Copy the static health check files into server container (
Browse files Browse the repository at this point in the history
…#3351)

### Motivation

Server container failed v1/health endpoint check because missing the
static files from lambda-tiler.

### Modifications

Copy the file into the server container

### Verification
#### Failure without static files

![image](https://github.com/user-attachments/assets/caad587c-84a7-4f3c-ad6b-5ba4303e4691)
#### Success with static files

![image](https://github.com/user-attachments/assets/03620ad1-dabf-48b7-b18d-690881b3a96b)
  • Loading branch information
Wentao-Kuang authored Sep 29, 2024
1 parent 2aea3c7 commit ecfee1d
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/server/
cp -r packages/lambda-tiler/static/ packages/cli/
- name: Log in to registry
Expand Down
3 changes: 3 additions & 0 deletions packages/server/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,9 @@ WORKDIR /app/basemaps
COPY ./basemaps-landing*.tgz /app/basemaps/
COPY ./basemaps-server*.tgz /app/basemaps/

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

RUN npm install ./basemaps-landing*.tgz
RUN npm install ./basemaps-server*.tgz

Expand Down

0 comments on commit ecfee1d

Please sign in to comment.