Skip to content
This repository was archived by the owner on Feb 12, 2021. It is now read-only.

Convert launching-containers example to use template units.#367

Merged
robszumski merged 3 commits intocoreos:masterfrom
emmanuel:master
Nov 4, 2014
Merged

Convert launching-containers example to use template units.#367
robszumski merged 3 commits intocoreos:masterfrom
emmanuel:master

Conversation

@emmanuel
Copy link
Copy Markdown
Contributor

No description provided.

@emmanuel
Copy link
Copy Markdown
Contributor Author

@robszumski let me know if this pull request is an adequate update to keep the docs in sync with coreos/unit-examples#5.

@robszumski
Copy link
Copy Markdown
Member

Since this is a pretty popular guide, I want to run through it end to end this afternoon and test. Will let you know how it goes.

@emmanuel
Copy link
Copy Markdown
Contributor Author

Great.

Glad to hear you’re going to run through it. I was thinking ‘hmm, this is a little dicey to update the docs without running the code’ :).=

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we can get rid of "Be sure to launch both instances of the template (apache-discovery@.service), by starting apache-discovery@1 and apache-discovery@2" since it's mentioned below to start them.

@robszumski
Copy link
Copy Markdown
Member

Made a few comments. Everything worked fine for me on a 3 machine cluster.

@robszumski
Copy link
Copy Markdown
Member

Can you find a few minutes to integrate the feedback?

@emmanuel
Copy link
Copy Markdown
Contributor Author

emmanuel commented Nov 4, 2014

oops. I had the first commit sitting locally. Added a second commit for the other feedback. LMK if these edits fit the bill.

@robszumski
Copy link
Copy Markdown
Member

Awesome, LGTM. Thanks!

robszumski added a commit that referenced this pull request Nov 4, 2014
Convert launching-containers example to use template units.
@robszumski robszumski merged commit 1c8e047 into coreos:master Nov 4, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants