Need to use proper docker formatting for docker engine API

[marcusdandrea]

Please expose port 1433 in the docker file and change formatting of this image to allow proper docker API calls. Thank you.

[LuisBosquez]

@marcusdandrea Could you please elaborate on what proper Docker API calls means? Also, about how not exposing port 1433 affecting you?

[marcusdandrea]

Belows is a sample YAML that I'm passing over the Docker Engine API which normally works for most Windows container apps but not SQL. If I issue a docker run on the windows host with the port it works fine. I tried a variation of the specific port as well as all ports and both result in a negative outcome. I believe the docker expose ports command needs to be in the Dockerfile which may resolve this problem.

sqlserver:
image: microsoft/mssql-server-windows:latest
host: host1
mem_min: 500m
#ports:

- 1433:1433

publish_all_ports: true

[twright-msft]

@marcusdandrea Just wanted to double check here. It looks like you commented out the 'ports' attribute in the dockerfile. Was that intentional?

[marcusdandrea]

It was as I tested with publish all ports and with static ports.

…

On Jan 19, 2017, at 12:18 AM, Travis Wright ***@***.***> wrote: @marcusdandrea Just wanted to double check here. It looks like you commented out the 'ports' attribute in the dockerfile. Was that intentional? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.