Rush has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/342161 )

Change subject: toolschecker: remove precise checks
......................................................................


toolschecker: remove precise checks

The precise OGE exec nodes are being shut down on 2017-03-14. Once they
are down there will be no need to validate that jobs can be started on
them.

* remove toolchecker definitions
* remove icinga monitoring job

Bug: T94792
Change-Id: I682ea213afafab6221fcea36f1ec7c0044d34ae2
---
M modules/icinga/manifests/monitor/toollabs.pp
M modules/toollabs/files/toolschecker.py
M modules/toollabs/manifests/checker.pp
3 files changed, 0 insertions(+), 24 deletions(-)

Approvals:
  Rush: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/modules/icinga/manifests/monitor/toollabs.pp 
b/modules/icinga/manifests/monitor/toollabs.pp
index 8d3ad78..185c449 100644
--- a/modules/icinga/manifests/monitor/toollabs.pp
+++ b/modules/icinga/manifests/monitor/toollabs.pp
@@ -117,14 +117,6 @@
         retry_interval => 5,
     }
 
-    monitoring::service { 'tools-checker-grid-start-precise':
-        description    => 'Start a job and verify on Precise',
-        check_command  => "${checker}!/grid/start/precise!OK",
-        host           => $test_entry_host,
-        check_interval => 5,
-        retry_interval => 5,
-    }
-
     monitoring::service { 'tools-checker-etcd-flannel':
         description   => 'All Flannel etcd nodes are healthy',
         check_command => "${checker}!/etcd/flannel!OK",
diff --git a/modules/toollabs/files/toolschecker.py 
b/modules/toollabs/files/toolschecker.py
index a0736fb..87d9deb 100644
--- a/modules/toollabs/files/toolschecker.py
+++ b/modules/toollabs/files/toolschecker.py
@@ -186,11 +186,6 @@
         return False
 
 
-@check('/continuous/precise')
-def continuous_job_precise():
-    return job_running('test-long-running-precise')
-
-
 @check('/continuous/trusty')
 def continuous_job_trusty():
     return job_running('test-long-running-trusty')
@@ -227,11 +222,6 @@
 @check('/grid/start/trusty')
 def grid_check_start_trusty():
     return grid_check_start('trusty')
-
-
-@check('/grid/start/precise')
-def grid_check_start_precise():
-    return grid_check_start('precise')
 
 
 def db_read_write_check(host, db):
diff --git a/modules/toollabs/manifests/checker.pp 
b/modules/toollabs/manifests/checker.pp
index c968e4e..6c9d1ad 100644
--- a/modules/toollabs/manifests/checker.pp
+++ b/modules/toollabs/manifests/checker.pp
@@ -70,14 +70,8 @@
         'continuous_job_trusty'  => {
             path                 => '/continuous/trusty',
         },
-        'continuous_job_precise' => {
-            path                 => '/continuous/precise',
-        },
         'grid_start_trusty'      => {
             path                 => '/grid/start/trusty',
-        },
-        'grid_start_precise'     => {
-            path                 => '/grid/start/precise',
         },
         'cron'                   => {
             path                 => '/toolscron',

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I682ea213afafab6221fcea36f1ec7c0044d34ae2
Gerrit-PatchSet: 4
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: BryanDavis <[email protected]>
Gerrit-Reviewer: Andrew Bogott <[email protected]>
Gerrit-Reviewer: Coren <[email protected]>
Gerrit-Reviewer: Giuseppe Lavagetto <[email protected]>
Gerrit-Reviewer: Madhuvishy <[email protected]>
Gerrit-Reviewer: Merlijn van Deen <[email protected]>
Gerrit-Reviewer: Rush <[email protected]>
Gerrit-Reviewer: Volans <[email protected]>
Gerrit-Reviewer: Yuvipanda <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

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

Reply via email to