On Tue, Mar 02, 2021 at 02:07:11PM +0800, Wu Fengguang wrote:

.log should also be skipped.

Ok. i will fix it.

Thanks, Fengguang

On Tue, Mar 02, 2021 at 09:30:06AM +0800, Wang Chenglong wrote:

...

[Why] The 'xxx.message' in es-jobs just show the error message, it doesn't has to be summed.

before: sum.stats.ansible_test.error.No-package-yum-utils-available.message: 1 sum.stats.ansible_test.error.No-package-yum-utils-available: 1 raw.stats.ansible_test.error.No-package-yum-utils-available.message: ["{"ansible_loop_var": "item", "changed": false, "failures": ["No package yum-utils available."], "item": ["yum-utils", "device-mapper-persistent-data", "lvm2"], "msg": "Failed to install some of the specified packages", "rc": 1, "results": []}"] raw.stats.ansible_test.error.No-package-yum-utils-available: [1]

after: sum.stats.ansible_test.error.No-package-yum-utils-available: 1 raw.stats.ansible_test.error.No-package-yum-utils-available.message: ["{"ansible_loop_var": "item", "changed": false, "failures": ["No package yum-utils available."], "item": ["yum-utils", "device-mapper-persistent-data", "lvm2"], "msg": "Failed to install some of the specified packages", "rc": 1, "results": []}"] raw.stats.ansible_test.error.No-package-yum-utils-available: [1]

Signed-off-by: Wang Chenglong 18509160991@163.com

lib/es_jobs.rb | 2 ++ 1 file changed, 2 insertions(+)

diff --git a/lib/es_jobs.rb b/lib/es_jobs.rb index c5c0f0d..b0d0fb2 100644 --- a/lib/es_jobs.rb +++ b/lib/es_jobs.rb @@ -124,6 +124,8 @@ class ESJobs

def stats_count(result) result['raw.stats'].each do |key, value|

•  next if key.end_with?('.message')
  

• if function_stat?(key)
    result['sum.stats'][key] = value.compact.size
  else
  

-- 2.23.0