-
Notifications
You must be signed in to change notification settings - Fork 5
feat(postgres): modify the docker base image as postgres:11-alpine #5
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
Conversation
1d1d8a0 to
e1b0c3b
Compare
|
Looks good! It's exciting that it's only 1/3 the size of the original postgres image. @duanhongyi , is this ready for review and merge? |
yes |
till
left a comment
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.
Mostly a few questions.
Nice work! The size improvement will be significant!
till
left a comment
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.
Thanks for responding to all my questions.
Good to merge, IMO!
34e3648 to
ea21ee1
Compare
|
close, see #7 |
|
@duanhongyi , I'm going to reopen merge this PR as it has better history for the approval process. |
|
Merging this in now and going to build a new tag v2.7.1 so we can make Hephy Workflow v2.20 start up faster @kingdonb @duanhongyi 😁 |
Cryptophobia
left a comment
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.
LGTM
|
Yay!!! :D |
Using postgres:11-alpine instead of the postgres:11, the size is close to the original 1/3.