Browse Source

Include 'total' and 'ok' in check results

* Addresses https://bugzilla.redhat.com/show_bug.cgi?id=1389263
Tim Bielawa 8 years ago
parent
commit
4bde8aa816

+ 4 - 0
roles/openshift_certificate_expiry/filter_plugins/oo_cert_expiry.py

@@ -51,9 +51,13 @@ Example playbook usage:
 
         total_warnings = sum([hostvars[h]['check_results']['summary']['warning'] for h in play_hosts])
         total_expired = sum([hostvars[h]['check_results']['summary']['expired'] for h in play_hosts])
+        total_ok = sum([hostvars[h]['check_results']['summary']['ok'] for h in play_hosts])
+        total_total = sum([hostvars[h]['check_results']['summary']['total'] for h in play_hosts])
 
         json_result['summary']['warning'] = total_warnings
         json_result['summary']['expired'] = total_expired
+        json_result['summary']['ok'] = total_ok
+        json_result['summary']['total'] = total_total
 
         return json_result
 

+ 2 - 1
roles/openshift_certificate_expiry/library/openshift_cert_expiry.py

@@ -246,7 +246,8 @@ Return:
         'total': len(items),
         'ok': 0,
         'warning': 0,
-        'expired': 0
+        'expired': 0,
+        'total': len(items)
     }
 
     summary_results['expired'] = len([c for c in items if c['health'] == 'expired'])