Watchtower: Blacklist in addition to Whitelist

Created on 8 May 2020  路  9Comments  路  Source: containrrr/watchtower

Currently there is a whitelist behavior you can enable via filter_by_enable_label but a lot of setups would benefit instead from setting a blacklist of containers by label, simply because there is so many containers that would need to get a label added, versus just a few that shouldn't be checked each time - e.g. custom locally built containers.

Low Awaiting user Enhancement

Most helpful comment

@Miosame we are planning to mention it soon: https://github.com/containrrr/watchtower/pull/511#issuecomment-622283650

All 9 comments

Hey,

Thank you for your issue!

That is already possible, by setting com.centurylinklabs.watchtower.enable to false without passing --label-enable.

Hope that helps :)

@simskij so how would watchtower then decide what containers it _should_ update? the proposed issue I have is to just add a label to containers it should straight up ignore, basically the same behavior as enable - but disable.

So the containers that have com.centurylinklabs.watchtower.enable set to false will be ignored? if that's the case, that would be extremely helpful to mention in the docs.

@Miosame we are planning to mention it soon: https://github.com/containrrr/watchtower/pull/511#issuecomment-622283650

It is in the docs, although not that clearly described:

By default, watchtower will watch all containers. However, sometimes only some containers should be updated.

If you need to exclude some containers, set the com.centurylinklabs.watchtower.enable label to false.

https://containrrr.github.io/watchtower/container-selection/

I see, it was a separate page entirely, seeing that everything else was listed in Arguments too incl. examples and that was always the page I hit by googling things, I never went outside that page. Maybe a small note in the enable argument section linking back would help others alike to find it? :smile:

That would be great! Would you mind creating a PR? :) I'd be happy to help you getting it merged.

@simskij absolutely, I can see multiple ways of how this could be handled, one of these in #540

Closing this now that the PR is merged. Thank you for your contribution! 馃檹

To avoid important communication to get lost in a closed issues no one monitors, I'll go ahead and lock this issue. If you want to continue the discussion, please open a new issue. Thank you! 馃檹馃徏

Was this page helpful?
0 / 5 - 0 ratings

Related issues

MakerTim picture MakerTim  路  6Comments

eloo picture eloo  路  5Comments

ghost picture ghost  路  4Comments

ag0r4n picture ag0r4n  路  4Comments

Fredrik81 picture Fredrik81  路  5Comments