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

Docs: Troubleshooting help for Docker Swarm labels #4751

Merged
merged 3 commits into from May 3, 2019

Conversation

gregberns
Copy link

@gregberns gregberns commented Apr 9, 2019

What does this PR do?

This PR adds helpful troubleshooting guidelines for configuring a service from the docker-compose file when attempting to automatically pick up services from Docker Swarm.

Motivation

I literally have spent the last 3 full days trying to configure my service to get Traefik to redirect traffic to Docker Swarm services. The problem was simply solved by ensuring traefik.docker.network and traefik.port were included. Was very frustrating and I want to make sure others don't need to go through the same frustration.

Additional Notes

Please feel free to reformat/re-write based on the project's documentation practices. I just want to make sure anyone attempting to configure Docker Swarm services know what is required.
And thanks to #3535 to help me figure this out.

Copy link
Member

@ldez ldez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Member

@jbdoumenjou jbdoumenjou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@dduportal dduportal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, this is a really valuable feedback!
LGTM 👍

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

Successfully merging this pull request may close these issues.

None yet

6 participants