conf.d files does not get included from nginx

[kritzi-at]

Your setup

Formula commit hash / release tag

93024f7

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:
- server:
- listen: 80
- server_name: xxx
- return: 301 https://$server_name$request_uri

---

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]

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.