bundles/icinga2/files/icinga2.conf

@@ -5,6 +5,6 @@ include <itl>
 include <plugins>
 include <plugins-contrib>
 
-include "features-enabled/*.conf"
+include "features.d/*.conf"
 include_recursive "conf.d"
 include "hosts.d/*.conf"

bundles/icinga2/items.py

@@ -28,16 +28,7 @@ directories = {
             'svc_systemd:icinga2.service:restart',
         ],
     },
-    '/etc/icinga2/features-available': {
+    '/etc/icinga2/features.d': {
-        'purge': True,
-        'owner': 'nagios',
-        'group': 'nagios',
-        'mode': '0750',
-        'triggers': [
-            'svc_systemd:icinga2.service:restart',
-        ],
-    },
-    '/etc/icinga2/features-enabled': {
         'purge': True,
         'owner': 'nagios',
         'group': 'nagios',
@@ -178,6 +169,50 @@ files = {
             'svc_systemd:icinga2.service:restart',
         ],
     },
+    '/etc/icinga2/features.d/ido-pgsql.conf': {
+        'source': 'features/ido-pgsql.conf',
+        'content_type': 'mako',
+        'owner': 'nagios',
+        'group': 'nagios',
+        'context': {
+            'db_password': node.metadata.get('postgresql/roles/icinga2/password')
+        },
+        'triggers': [
+            'svc_systemd:icinga2.service:restart',
+        ],
+    },
+    '/etc/icinga2/features.d/syslog.conf': {
+        'source': 'features/syslog.conf',
+        'owner': 'nagios',
+        'group': 'nagios',
+        'triggers': [
+            'svc_systemd:icinga2.service:restart',
+        ],
+    },
+    '/etc/icinga2/features.d/notification.conf': {
+        'source': 'features/notification.conf',
+        'owner': 'nagios',
+        'group': 'nagios',
+        'triggers': [
+            'svc_systemd:icinga2.service:restart',
+        ],
+    },
+    '/etc/icinga2/features.d/checker.conf': {
+        'source': 'features/checker.conf',
+        'owner': 'nagios',
+        'group': 'nagios',
+        'triggers': [
+            'svc_systemd:icinga2.service:restart',
+        ],
+    },
+    '/etc/icinga2/features.d/api.conf': {
+        'source': 'features/api.conf',
+        'owner': 'nagios',
+        'group': 'nagios',
+        'triggers': [
+            'svc_systemd:icinga2.service:restart',
+        ],
+    },
     '/var/lib/icinga2/certs/ca.crt': {
         'content_type': 'download',
         'source': f'https://letsencrypt.org/certs/isrg-root-x1-cross-signed.pem',
@@ -192,38 +227,6 @@ files = {
     },
 }
 
-# FEATURES
-
-for feature, context in {
-    'syslog': {},
-    'notification': {},
-    'checker': {},
-    'api': {},
-    'ido-pgsql': {
-        'db_password': node.metadata.get('postgresql/roles/icinga2/password'),
-    },
-}.items():
-    files[f'/etc/icinga2/features-available/{feature}.conf'] = {
-        'content_type': 'mako' if context else 'text',
-        'context':  context,
-        'source': f'features/{feature}.conf',
-        'owner': 'nagios',
-        'group': 'nagios',
-        'triggers': [
-            'svc_systemd:icinga2.service:restart',
-        ],
-    }
-    symlinks[f'/etc/icinga2/features-enabled/{feature}.conf'] = {
-        'target': f'/etc/icinga2/features-available/{feature}.conf',
-        'owner': 'nagios',
-        'group': 'nagios',
-        'triggers': [
-            'svc_systemd:icinga2.service:restart',
-        ],
-    }
-
-# HOSTS
-
 for other_node in repo.nodes:
     if other_node.dummy:
         continue

bundles/mailserver-autoconfig/metadata.py

@@ -24,7 +24,6 @@ def nginx(metadata):
                     'context': {
                         'root': f"/var/www/{metadata.get('mailserver/autoconfig_hostname')}",
                     },
-                    'check_path': '/mail/config-v1.1.xml',
                 },
             },
         },

bundles/nginx/metadata.py

@@ -1,7 +1,5 @@
-from shlex import quote
 from ipaddress import ip_interface
 
-
 defaults = {
     'apt': {
         'packages': {
@@ -113,19 +111,3 @@ def letsencrypt(metadata):
             },
         },
     }
-
-
-@metadata_reactor.provides(
-    'monitoring/services',
-)
-def monitoring(metadata):
-    return {
-        'monitoring': {
-            'services': {
-                f'HTTP {hostname}': {
-                    'vars.command': f"""/usr/bin/curl {quote(hostname + vhost.get('check_path', ''))} -IL --fail"""
-                }
-                    for hostname, vhost in metadata.get('nginx/vhosts').items()
-            },
-        },
-    }