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

Exposed port via conf.yml is not mapping correctly #50

Closed
dacrome opened this issue Apr 27, 2015 · 5 comments
Closed

Exposed port via conf.yml is not mapping correctly #50

dacrome opened this issue Apr 27, 2015 · 5 comments
Labels

Comments

@dacrome
Copy link
Contributor

dacrome commented Apr 27, 2015

When set in conf.yml the ports like in the USAGE

ports:
  - 13306 3306

the container will be started with the following port configuration:
13306/tcp, 0.0.0.0:32778->3306/tcp

So the mapping is wrong.

@alexec
Copy link
Owner

alexec commented Apr 27, 2015

Hi, again, I don't suppose you have an example you could share with me? It'll make my life easier!

@alexec alexec added the bug label Apr 27, 2015
@dacrome
Copy link
Contributor Author

dacrome commented Apr 27, 2015

It's the same example that I mentioned in issue #49, so it includes the setup to reproduce both issues.

Just checkout the running-against-docker-container branch of my repo. I set the ports in the mysql and postgres conf.yml files in src/main/docker

To run the tests:

mvn verify -P postgres

or

mvn verify -P mysql

@alexec
Copy link
Owner

alexec commented Apr 29, 2015

Test case, create a port forward from 13306 on host to 3306 on a MySQL Container:

docker run -p 127.0.0.1:8888:3306 -dMYSQL_ROOT_PASSWORD=secret mysql
Alex-Collinss-MacBook:docker alexc$ docker ps
CONTAINER ID        IMAGE               COMMAND                CREATED              STATUS              PORTS                       NAMES
d183fb88e8af        mysql:5             "/entrypoint.sh mysq   About a minute ago   Up About a minute   127.0.0.1:8888->3306/tcp   thirsty_payne   

@alexec
Copy link
Owner

alexec commented Apr 30, 2015

Fixed. Please test and let me know if you have any issues.

@alexec alexec closed this as completed Apr 30, 2015
@dacrome
Copy link
Contributor Author

dacrome commented Apr 30, 2015

works like a charm. thanks again!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants