search

saltstack-formulas / nginx-formula

Nginx Salt Formula
http://docs.saltstack.com/en/latest/topics/development/conventions/formulas.html
Other
163 stars 421 forks source link

conf.d files does not get included from nginx #287

Open kritzi-at opened 3 years ago

kritzi-at commented 3 years ago

Your setup

Formula commit hash / release tag

93024f744415dfdb4c56b9cdb9276b9aec0c942f

Versions reports (master & minion)

salt --versions-report Salt Version: Salt: 3003.2

Dependency Versions: cffi: 1.11.5 cherrypy: unknown dateutil: 2.6.1 docker-py: Not Installed gitdb: 4.0.1 gitpython: 3.0.9 Jinja2: 2.10.1 libgit2: 0.26.8 M2Crypto: 0.35.2 Mako: Not Installed msgpack: 0.6.2 msgpack-pure: Not Installed mysql-python: 1.4.6 pycparser: 2.14 pycrypto: Not Installed pycryptodome: Not Installed pygit2: 0.26.4 Python: 3.6.8 (default, Jul 15 2021, 15:17:46) python-gnupg: Not Installed PyYAML: 3.12 PyZMQ: 19.0.0 smmap: 3.0.1 timelib: Not Installed Tornado: 4.5.3 ZMQ: 4.3.4

System Versions: dist: centos 8 locale: UTF-8 machine: x86_64 release: 4.18.0-331.el8.x86_64 system: Linux version: CentOS Stream 8

salt-minion --versions-report Salt Version: Salt: 3003.2

Dependency Versions: cffi: Not Installed cherrypy: Not Installed dateutil: 2.8.1 docker-py: Not Installed gitdb: Not Installed gitpython: Not Installed Jinja2: 2.10.1 libgit2: Not Installed M2Crypto: 0.35.2 Mako: Not Installed msgpack: 0.6.2 msgpack-pure: Not Installed mysql-python: Not Installed pycparser: Not Installed pycrypto: Not Installed pycryptodome: Not Installed pygit2: Not Installed Python: 3.6.8 (default, Jul 15 2021, 15:17:46) python-gnupg: Not Installed PyYAML: 3.12 PyZMQ: 22.1.0 smmap: Not Installed timelib: Not Installed Tornado: 4.5.3 ZMQ: 4.3.4

System Versions: dist: centos 8 locale: UTF-8 machine: x86_64 release: 4.18.0-310.el8.x86_64 system: Linux version: CentOS Stream 8

Pillar / config used

nginx: servers: managed: http_redirect: enabled: true config:

Bug details

Describe the bug

Configuration Files created in /etc/nginx/conf.d/ are ignored becaus they missing the .conf extension. nginx.conf is set to include "conf. d/*. conf"

Steps to reproduce the bug

Use this formula on a distribution that uses conf.d directory.

Expected behaviour

The configuration files should be created with the correct extension.

tacerus commented 1 year ago

Hi,

did you apply the nginx.config state as well? It should write a nginx.conf file which includes conf.d/* (without the .conf extension), making all files in conf.d get loaded.