fix: trigger for non-active slots

Author: casperr0

mamonsu/plugins/pgsql/xlog.py

@@ -6,6 +6,7 @@
 
 NUMBER_NON_ACTIVE_SLOTS = 0
 
+
 class Xlog(Plugin):
     DEFAULT_CONFIG = {'lag_more_than_in_sec': str(60 * 5)}
 
@@ -31,7 +32,7 @@ class Xlog(Plugin):
     key_wall = 'pgsql.wal.write{0}'
     key_count_wall = "pgsql.wal.count{0}"
     key_replication = "pgsql.replication_lag{0}"
-    key_non_active_slots = "pgsql.replication.non_active_slots"
+    key_non_active_slots = "pgsql.replication.non_active_slots{0}"
     AgentPluginType = 'pg'
 
     def run(self, zbx):
@@ -136,10 +137,11 @@ def triggers(self, template):
                     'on {HOSTNAME} (value={ITEM.LASTVALUE})',
             'expression': '{#TEMPLATE:' + self.right_type(self.key_replication, "sec") + '.last()}>' +
             self.plugin_config('lag_more_than_in_sec')
-        }) + template.trigger({
+        })
+        triggers += template.trigger({
             'name': 'PostgreSQL number of non-active replication slots '
                     'on {HOSTNAME} (value={ITEM.LASTVALUE})',
-            'expression': '{#TEMPLATE:' + self.right_type(self.key_replication, "sec") + '.last()}≠' +
+            'expression': '{#TEMPLATE:' + self.right_type(self.key_non_active_slots) + '.last()}≠' +
             str(NUMBER_NON_ACTIVE_SLOTS)
         })
         return triggers