Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

"smp" metrics are disabled #2375

Open
xemul opened this issue Jul 29, 2024 · 2 comments
Open

"smp" metrics are disabled #2375

xemul opened this issue Jul 29, 2024 · 2 comments

Comments

@xemul
Copy link
Contributor

xemul commented Jul 29, 2024

0634989 contains such hunk around "smp" group

@@ -2867,49 +2749,22 @@ size_t smp_message_queue::process_incoming() {
 
 void smp_message_queue::start(unsigned cpuid) {
     _tx.init();
+    namespace sm = seastar::metrics;
     char instance[10];
     std::snprintf(instance, sizeof(instance), "%u-%u", engine().cpu_id(), cpuid);
-    _collectd_regs = scollectd::registrations({
+    _metrics.add_group("smp", {
             // queue_length     value:GAUGE:0:U
             // Absolute value of num packets in last tx batch.
-            scollectd::add_disabled_polled_metric(scollectd::type_instance_id("smp"
-                    , instance
-                    , "queue_length", "send-batch")
-            , scollectd::make_typed(scollectd::data_type::GAUGE, _last_snt_batch)
-            ),
-            scollectd::add_disabled_polled_metric(scollectd::type_instance_id("smp"
-                    , instance
-                    , "queue_length", "receive-batch")
-            , scollectd::make_typed(scollectd::data_type::GAUGE, _last_rcv_batch)
-            ),
-            scollectd::add_disabled_polled_metric(scollectd::type_instance_id("smp"
-                    , instance
-                    , "queue_length", "complete-batch")
-            , scollectd::make_typed(scollectd::data_type::GAUGE, _last_cmpl_batch)
-            ),
-            scollectd::add_disabled_polled_metric(scollectd::type_instance_id("smp"
-                    , instance
-                    , "queue_length", "send-queue-length")
-            , scollectd::make_typed(scollectd::data_type::GAUGE, _current_queue_length)
-            ),
+            sm::make_queue_length("send_batch_queue_length", _last_snt_batch, sm::description("Current send batch queue length"), sm::metric_disabled, instance),
+            sm::make_queue_length("receive_batch_queue_length", _last_rcv_batch, sm::description("Current receive batch queue length"), sm::metric_disabled, instance),
+            sm::make_queue_length("complete_batch_queue_length", _last_cmpl_batch, sm::description("Current complete batch queue length"), sm::metric_disabled, instance),
+            sm::make_queue_length("send_queue_length", _current_queue_length, sm::description("Current send queue length"), sm::metric_disabled, instance),
...

so all the "smp" metrics were marked as "disabled" and they are indeed not reported AFAICT

@amnonh , despite it's been long ago, please comment

@tchaikov
Copy link
Contributor

tchaikov commented Sep 7, 2024

@amnonh hi Amnon, ping?

@amnonh
Copy link
Contributor

amnonh commented Sep 7, 2024

What information do you need? Those metrics are quadratic in their nature and were disabled

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants