Update host integration article#8822
Conversation
|
(I'm on mobile, so unable to add inline comments) Should the link to the previous version be a relative link? Ie drop the domain-name from the url? Just so that it is able to work on the staging/beta docs. Not sure, I'm not a maintainer |
|
Good point. Updated the link. |
|
Thanks! |
There was a problem hiding this comment.
this sentence is way too mega-long, it could probably be simplified by swapping things around :)
|
Ok, I've incorporated those suggestions and changed a few more things. |
|
Yeah, no prob. |
|
Squashed. |
38e2b1d to
041576f
Compare
There was a problem hiding this comment.
Comma after "i.e." please. I also think the sentence is flipped, and the clause "if restart policies don't suit your needs" should go at the beginning. Readers expect contextualizing info like this at the beginning.
|
LGTM once my copy edit nitpick is addressed. Will merge once that's fixed. Many thanks for the contribution. |
f3c5af1 to
9542ea7
Compare
|
How's that? |
Update for changes in docker 1.2. Running the docker daemon with "-r=false" has been deprecated in favor of per-container restart policies. Signed-off-by: wilsaj <[email protected]>
|
Terrific! Thanks. |
Update host integration article
Update host integration article (cherry picked from commit c3762c3)
Update for changes in docker 1.2. Running the docker daemon with "-r=false" has been deprecated in favor of per-container restart policies.