🐛(summary) fix hot reloading in tilt stack for summary image/pod
Remove default unprivileged Docker user that was incompatible with hot reloading in tilt stack. Update tilt config to resolve path issues. CI builds still use unprivileged user, making this change safe while enabling proper development workflow with hot reloading functionality.
This commit is contained in:
committed by
aleb_the_flash
parent
a6aa77cb97
commit
31ac4cd767
@@ -58,10 +58,10 @@ docker_build(
|
||||
'localhost:5001/meet-summary:latest',
|
||||
context='../src/summary',
|
||||
dockerfile='../src/summary/Dockerfile',
|
||||
only=['.', '../../docker', '../../.dockerignore'],
|
||||
only=['.'],
|
||||
target = 'production',
|
||||
live_update=[
|
||||
sync('../src/summary', '/home/summary'),
|
||||
sync('../src/summary', '/app'),
|
||||
]
|
||||
)
|
||||
clean_old_images('localhost:5001/meet-summary')
|
||||
|
||||
@@ -12,11 +12,11 @@ FROM base AS production
|
||||
|
||||
WORKDIR /app
|
||||
|
||||
# Un-privileged user running the application
|
||||
ARG DOCKER_USER
|
||||
USER ${DOCKER_USER}
|
||||
|
||||
COPY --from=builder /usr/local /usr/local
|
||||
COPY ./summary /app/summary
|
||||
|
||||
# Un-privileged user running the application
|
||||
ARG DOCKER_USER=1000:1000
|
||||
USER ${DOCKER_USER}
|
||||
|
||||
CMD ["uvicorn", "summary.main:app", "--proxy-headers", "--host", "0.0.0.0", "--port", "8000"]
|
||||
|
||||
Reference in New Issue
Block a user