First Last Prev Next    This bug is not in your last search results.
Bug 55068 - check_univention_joinstatus periodically causes high load on slapd
check_univention_joinstatus periodically causes high load on slapd
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: Monitoring (Prometheus or Nagios)
UCS 5.0
Other Linux
: P5 normal (vote)
: UCS 5.0-2-errata
Assigned To: Siavash Sefid Rodi
Arvid Requate
:
Depends on: 54748
Blocks:
  Show dependency tree / graph
 
Reported: 2022-08-04 12:15 CEST by Arvid Requate
Modified: 2022-08-05 12:35 CEST (History)
1 user (show)

See Also:
What kind of report is it?: Bug Report
What type of bug is this?: 5: Major Usability: Impairs usability in key scenarios
Who will be affected by this bug?: 3: Will affect average number of installed domains
How will those affected feel about the bug?: 2: A Pain – users won’t like this once they notice it
User Pain: 0.171
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional): Regression, UCS Performance
Max CVSS v3 score:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Arvid Requate univentionstaff 2022-08-04 12:15:46 CEST 
check_univention_joinstatus from univention-monitoring-client periodically causes high load on slapd. Origin seems to be a typo in the ldapsearch options which should be fixed like htis:
=========
diff --git a/monitoring/univention-monitoring-client/alerts/check_univention_joinstatus b/monitoring/univention-monitoring-client/alerts/check_univention_joinstatus
index 9c5d8dab36..277f8077a6 100755
--- a/monitoring/univention-monitoring-client/alerts/check_univention_joinstatus
+++ b/monitoring/univention-monitoring-client/alerts/check_univention_joinstatus
@@ -97,7 +97,7 @@ class JoinStatusCheck(Alert):
             '-D', ucr['ldap/hostdn'],
             '-w', self.secret,
             '-b', ucr['ldap/base'],
-            's', 'base'
+            '-s', 'base'
         ]
         try:
             subprocess.check_call(cmd, stdout=subprocess.DEVNULL)
@@ -114,7 +114,7 @@ class JoinStatusCheck(Alert):
             '-D', ucr['ldap/hostdn'],
             '-w', self.secret,
             '-b', ucr['ldap/base'],
-            's', 'base'
+            '-s', 'base'
         ]
         try:
             subprocess.check_call(cmd, stdout=subprocess.DEVNULL)
@@ -133,7 +133,7 @@ class JoinStatusCheck(Alert):
             '-D', ucr['ldap/hostdn'],
             '-w', self.secret,
             '-b', ucr['ldap/base'],
-            's', 'base'
+            '-s', 'base'
         ]
         try:
             subprocess.check_call(cmd, stdout=subprocess.DEVNULL)

=====
Comment 1 Siavash Sefid Rodi univentionstaff 2022-08-04 12:36:58 CEST 
Fixed typos as described above, should create smaller output, thus improve performance
Comment 2 Arvid Requate univentionstaff 2022-08-04 14:52:27 CEST 
839b85328e | Advisory: Add bug number

Verified:
* Code review
* Package update
* Advisory
First Last Prev Next    This bug is not in your last search results.