Bug 27548

Summary: Пересобрать libcollectdclient с COLLECT_DEBUG false
Product: Sisyphus Reporter: Dmitriy Kruglikov <Dmitriy.Kruglikov>
Component: libcollectdclientAssignee: Anton Farygin <rider>
Status: CLOSED FIXED QA Contact: qa-sisyphus
Severity: major    
Priority: P3 CC: asy, at, cas, crux, ender, lav, ldv, mike, qa_viy, rider, shaba, viy
Version: unstable   
Hardware: all   
OS: Linux   
URL: https://github.com/collectd/collectd/issues/105

Description Dmitriy Kruglikov 2012-07-17 10:22:27 MSK
В следствие того, что libcollectdclient собран с COLLECT_DEBUG, в выхлопе collectdctl и collectd-nagios присутсвует отладочная информация, которая явно не нужна.

Пример:
# collectd-nagios -s /var/run/collectd-unixsock -H host.node -n load/load -d shortterm -w 2 -c 10 
send:    --> GETVAL "host.node/load/load"
receive: <-- 3 Values found
receive: <-- shortterm=4.800000e-01
receive: <-- midterm=1.600000e+00
receive: <-- longterm=1.300000e+00
OKAY: 0 critical, 0 warning, 1 okay | shortterm=0.480000;;;;

В интерфейсе Nagios, в ячейке информации о состоянии сервися видим  
send:    --> GGETVAL "host.node/load/load"
вместо 
0 critical, 0 warning, 1 okay 

Аналогично, в выхлопе # collectdctl listval
send:    --> LISTVAL
receive: <-- 196 Values found
receive: <-- 1342505724.426 host.node/conntrack/conntrack
receive: <-- 1342505726.892 host.node/cpu-0/cpu-idle
[скипнуто много]
вместо ожидаемого
host.node/conntrack/conntrack
host.node/cpu-0/cpu-idle
host.node/cpu-0/cpu-interrupt
host.node/cpu-0/cpu-nice
host.node/cpu-0/cpu-softirq
host.node/cpu-0/cpu-steal
и так далее.

Анализ кода src/libcollectdclient/client.c показал наличие
#if COLLECT_DEBUG
# define LCC_DEBUG(...) printf (__VA_ARGS__)
#else
# define LCC_DEBUG(...) /**/
#endif

и, соответственно
LCC_DEBUG ("send:    --> %s\n", command);
или
LCC_DEBUG ("receive: <-- %s\n", buffer);

Где и как устанавливать COLLECT_DEBUG в состояние false мне найти не удалось.
Comment 1 Michael Shigorin 2012-07-17 12:41:46 MSK
Тестовый таск собирается, просьба проверить: http://git.altlinux.org/tasks/75491/

-        --enable-debug \
+        --disable-debug \
Comment 2 Dmitriy Kruglikov 2012-07-17 13:34:15 MSK
(В ответ на комментарий №1)
> Тестовый таск собирается, просьба проверить:
> http://git.altlinux.org/tasks/75491/
>
Проверено. Работает. Спасибо.