label size extension

2026-07-07T13:37:15Z

**Было:** Prometheus decoder ограничивает длину значений меток метрик максимум 256 символами. Попытка передать метрику с более длинным значением метки приводит к исключению `TPrometheusDecodeException`. **Стало:** Максимальная длина значения метки расширена до 1024 символов. Системы, которые генерируют метрики с более длинными значениями (например, с ID источников данных, URL, или сложных идентификаторов), теперь могут успешно передавать такие метрики в Unified Agent без ошибок. **Практический эффект:** Улучшение совместимости с внешними системами мониторинга и приложениями, которые требуют передачи длинных меток. Существующие системы с метками < 256 символов продолжают работать без изменений. --- ## С точки зрения разработчика ### Структурные изменения Расширение требовало изменения способа хранения длины строки в пуле коротких строк: 1. **Prometheus decoder** (library/cpp/monlib): - `MAX_LABEL_VALUE_LEN`: 256 → 1024 2. **Short string pool** (logbroker/unified_agent/common): - `MaxLabelSize`: 255 → 1024 - **Ключевое изменение:** Размер строки теперь хранится как `ui16` (2 байта) вместо `unsigned char` (1 байт) - Было: `*slot->Payload() = static_cast(s.size())` - Стало: `memcpy(slot->Payload(), &size, sizeof(ui16))` - Это позволяет хранить строки до 65535 символов (но логически ограничено на уровне валидации) ### Изменённые файлы | Файл | Основные изменения | |------|------------------| | `prometheus_decoder.cpp` | `MAX_LABEL_VALUE_LEN: 256 → 1024` | | `prometheus_decoder_ut.cpp` | +2 юнит-теста для новых граничных значений | | `short_string_pool.h` | `MaxLabelSize: 255 → 1024`, operator[] использует ui16 для чтения размера | | `short_string_pool.cpp` | Allocation и copy логика адаптирована для ui16 | ### Тестирование Добавлены два новых юнит-теста: - **LabelValueAtNewLimitIsAccepted:** Метрика с меткой ровно 1023 символа успешно парсится ``` "m{l=\"" + string(1023, 'a') + "\"} 1\n" ``` - **LabelValueOverNewLimitStillThrows:** Метрика с меткой ≥1024 символов выбрасывает исключение ``` "m{l=\"" + string(1024, 'a') + "\"} 1\n" // → TPrometheusDecodeException: "trying to parse too long label value, size >= 1024" ``` ### Безопасность и совместимость - ✅ Обратно совместимо: строки < 256 символов работают так же - ✅ Валидация: лимит 1024 проверяется на уровне Ref() в пуле - ✅ Граничные случаи: добавлены тесты для 1023 и 1024 символов - ✅ Выделение памяти: правильно рассчитано с учётом новых 2 байт для размера - ✅ Я не нашел ситуации когда закодированная строка (длина + строка) улетала куда-то из пула и потом пыталась раскодироваться. Так что считаю, что миграции никакой не надо и все обратно совместимо. НАДЕЮСЬ Я НЕ ОШИБАЮСЬ commit_hash:b30852954951bd8e65d69777a59ffeb58da76b47

[library/cpp/monlib] add v3 spack encode/decode

2026-04-06T14:08:32Z

commit_hash:91ce0e2023d29ef78fadeaded314ea236f316b8e

[cpp/monlib] add ability to mangle name in prometheus decoder

2026-03-02T10:04:29Z

commit_hash:ef6584bf0941f50d0833dacd3b5d14cfc1cd30a8

Enhance library/cpp/monlib exception verbosity for debugging

2026-02-27T08:46:39Z

Этот эксепшн ронял YDB RU PRESTABLE минимум дважды: \- 2025-09-10 \- 2026-02-19 Патч призван упростить отладку падений и поиски руткоза commit_hash:72d859bae3ab265a2807e566c763bfb70a92a61a

Fix prometheus nan parsing

2025-12-12T14:14:17Z

commit_hash:ea34eba995ae4c386785fe89df58c439a2e3ea63

[fetcher] use lz4 encoding

2025-09-01T10:37:50Z

[fetcher] use lz4 encoding commit_hash:e29504c1f68be2a5d138dbba0ed461b2f2568b97

monlib : memOnly support for JSON encoding/decoding (cpp)

2025-03-06T11:53:44Z

json encoder memonly attempt commit_hash:0cef79f66356a94c398c76dc9baf2fe3ea9a6948

monlib : adding memOnly flag support for cpp lib

2025-03-03T09:35:24Z

commit_hash:cffc55ecd6d0ea22c3c2ce52f21e6aba6da16a15

Intermediate changes

2025-01-29T21:24:09Z

commit_hash:c284e5d683d750f2d89e1bcccb3f4d334b47d60c

Intermediate changes

2025-01-28T08:56:03Z

commit_hash:93f812fd528231c67adb63e20e083855d68bfe8d

ydb/library/cpp/monlib/encode, branch main

label size extension

[library/cpp/monlib] add v3 spack encode/decode

[cpp/monlib] add ability to mangle name in prometheus decoder

Enhance library/cpp/monlib exception verbosity for debugging

Fix prometheus nan parsing

[fetcher] use lz4 encoding

monlib : memOnly support for JSON encoding/decoding (cpp)

monlib : adding memOnly flag support for cpp lib

Intermediate changes

Intermediate changes