Skip to content
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

Deploy image to OpenShift 3 #496

Closed

Conversation

jcantrill
Copy link
Contributor

@fbricon @mishaone First pass at the docs. Need to be validated that links and formatting are correct, but he bulk of the content is there. Probably will change if we add the required enhancements before release

- Service
- Route 'Optional'

*Note:* The following steps require:
Copy link
Member

Choose a reason for hiding this comment

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

[NOTE]
====================================================
The following steps require:

- Connection to a Docker daemon already exist
- Connection to an OpenShift 3 cluster already exist
- Openshift 3 project exist
====================================================

Copy link
Contributor Author

Choose a reason for hiding this comment

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

resolved

@mishaone
Copy link
Contributor

Thanks, @jcantrill. Looks good! When this request is merged, I'll replicate the information for the enterprise docs.

@jcantrill
Copy link
Contributor Author

@mishaone updated per @fbricon comments

- Service
- Route 'Optional'

[Note]
Copy link
Member

Choose a reason for hiding this comment

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

in asciidoc you just write 👍

NOTE: The following steps require: 
. Connection to a Docker demon already exist.
....

then it will be rendered nicely.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

resolved

@jcantrill
Copy link
Contributor Author

updated per @maxandersen comments. Not sure I fully addressed his requests

@fbricon
Copy link
Member

fbricon commented Oct 2, 2015

Merged as cee3c29 after some minor wording/formatting modifications

@fbricon fbricon closed this Oct 2, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants