Dzahn has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/249328

Change subject: labs kvm ssl cert monitoring: fix it
......................................................................

labs kvm ssl cert monitoring: fix it

Change-Id: I44fb35f362898d178d11f300c23ff227e59de5ba
---
M modules/nagios_common/files/check_commands/check_ssl.cfg
M modules/openstack/manifests/nova/compute.pp
2 files changed, 12 insertions(+), 7 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/operations/puppet 
refs/changes/28/249328/1

diff --git a/modules/nagios_common/files/check_commands/check_ssl.cfg 
b/modules/nagios_common/files/check_commands/check_ssl.cfg
index cfd0fcc..23c8cd6 100644
--- a/modules/nagios_common/files/check_commands/check_ssl.cfg
+++ b/modules/nagios_common/files/check_commands/check_ssl.cfg
@@ -10,8 +10,3 @@
     command_line    $USER1$/check_ssl --warning 60 --critical 30 -H 
$HOSTADDRESS$ -p 636 --cn $ARG1$
 }
 
-# check SSL certs file directly, regardless of port
-define command{
-    command_name    check_ssl_certfile
-    command_line    $USER1$/check_ssl_certfile $ARG1$
-}
diff --git a/modules/openstack/manifests/nova/compute.pp 
b/modules/openstack/manifests/nova/compute.pp
index a8422f6..039818e 100644
--- a/modules/openstack/manifests/nova/compute.pp
+++ b/modules/openstack/manifests/nova/compute.pp
@@ -30,9 +30,19 @@
             target  => "/etc/ssl/localcerts/${certname}.crt",
             require => Sslcert::Certificate[$certname],
         }
-        monitoring::service { 'kvm_cert':
+
+        file { '/usr/local/lib/nagios/plugins/check_ssl_certfile':
+            ensure => present,
+            owner  => 'root',
+            group  => 'root',
+            mode   => '0755',
+            source => 
'puppet:///modules/nagios_common/check_commands/check_ssl_certfile',
+        }
+
+        # T116332
+        nrpe::monitor_service { 'kvm_ssl_cert':
             description   => 'kvm ssl cert',
-            check_command => 
"check_ssl_certfile!/etc/ssl/localcerts/${certname}.crt",
+            nrpe_command  => "/usr/local/lib/nagios/plugins/check_ssl_certfile 
${certname}",
         }
 
         file { '/var/lib/nova/cacert.pem':

-- 
To view, visit https://gerrit.wikimedia.org/r/249328
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I44fb35f362898d178d11f300c23ff227e59de5ba
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Dzahn <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to