-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
doc: add multi stage build to the native guide #2859
doc: add multi stage build to the native guide #2859
Conversation
@loicmathieu Nice. Maybe add a note on the minimum required docker version to run multistage build? |
@machi1990 it's Docker 17.05 ... two years old, maybe it's not worth specifying it ... |
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!
We would need one of our container guy to review that but I added a few suggestions on the wording.
@geoand could you take a look and check it's sufficiently clear?
@gsmet Sure thing, I will check it later on tonight |
1a3ae56
to
a9b5c81
Compare
a9b5c81
to
f134fcd
Compare
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, @gsmet ?
The changes appear to have been made, can you re-review @gsmet ? |
f134fcd
to
150bf14
Compare
Hello, I rebase on master as the native doc guide changes. |
@gsmet I went ahead and merged in master, probably worth having it in 0.18? |
@emmanuelbernard I cherry-picked it to the 0.18 branch. |
This adds some documentation to the native guide to show how to generate a native image using a multi-stage docker build.
It refers to this discussion : #2814