Skip to content
This repository has been archived by the owner on Jul 2, 2023. It is now read-only.

Add port-publishing use case #2

Merged
merged 1 commit into from
Jun 15, 2018
Merged

Conversation

l-dobrev
Copy link
Contributor

Add a paragraph explaining how to use this image to publish ports on existing containers without recreating them.

@ozbillwang
Copy link
Contributor

Can you change the line ### Getting Started to ### Example with consistent statement.

@l-dobrev l-dobrev force-pushed the master branch 3 times, most recently from e1f97ec to fa594de Compare June 14, 2018 09:38
@l-dobrev
Copy link
Contributor Author

How's that?

Also fixed a missing back-slash in the example.
Also changed upper-case arguments to lower-case to match the documentation docker run --rm alpine/socat -h.

@ozbillwang
Copy link
Contributor

ozbillwang commented Jun 14, 2018

I can't make it work with --link example-container=target, I have replaced example-container name to a real one.

seems = can't be used with option --link, which docker version you have?

Here is mine:

$ docker -v
Docker version 18.03.1-ce, build 9ee9f40

Add a paragraph explaining how to use this image to publish ports on existing containers without recreating them.
@l-dobrev
Copy link
Contributor Author

You are right!
The correct separator was ":" not "=".
I have no excuse for missing that. :(

@ozbillwang ozbillwang merged commit b9a0898 into alpine-docker:master Jun 15, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants