in_node_exporter_metrics: Update metrics value only if succeeded

Signed-off-by: Hiroshi Hatake <[email protected]>

Author: cosmo0920

plugins/in_node_exporter_metrics/ne_cpu_linux.c

@@ -152,15 +152,15 @@ static int cpu_thermal_update(struct flb_ne *ctx, uint64_t ts)
                           "CPU is missing core_throttle_count: %s",
                           entry->str);
         }
-
-        snprintf(tmp1, sizeof(tmp1) -1, "%" PRIu64, core_id);
-        snprintf(tmp2, sizeof(tmp2) -1, "%" PRIu64, physical_package_id);
-
-        /* Set new value */
-        cmt_counter_set(ctx->cpu_core_throttles, ts,
-                        (double) core_throttle_count,
-                        2, (char *[]) {tmp1, tmp2});
-
+        else {
+            snprintf(tmp1, sizeof(tmp1) -1, "%" PRIu64, core_id);
+            snprintf(tmp2, sizeof(tmp2) -1, "%" PRIu64, physical_package_id);
+
+            /* Set new value */
+            cmt_counter_set(ctx->cpu_core_throttles, ts,
+                            (double) core_throttle_count,
+                            2, (char *[]) {tmp1, tmp2});
+        }
 
         /* Only update this entry once */
         if (package_throttles_set[physical_package_id] != 0) {
@@ -178,11 +178,12 @@ static int cpu_thermal_update(struct flb_ne *ctx, uint64_t ts)
                           "CPU is missing package_throttle_count: %s",
                           entry->str);
         }
-
-        /* Set new value */
-        cmt_counter_set(ctx->cpu_package_throttles, ts,
-                        (double) package_throttle_count,
-                        1, (char *[]) {tmp2});
+        else {
+            /* Set new value */
+            cmt_counter_set(ctx->cpu_package_throttles, ts,
+                            (double) package_throttle_count,
+                            1, (char *[]) {tmp2});
+        }
     }
     flb_slist_destroy(&list);