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

Add networking example #9542

Merged
merged 4 commits into from Nov 30, 2022
Merged

Add networking example #9542

merged 4 commits into from Nov 30, 2022

Conversation

Janik-Haag
Copy link
Contributor

It took me a while to realize that i have to put the container and traefik in the same network so i thought it should be added to the documentation.

What does this PR do?

Telling people to use a external network with docker containers

Motivation

It took me a while to realize that that's the fault

It took me a while to realize that i have to put the container and traefik in the same network so i thought it should be added to the documentation.
@nmengin
Copy link
Contributor

nmengin commented Nov 25, 2022

Hello @Janik-Haag,

Thank you for your contribution.

We have moved your network example into the User Guide section to keep a very basic example in the "Quick Start" section.

@nmengin nmengin added this to To review in v2 via automation Nov 25, 2022
@ldez ldez changed the title Added Networking Added networking example Nov 28, 2022
@nmengin nmengin requested a review from mpl November 29, 2022 16:08
Copy link
Member

@rtribotte rtribotte left a comment

Choose a reason for hiding this comment

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

Thanks 👍

Copy link
Contributor

@nmengin nmengin left a comment

Choose a reason for hiding this comment

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

LGTM 📖

@traefiker traefiker merged commit af71443 into traefik:v2.9 Nov 30, 2022
v2 automation moved this from To review to Done Nov 30, 2022
@tomMoulard tomMoulard changed the title Added networking example Add networking example Dec 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
v2
Done
Development

Successfully merging this pull request may close these issues.

None yet

5 participants