-
Notifications
You must be signed in to change notification settings - Fork 275
Open
Description
In the Legacy branch the dockerfiles used to expose the consul ports: https://github.com/gliderlabs/docker-consul/blob/legacy/Dockerfile#L31
This has been removed in the current Dockerfiles. Was there motivation for this or was it simply an oversight? I maintain a test suite that spins up ephemeral Consul containers to test API calls against and it's easier to map ports when they are exposed, so if this is simply an oversight I'll open a PR to add them back.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels