From 7ba2d62e57d013c017f7f68ddfde3cede1da103a Mon Sep 17 00:00:00 2001 From: Jeff Cantrill Date: Fri, 22 Jan 2021 16:40:23 -0500 Subject: [PATCH] fix #228. Don't report cache miss if skipping namespace metadata --- lib/fluent/plugin/kubernetes_metadata_cache_strategy.rb | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/lib/fluent/plugin/kubernetes_metadata_cache_strategy.rb b/lib/fluent/plugin/kubernetes_metadata_cache_strategy.rb index c1a8c49..c09915e 100644 --- a/lib/fluent/plugin/kubernetes_metadata_cache_strategy.rb +++ b/lib/fluent/plugin/kubernetes_metadata_cache_strategy.rb @@ -30,8 +30,10 @@ def get_pod_metadata(key, namespace_name, pod_name, record_create_time, batch_mi (m.nil? || m.empty?) ? {'pod_id'=>ids[:pod_id]} : m end metadata.merge!(@namespace_cache.fetch(ids[:namespace_id]) do - @stats.bump(:namespace_cache_miss) - m = fetch_namespace_metadata(namespace_name) unless @skip_namespace_metadata + m = unless @skip_namespace_metadata + @stats.bump(:namespace_cache_miss) + fetch_namespace_metadata(namespace_name) + end (m.nil? || m.empty?) ? {'namespace_id'=>ids[:namespace_id]} : m end) else