Skip to content

Commit 5f3e37f

Browse files
authored
BREAKING CHANGE(docker): update note about frontend port change to 8080 (#2015)
## Description Added comment about port change Added correct breaking change title to bump major release version update ## Motivation Background on use case, changes needed ## Fixes: Please provide a list of the fixes implemented in this PR * Items added ## Changes: Please provide a list of the changes implemented by this PR * changes made ## Tests included - [ ] Included for each change/fix? - [ ] Passing? (Merge will not be approved unless this is checked) ## Documentation - [ ] swagger documentation updated \[required\] - [ ] official documentation updated \[nice-to-have\] ### official documentation info If you have updated the official documentation, please provide PR # and URL of the pages where the updates are included ## Backend version - [ ] Does it require a specific version of the backend - which version of the backend is required:
1 parent 4499388 commit 5f3e37f

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

Dockerfile

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,8 @@ RUN npm ci
88
COPY . /frontend/
99
RUN npx ng build
1010

11+
# NOTE: As of Sept 22, frontend is served on port 8080 (previously 80).
12+
# Make sure deployments, ingress, and services reference 8080 instead of 80.
1113
FROM docker.io/nginxinc/nginx-unprivileged:1.26.3
1214
USER root
1315
RUN rm -rf /usr/share/nginx/html/*

0 commit comments

Comments
 (0)