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

ports not accepting a range in yml #1102

Closed
awsmsrc opened this issue Mar 12, 2015 · 20 comments
Closed

ports not accepting a range in yml #1102

awsmsrc opened this issue Mar 12, 2015 · 20 comments

Comments

@awsmsrc
Copy link

awsmsrc commented Mar 12, 2015

the command following command works as of docker 1.5:

docker run -p 3000-3002:3000-3002 port_test

specifying ports this way in the docker-compose.yml always results in a cannot start container exit status 2

@tvcutsem
Copy link

Ran into the same issue yesterday. Specifying the port range as a string also doesn't help. Here's the offending section from my yml file:

ports:
  - "50000-50050:50000-50050"

@dougborg
Copy link

We have run into this issue as well. We were expecting something like this to "just work":

ports:
  - "1000-1019"

@yuval-k
Copy link

yuval-k commented Mar 26, 2015

#dibs

@awsmsrc
Copy link
Author

awsmsrc commented Mar 26, 2015

gonna work on this one at the docker party tonight

@yuval-k
Copy link

yuval-k commented Mar 26, 2015

hi @awsmsrc, I already started working on this, FYI - do you want to help us?

yuval-k added a commit to yuval-k/compose that referenced this issue Mar 26, 2015
Signed-off-by: Yuval Kohavi <yuval.kohavi@gmail.com>
yuval-k added a commit to yuval-k/compose that referenced this issue Mar 26, 2015
Signed-off-by: Yuval Kohavi <yuval.kohavi@gmail.com>
yuval-k added a commit to yuval-k/compose that referenced this issue Mar 26, 2015
Signed-off-by: Yuval Kohavi <yuval.kohavi@gmail.com>
@DXist
Copy link

DXist commented Apr 9, 2015

+1

yuval-k added a commit to yuval-k/compose that referenced this issue Apr 10, 2015
Signed-off-by: Yuval Kohavi <yuval.kohavi@gmail.com>

Signed-off-by: Yuval Kohavi <yuval.kohavi@gmail.com>
yuval-k added a commit to yuval-k/compose that referenced this issue Apr 10, 2015
Signed-off-by: Yuval Kohavi <yuval.kohavi@gmail.com>

Signed-off-by: Yuval Kohavi <yuval.kohavi@gmail.com>
yuval-k added a commit to yuval-k/compose that referenced this issue Apr 10, 2015
Signed-off-by: Yuval Kohavi <yuval.kohavi@gmail.com>

Signed-off-by: Yuval Kohavi <yuval.kohavi@gmail.com>
@benbot
Copy link

benbot commented Jun 13, 2015

Has this been in any release yet?

@awsmsrc
Copy link
Author

awsmsrc commented Jun 29, 2015

yeah would be good to get an update @aanand

@cloetzi
Copy link

cloetzi commented Jun 30, 2015

+1

@aanand
Copy link

aanand commented Jul 2, 2015

It's been implemented by @uvgroovy in #1191, but needs rebasing.

@yuval-k
Copy link

yuval-k commented Jul 6, 2015

@aanand rebasing done

@reidrac
Copy link

reidrac commented Jul 24, 2015

+1

@willbarrett
Copy link

Looks like the implementation just needs docs and a rebase. I'm sure we'd all be grateful to @uvgroovy if that happened soon :)

@yuval-k
Copy link

yuval-k commented Jul 28, 2015

Hi,
I just realized that missed the documentation request from the PR... sorry for that - on it right now
thanks @willbarrett

@tpires
Copy link

tpires commented Aug 31, 2015

Can someone close this? Looks like this is already merged here.

@dnephin dnephin closed this as completed Aug 31, 2015
@jasonm-makerbot
Copy link

👍

@brunoborges
Copy link

Why only "ports:" supports range? Why not "expose:" too?

Also, is it possible to fix the documentation to go into more detail on the differences between ports and expose? It is still not clear on what happens in each case.

@vpillac
Copy link

vpillac commented Jan 6, 2016

Hi, my docker-compose files stopped working after updating to docker-compose version 1.5.2 (from MacOS Docker Toolbox 1.9.1).

Service 'xxx' configuration key 'expose' '0' is invalid: should be of the format 'PORT[/PROTOCOL]'

While the file contains

  expose:
  - "8200-8219"

Replacing the range by a single port "resolves" the issue.

@dnephin
Copy link

dnephin commented Jan 6, 2016

Ah, I guess that is valid. Could you open a new issue about it?

I wonder if that was added recently to expose, I don't remember seeing it in the docs when we implemented it.

@vpillac
Copy link

vpillac commented Jan 6, 2016

New issue created: #2611

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 a pull request may close this issue.