-
Notifications
You must be signed in to change notification settings - Fork 4.5k
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
[Feature-docker][all-service] refactor the Dockerfile for smaller size #3291
Conversation
* refactor the Dockerfile and build script * ignore some files and directory
docker/build/Dockerfile
Outdated
@@ -1,82 +1,49 @@ | |||
# |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why delete the license header?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added back
# docker build | ||
echo -e "docker build --build-arg VERSION=${VERSION} -t $DOCKER_REPO:${VERSION} $(pwd)/docker/build/\n" | ||
sudo docker build --build-arg VERSION=${VERSION} -t $DOCKER_REPO:${VERSION} $(pwd)/docker/build/ | ||
echo -e "docker build -f docker/build/Dockerfile -t $DOCKER_REPO:${VERSION} $(pwd)\n" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
alter the path?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Because of at the builder phase, copy src should from root path.
Hi, please resolve the conflict.. If there is no update after 7 days, the PR will be closed, and you can reopen it when your problem is resolved. |
… nginx and improve log out (#4529) * [Improvement][Docker] Reduce docker image size * [Improvement][Docker] Improve log out * [Improvement][Docker] Chmod +x to script files * [Improvement][Docker] Remove frontend and fix docker statck deploy * [Fix][Docker] Fix e2e test * [Improvement][Docker] Revert log appender * [Improvement][Docker] Remove stdout log appender
Tips
What is the purpose of the pull request
Brief change log
Verify this pull request
(Please pick either of the following options)
This pull request is code cleanup without any test coverage.