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

Expose links #90

Merged
merged 1 commit into from
Jun 28, 2017
Merged

Expose links #90

merged 1 commit into from
Jun 28, 2017

Conversation

paultyng
Copy link
Contributor

Exposes Links for RunWithOpts so that you can run two containers that can communicate via a dns alias.

@aeneasr
Copy link
Member

aeneasr commented Jun 28, 2017

Thank you! Could you please sign the doc using git commit --amend -s and force-push the change? :)

Signed-off-by: Paul Tyng <ptyng@underarmour.com>
@paultyng
Copy link
Contributor Author

paultyng commented Jun 28, 2017

Sorry about that, done.

I needed to expose a couple other things for my uses, would you prefer a new PR for each one, or just add in to this PR?

Specifically I added Entrypoint and ExposedPorts

@aeneasr
Copy link
Member

aeneasr commented Jun 28, 2017

Make it in a separate PR :)

@aeneasr aeneasr merged commit f2a4ab9 into ory:v3 Jun 28, 2017
@paultyng paultyng deleted the pt/links branch June 28, 2017 14:09
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

2 participants