feat: docker deployment #16
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
… and development, added steps on how to run it into README.md
Owner
|
It's great, there's no need to migrate db. Because I'll do an automatic migration. I'll write a Nitrojs plugin. |
Contributor
Author
|
Okay get it, that way it's even better. |
Owner
|
dockerhub first test please check. |
- Add GitHub Actions workflow for automated Docker Hub publishing - Increase Node.js heap size to fix build memory issues - Update docker-compose.yaml to use published image - Support multi-platform builds (linux/amd64, linux/arm64)
Owner
|
Please check the codes I sent again and adjust them as you wish, then let's proceed. |
- Add comprehensive .dockerignore rules to exclude unnecessary files - Reduce build context from ~5.59MB to ~534KB (90% reduction) - Exclude Git, documentation, test files, and mobile SDKs from Docker image - Improve build performance and security by filtering sensitive files - Maintain required files like package.json and pnpm-lock.yaml for builds
Contributor
Author
|
Okay I will check it out. |
… and development, added steps on how to run it into README.md
- Add GitHub Actions workflow for automated Docker Hub publishing - Increase Node.js heap size to fix build memory issues - Update docker-compose.yaml to use published image - Support multi-platform builds (linux/amd64, linux/arm64)
- Add comprehensive .dockerignore rules to exclude unnecessary files - Reduce build context from ~5.59MB to ~534KB (90% reduction) - Exclude Git, documentation, test files, and mobile SDKs from Docker image - Improve build performance and security by filtering sensitive files - Maintain required files like package.json and pnpm-lock.yaml for builds
# Conflicts: # README.md
Contributor
Author
|
I went through the github actions yaml file you added. It is pretty neat, something that I wanted to make as a next step. I have removed the migrate container from my docker compose as well as the migration stage in Dockerfile. |
Owner
|
Great, thank you ❤️ |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
It took me a while as I haven't been doing anykind of a similar project to understand how Nitro, Nuxt works. Although I have successfully been able to create docker compose with profiles for different usecases and a multistage Dockerfile. I really enjoyed doing it. Thanks for experience and I look forward for next contributions and hope you like this one.
See below for changes.
Docker support and configuration:
Dockerfilewith separate targets for development, production, and database migration, enabling efficient builds and environment-specific containers.docker-compose.yamlfile to orchestrate services for Postgres, Redis, development, production, and migration, with appropriate environment variables, volumes, and dependencies..dockerignorefile to exclude unnecessary files and directories (likenode_modules,.idea,tests,.nuxt, andios) from Docker build context, reducing image size.Environment and documentation updates:
.env.exampleto include required Postgres environment variables (POSTGRES_USER,POSTGRES_PASSWORD,POSTGRES_DB) for Docker Compose compatibility.README.mdwith detailed instructions for running the application using Docker and Docker Compose, including migration and production deployment steps.