{host="11"} {host="21"} кардиналити 2

спасибо. на мой вкус выходит, что здесь достаточно простого слова "количество"

:)

количество метрик не кардиналити

кардиналити это полный набор всех вариатов тегов

мощность множества. к сожалению, соответствующая статья в википедии представляет собой дикий неудобоваримый кирпич.

да. у парней из инфлюксдата есть наудивление толковая девочка пиарщик которая как то рассказывала что же это такое.

на понятных примерах

вполне возможно. но было бы больше толка, если б они наняли ещё и годных программистов.

коллеги, а чего такое внезапно с телеграфом случилось?

он зачем-то стал все слеши в подчеркивания превращать

вот так

хм. у меня нет

1,3,2

$ dpkg -l|grep teleg ii telegraf 1.3.2-1 amd64 Plugin-driven server agent for reporting metrics into InfluxDB.

нашел

func sanitize(value string) string { return invalidNameCharRE.ReplaceAllString(value, "_") }

это в https://github.com/influxdata/telegraf/blob/master/plugins/outputs/prometheus_client/prometheus_client.go

вот в этом коммите: https://github.com/influxdata/telegraf/commit/949072e8dc9f000f77db70bf1daed5408d33f035

ага. у меня версия то 1,3,2 но телегрфа не рестартнут.

такая же бойда

ггг

обнимемся, бро (not gay)

надо снова идти к чуваку

чо, пойти им баг назвездеть штоле

и говорить что он негодяй

чо, пойти им баг назвездеть штоле

ога да.

четыре дня назад все сломали

ой вей. это ведь в кардиналити все пойдет :(

ага

var invalidNameCharRE = regexp.MustCompile([^a-zA-Z0-9_])

маловато

ну надо слеш сюда добавить, да

только имя какое-то инвертированное

это же по идее validNameCharRE

не?

то есть, если я верно понял логику, то все символы, НЕ входящие в этот регексп, будут заменены на "_"

ну в контексте использования всё верно return invalidNameCharRE.ReplaceAllString(value, "_")

ну ок, ладно

го - для роботов, я всегда это говорил :troll:

слава роботам!

https://github.com/influxdata/telegraf/issues/2937

Aleksey плюсуй ;)

ак видимо надо пересобрать с праивльным патчем

и выкатить на сервер

посмотрет ьчего еще поломалось

можно попробовать пулриквезднуть

бляяяяя

пидоры

там еще и минуса нет

все хостнеймы по пизде

ну ёбана, а...

вот я и говорю надо бы подумать какие еще оставить

_-./

а точку заекранить не надо?

надо конечно

у го ж не pcre, там свой путь какой-то

дада. re2

$ ag invalidNameCharRE plugins/outputs/prometheus_client/prometheus_client.go 20:var invalidNameCharRE = regexp.MustCompile(`[^a-zA-Z0-9_]`) 171: return invalidNameCharRE.ReplaceAllString(value, "_")

отлично

оно всего в одном месте используется

они там ебнулись точно.

доброе утро.

еще и минус надо экранировать

ERROR: S client not available

@freeseacher var invalidNameCharRE = regexp.MustCompile(`[^a-zA-Z0-9_\-/\.]`)

https://gist.github.com/Bregor/c48505bc8cad3cd8f17c597e7c8f7c2b

$ go run sanitize.go dots__..._bucks______slashes__///_minuses__---_underscores______pluses_____

точки, минусы и слеши на месте

@freeseacher https://github.com/Bregor/telegraf/blob/master/plugins/outputs/prometheus_client/prometheus_client_test.go#L127-L148

они там специально даже в тестах минусики проверяют

казлы

то есть я боюсь, что пулриквест нихера не примут и баг закроют, как вонтфикс

то есть я боюсь, что пулриквест нихера не примут и баг закроют, как вонтфикс

Я думаю если сделать ссылку на валидацию в самом Прометее.- может поправят

еще б найти ее там...

надо сказать, в прометее куча отдельных валидаторов для разных кейсов

инфлюксовцы совершенно явно беспалева взяли вот это: https://github.com/prometheus/prometheus/blob/master/util/strutil/strconv.go#L23

и даже функцию sanitize слово в слово притырили из этого пакета

https://github.com/prometheus/prometheus/blob/master/util/strutil/strconv.go#L42-L44

но при этом это санитизация _лейблов_

а они применяют ее к именам метрик

не могу найти в прометее регекспа для санитизации имен метрик

@freeseacher, @Civiloid, помогайте ;)

чот походу в прометее вообще не санитизируют имена метрик

как приехали, так и берут

санитизация есть только для лейблов

либо я чего-то глобально не понимаю

опять же, в инпутах есть стандартный плагин, следящий за файлами

ничего мне не мешает в имени файла использовать двоеточие, например

телеграф заменит это на андерскор

https://github.com/influxdata/telegraf/issues/2937#issuecomment-309270989

написал пока такой каммент

но чот не уверен, что это как-то вразумит товарищей...

Прометей принимает как есть, если парсинг не зафейлился

Посмотрите в код либы экспортера

но они же при этом не заменяют кучу символов на андерскоры же