From 927a8f2e0c186adb4cd2d91a555878ab6d1679c5 Mon Sep 17 00:00:00 2001
From: Rico Ullmann <rico@ullmann.space>
Date: Tue, 21 Dec 2021 17:16:50 +0100
Subject: [PATCH] nginx: remove icinga2, telegraf and pacman references

---
 bundles/nginx/items.py    | 19 +++++++++----------
 bundles/nginx/metadata.py | 31 -------------------------------
 2 files changed, 9 insertions(+), 41 deletions(-)

diff --git a/bundles/nginx/items.py b/bundles/nginx/items.py
index 88852e0..cf7e7e5 100644
--- a/bundles/nginx/items.py
+++ b/bundles/nginx/items.py
@@ -1,11 +1,10 @@
 from datetime import datetime, timedelta
 
-if node.has_bundle('pacman'):
-    package = 'pkg_pacman:nginx'
-    username = 'http'
-else:
-    package = 'pkg_apt:nginx'
-    username = 'www-data'
+pkg_apt = {
+    'nginx': {
+        'needs': ['file:/etc/apt/sources.list.d/nginx.list'],
+    },
+}
 
 directories = {
     '/etc/nginx/sites': {
@@ -24,9 +23,9 @@ directories = {
         },
     },
     '/var/log/nginx-timing': {
-        'owner': username,
+        'owner': 'http',
         'needs': {
-            package,
+            'pkg_apt:nginx',
         },
     },
     '/var/www': {},
@@ -39,7 +38,7 @@ files = {
     '/etc/nginx/nginx.conf': {
         'content_type': 'mako',
         'context': {
-            'username': username,
+            'username': 'http',
             **node.metadata['nginx'],
         },
         'triggers': {
@@ -89,7 +88,7 @@ svc_systemd = {
         'needs': {
             'action:nginx-generate-dhparam',
             'directory:/var/log/nginx-timing',
-            package,
+            'pkg_apt:nginx',
         },
     },
 }
diff --git a/bundles/nginx/metadata.py b/bundles/nginx/metadata.py
index 81bc5aa..f802ea7 100644
--- a/bundles/nginx/metadata.py
+++ b/bundles/nginx/metadata.py
@@ -9,48 +9,17 @@ defaults = {
                 },
             },
         },
-        'packages': {
-            'nginx': {},
-        },
     },
     'backups': {
         'paths': {
             '/var/www',
         },
     },
-    'icinga2_api': {
-        'nginx': {
-            'services': {
-                'NGINX PROCESS': {
-                    'command_on_monitored_host': '/usr/local/share/icinga/plugins/check_systemd_unit nginx',
-                },
-                'NGINX STATUS': {
-                    'command_on_monitored_host': '/usr/local/share/icinga/plugins/check_nginx_status',
-                },
-            },
-        },
-    },
     'nginx': {
         'worker_connections': 768,
     },
-    'pacman': {
-        'packages': {
-            'nginx': {},
-        },
-    },
 }
 
-if node.has_bundle('telegraf'):
-    defaults['telegraf'] = {
-        'input_plugins': {
-            'builtin': {
-                'nginx': [{
-                    'urls': ['http://localhost:22999/server_status'],
-                }],
-            },
-        },
-    }
-
 
 @metadata_reactor.provides(
     'nginx/worker_processes',