Warning, Duplicate When Key about ansible-role-nginx HOT 5 OPEN

Introduced in this commit:
6863343

from ansible-role-nginx.

This commit also introduce by default nginx to load the mod-http-geoip module.
It would be better to not load modules by default. Let people decide what have to be loaded

cc @q2digger

from ansible-role-nginx.

@techzilla @rlex Thanks for the report. If some of you or @q2digger can fix, it will be very nice ;)

@gaelL No worries, it's the documentation, geoip isn't load by default ;)

from ansible-role-nginx.

There is more to it than just fixing "when"

After some digging i think that

(only for RHEL & CentOS).

Is wrong.

Ubuntu, nginx.org repo:

nginx-module-geoip - nginx GeoIP dynamic modules
nginx-module-geoip-dbg - debug symbols for the nginx-module-geoip
nginx-module-image-filter - nginx image filter dynamic module
nginx-module-image-filter-dbg - debug symbols for the nginx-module-image-filter
nginx-module-njs - nginx njs dynamic modules
nginx-module-njs-dbg - debug symbols for the nginx-module-njs
nginx-module-perl - nginx Perl dynamic module
nginx-module-perl-dbg - debug symbols for the nginx-module-perl
nginx-module-xslt - nginx xslt dynamic module
nginx-module-xslt-dbg - debug symbols for the nginx-module-xslt
nginx-nr-agent - New Relic agent for NGINX and NGINX Plus

Debian/ubuntu official repos also have modules (and they are pretty much the same):
https://packages.debian.org/search?keywords=libnginx&searchon=names&suite=stable&section=all
https://packages.ubuntu.com/search?keywords=libnginx&searchon=names&suite=bionic&section=all

So we have 3 versions of path difference between rhel/debian (maybe load it in vars/{debian,redhat}?)
RHEL uses (both mainline and official repos? Can someone with centos/any other rhel confirm?)

/urs/share/nginx/modules

(Worth mentioning that current path specified in task will not work because of "urs" instead of "usr")

Debian:

/usr/share/nginx/modules-available/

And debian from nginx.org repo:

/usr/lib/nginx/modules

This one is tricky - it only installs .so files without configs to enable, so we need to generate configs manually.

Any ideas?

from ansible-role-nginx.

ahh ok, I can see the duplicate when statements which are generating this warning.

I have hopefully included a fix for this in PR #235 .

from ansible-role-nginx.