Project

General

Profile

Actions

Review #573

closed

Please review the dc-monitoring role rework

Added by Marc Dequènes almost 7 years ago. Updated over 6 years ago.

Status:
Rejected
Priority:
Normal
Category:
Service :: Supervision
Start date:
2017-06-27
Branch:
rework_monitoring

Description

The goal is to modernize it, add all the missing pieces, and make it more of a real role with a proper API. Then we would be able to separate the role in a different repo.

So I've made it step by step. There's still a few things to do. For example the pollers' config is not yet handled. It may be difficult to handle the initial install with debconf too. So I would say this is for later unless you have some ideas.

I think the current changes already make it cleaner in may ways. Nevertheless I still feel there are too many DC-specific things. I used entrypoints to separate install and autodetection, but the later is very much linked to our ZBX templates, so maybe I should split the role in two. The SNMP config also is too specific. The provided script in /etc/zabbix/externalscripts is also not appropriate for a generic Zabbix installation role.

What would you think now would be the next steps? What changes would you suggest.

Actions #1

Updated by Marc Dequènes over 6 years ago

  • Status changed from New to Rejected

I decided to merge these changes as they were well tested. More changes are needed to modernize and also manages all aspects (like pollers' conffiles), so I will request another review once it is settled.

Actions

Also available in: Atom PDF