[ZOOKEEPER-3770] Monitoring script support for 3.6

• Type: Improvement
• Status: Open
• Priority: Minor
• Resolution: Unresolved
• Affects Version/s: 3.6.0, 3.6.1, 3.6.2
• Component/s: contrib
• Labels: pull-request-available

Environment

OS: Red Hat Enterprise Linux Server release 7.7

Zookeeper version: 3.6.0

Python version: 2.7.5

• Assignee: Unassigned
• Reporter: Michał Lisowski
• Created: 25/Mar/20 12:35
• Updated: 28/Mar/20 16:52

Description

The zookeeper-contrib-monitoring script needs fixes to support zookeeper 3.6.

Found issues:

• 4lw.commands.whitelist=mntr,ruok,stat is now required for the script to work, as 4 letter words commands are now all disabled by default (README has to be updated).
• zk_avg_latency metric is now float, the script has them all as strings anyway, and now it fails when trying to compare for example the '0.0' string to the warning and critical values.
• Now, since 'mntr' command returns much more data (even more with the Prometheus Metrics Exporter), python socket communication has to be changed to support much more data and data chunking.

Issue Links

• links to: GitHub Pull Request #1294