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

Docker-Multi Stage Build #770

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

Dishant1804
Copy link
Collaborator

Resolves #734

Modified Dockerfiles that previously lacked support for multi-stage builds, now the necessary files support mullti-stage builds

@github-actions github-actions bot added backend frontend docker Pull requests that update Docker code makefile labels Feb 7, 2025
@yashgoyal0110
Copy link
Contributor

@Dishant1804
Just wanted to know, have you compared build size and build time after making changes?

@yashgoyal0110
Copy link
Contributor

@Dishant1804 any update on this, are you still looking into it?

@Dishant1804
Copy link
Collaborator Author

@yashgoyal0110 let @arkid15r reply with the correct approach with the ci/cd failure that is happening, then i am more than happy to share the metrics, also i was busy for couple of days as we have personal lives too. I would kindly request you to please have a bit patience :)

@arkid15r arkid15r marked this pull request as draft February 23, 2025 22:21
@Dishant1804
Copy link
Collaborator Author

Dishant1804 commented Feb 24, 2025

@arkid15r made the changes, i let the poetry be installed in the runtime stage of test file inorder to run pytest :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend docker Pull requests that update Docker code frontend
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement docker multi-stage build
4 participants