Skip to content

docs: add instructions for containers filtering#2667

Merged
buchdag merged 3 commits into
mainfrom
docs/filtering-containers
Oct 25, 2025
Merged

docs: add instructions for containers filtering#2667
buchdag merged 3 commits into
mainfrom
docs/filtering-containers

Conversation

@buchdag

@buchdag buchdag commented Oct 19, 2025

Copy link
Copy Markdown
Member

Supersede #2659 as this is now handled by docker-gen itself.

ping @eorea @p12tic

@buchdag buchdag self-assigned this Oct 19, 2025
@buchdag buchdag added the type/docs PR with documentation only changes label Oct 19, 2025

@p12tic p12tic left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@p12tic

p12tic commented Oct 21, 2025

Copy link
Copy Markdown
Contributor

I would add the following example:

For example, setting adding DOCKER_CONTAINER_FILTERS environment variable to network=mynetwork will cause nginx-proxy to consider only containers connected to mynetwork network.

I suspect this would cover large majority of cases of running mulitple nginx-proxy instances.

@buchdag

buchdag commented Oct 21, 2025

Copy link
Copy Markdown
Member Author

@p12tic could your make it a review suggestion so I can commit it to the PR ?

Comment thread docs/README.md
@p12tic

p12tic commented Oct 24, 2025

Copy link
Copy Markdown
Contributor

@buchdag Done, thanks.

Comment thread docs/README.md
@buchdag buchdag merged commit 9cfc460 into main Oct 25, 2025
2 checks passed
@buchdag buchdag deleted the docs/filtering-containers branch October 25, 2025 09:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type/docs PR with documentation only changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants