(trafficserver) branch master updated: Make sure string metrics are dumped in RecDumpRecords (#13074)

[cm...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

cmcfarlen pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/master by this push:
     new e7e30bbea9 Make sure string metrics are dumped in RecDumpRecords (#13074)
e7e30bbea9 is described below

commit e7e30bbea9553d34b38247e96d1cc5128d8aaa60
Author: Chris McFarlen <ch...@mcfarlen.us>
AuthorDate: Wed Apr 15 18:07:09 2026 -0500

    Make sure string metrics are dumped in RecDumpRecords (#13074)
---
 src/records/CMakeLists.txt                    | 10 +++-
 src/records/RecCore.cc                        |  7 +++
 src/records/unit_tests/test_RecDumpRecords.cc | 81 +++++++++++++++++++++++++++
 3 files changed, 96 insertions(+), 2 deletions(-)

diff --git a/src/records/CMakeLists.txt b/src/records/CMakeLists.txt
index e168b7dce1..6412c6cee9 100644
--- a/src/records/CMakeLists.txt
+++ b/src/records/CMakeLists.txt
@@ -45,8 +45,14 @@ target_link_libraries(
 
 if(BUILD_TESTING)
   add_executable(
-    test_records unit_tests/unit_test_main.cc unit_tests/test_RecHttp.cc unit_tests/test_RecUtils.cc
-                 unit_tests/test_RecRegister.cc unit_tests/test_ConfigReloadTask.cc unit_tests/test_ConfigRegistry.cc
+    test_records
+    unit_tests/unit_test_main.cc
+    unit_tests/test_RecHttp.cc
+    unit_tests/test_RecUtils.cc
+    unit_tests/test_RecRegister.cc
+    unit_tests/test_ConfigReloadTask.cc
+    unit_tests/test_ConfigRegistry.cc
+    unit_tests/test_RecDumpRecords.cc
   )
   target_link_libraries(test_records PRIVATE records configmanager inkevent Catch2::Catch2 ts::tscore libswoc::libswoc)
   add_catch2_test(NAME test_records COMMAND test_records)
diff --git a/src/records/RecCore.cc b/src/records/RecCore.cc
index 9b5da228b8..c5daa449bf 100644
--- a/src/records/RecCore.cc
+++ b/src/records/RecCore.cc
@@ -28,6 +28,7 @@
 #include "records/RecDefs.h"
 #include "swoc/swoc_file.h"
 
+#include "ts/apidefs.h"
 #include "tscore/ink_platform.h"
 #include "tscore/ink_memory.h"
 #include "tscore/ink_string.h"
@@ -940,6 +941,12 @@ RecDumpRecords(RecT rec_type, RecDumpEntryCb callback, void *edata)
     callback(RECT_PLUGIN, edata, true, name.data(),
              type == Metrics::MetricType::COUNTER ? TS_RECORDDATATYPE_COUNTER : TS_RECORDDATATYPE_INT, &datum);
   }
+
+  ts::Metrics::StaticString::instance().for_each([&](const std::string &name, const std::string &value) {
+    datum.rec_string = const_cast<char *>(value.c_str());
+
+    callback(RECT_PLUGIN, edata, true, name.data(), TS_RECORDDATATYPE_STRING, &datum);
+  });
 }
 
 void
diff --git a/src/records/unit_tests/test_RecDumpRecords.cc b/src/records/unit_tests/test_RecDumpRecords.cc
new file mode 100644
index 0000000000..ba9144f9a4
--- /dev/null
+++ b/src/records/unit_tests/test_RecDumpRecords.cc
@@ -0,0 +1,81 @@
+/** @file
+
+   Catch-based tests for RecDumpRecords
+
+   @section license License
+
+   Licensed to the Apache Software Foundation (ASF) under one or more contributor license agreements.
+   See the NOTICE file distributed with this work for additional information regarding copyright
+   ownership.  The ASF licenses this file to you under the Apache License, Version 2.0 (the
+   "License"); you may not use this file except in compliance with the License.  You may obtain a
+   copy of the License at
+
+      http://www.apache.org/licenses/LICENSE-2.0
+
+   Unless required by applicable law or agreed to in writing, software distributed under the License
+   is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express
+   or implied. See the License for the specific language governing permissions and limitations under
+   the License.
+ */
+#include <catch2/catch_test_macros.hpp>
+#include <string>
+#include <vector>
+
+#include "../P_RecCore.h"
+#include "tsutil/Metrics.h"
+
+struct DumpEntry {
+  RecT        rec_type;
+  int         registered;
+  std::string name;
+  int         data_type;
+  std::string string_value;
+  RecInt      int_value{0};
+};
+
+static void
+collect_callback(RecT rec_type, void *edata, int registered, const char *name, int data_type, RecData *datum)
+{
+  auto     *entries = static_cast<std::vector<DumpEntry> *>(edata);
+  DumpEntry entry;
+
+  entry.rec_type   = rec_type;
+  entry.registered = registered;
+  entry.name       = name;
+  entry.data_type  = data_type;
+
+  if (data_type == RECD_STRING && datum->rec_string != nullptr) {
+    entry.string_value = datum->rec_string;
+  } else if (data_type == RECD_INT || data_type == RECD_COUNTER) {
+    entry.int_value = datum->rec_int;
+  }
+
+  entries->push_back(std::move(entry));
+}
+
+TEST_CASE("RecDumpRecords - StaticString metrics", "[librecords][RecDump]")
+{
+  const std::string test_name  = "proxy.test.dump.string_metric";
+  const std::string test_value = "test_string_value";
+
+  ts::Metrics::StaticString::createString(test_name, test_value);
+
+  std::vector<DumpEntry> entries;
+
+  RecDumpRecords(RECT_NULL, collect_callback, &entries);
+
+  bool found = false;
+
+  for (const auto &entry : entries) {
+    if (entry.name == test_name) {
+      found = true;
+      CHECK(entry.rec_type == RECT_PLUGIN);
+      CHECK(entry.registered == 1);
+      CHECK(entry.data_type == RECD_STRING);
+      CHECK(entry.string_value == test_value);
+      break;
+    }
+  }
+
+  REQUIRE(found);
+}