Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

slimmed down docker container #356

Merged
merged 1 commit into from
Jun 20, 2024
Merged

Conversation

nexus-uw
Copy link
Contributor

  • use intern container to run webpack + ts build and then copy over the build results to final container
  • moved some build deps over to devDeps to slim down runtime deps in final container
  • added build + dep folders to docker ignore (only matters for building locally, but asserts that only the built files are used in the container)

- use intern container to run webpack + ts build and then copy over the build results to final container
- moved some build deps over to devDeps to slim down runtime deps in final container
- added build + dep folders to docker ignore  (only matters for building locally, but asserts that only the built files are used in the container)
@nexus-uw
Copy link
Contributor Author

result

using the temp build container slimmed 755MB -> 233MB and moving a couple of those build deps to devDeps slimmed it down from 233MB -> 169MB

Screenshot 2024-06-18 14 21 39

testing

docker run -p 0.0.0.0:3001:3001 ass2
visited instance and was able to create a chat session

@muke1908 muke1908 merged commit 7ae76f8 into muke1908:master Jun 20, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants