opendev/puppet-elastic_recheck
Code Issues Proposed changes

Rework uncategorized script to handle multiple html files

Browse Source 
In a recent elastic-recheck change we updated the output of the
uncategorized command to split between the integrated gate failues
and everything else. In the future we'll add a configurable views
system where we can construct multiple different groups. The days
of a single uncategorized.html file have disappeared. This commit
adds the necessary support to the cron job so it can handle an
multiple number of html files.

Change-Id: I12690cecd3b5d27fa3e508f7ea4daa43696423b8
This commit is contained in:
Matthew Treinish 2016-08-10 12:29:58 -04:00
parent 730b85138a
commit c15da7419e
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
manifests/cron.pp
View File

@ -25,7 +25,7 @@ class elastic_recheck::cron (
$er_state_path = '/var/lib/elastic-recheck'
$graph_all_cmd = "elastic-recheck-graph /opt/elastic-recheck/queries -o all-new.json ${graph_all_cmd_options} && mv all-new.json all.json"
$graph_gate_cmd = "elastic-recheck-graph /opt/elastic-recheck/queries -o gate-new.json -q gate ${graph_gate_cmd_options} && mv gate-new.json gate.json"
$uncat_cmd = "elastic-recheck-uncategorized -d /opt/elastic-recheck/queries -t /usr/local/share/elastic-recheck/templates -o uncategorized-new.html ${uncat_cmd_options} && mv uncategorized-new.html uncategorized.html"
$uncat_cmd = "elastic-recheck-uncategorized -d /opt/elastic-recheck/queries -t /usr/local/share/elastic-recheck/templates -o new ${uncat_cmd_options} && mv new/*.html ."
cron { 'elastic-recheck-all':
user => 'recheck',
@ -50,7 +50,7 @@ class elastic_recheck::cron (
user => 'recheck',
minute => ['20', '50'],
hour => '*',
command => "cd ${er_state_path} && er_safe_run.sh ${uncat_cmd}",
command => "cd ${er_state_path} && mkdir new && er_safe_run.sh ${uncat_cmd} && rm -r new",
environment => 'PATH=/usr/bin:/bin:/usr/sbin:/sbin:/usr/local/bin',
require => Class['elastic_recheck']
}