-
Notifications
You must be signed in to change notification settings - Fork 735
Tracing documentation #1331
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
Tracing documentation #1331
Conversation
✅ Documentation buildRevision builded successful Build logsWarnings (1) |
✅ Documentation buildRevision builded successful Build logsWarnings (1) |
✅ Documentation buildRevision built successfully Build logsWarnings (1) |
✅ Documentation buildRevision built successfully Build logsWarnings (1) |
✅ Documentation buildRevision built successfully |
✅ Documentation buildRevision built successfully |
✅ Documentation buildRevision built successfully |
✅ Documentation buildRevision built successfully |
❌ Documentation buildRevision build failed Build logsErrors (1)❌ No such file or has no access to /ru/administration/tracing/sampling.md |
✅ Documentation buildRevision built successfully |
✅ Documentation buildRevision built successfully |
✅ Documentation buildRevision built successfully |
| @@ -0,0 +1,155 @@ | |||
| # Настройка трассировки {{ ydb-short-name }} | |||
|
|
|||
| Для настройки трейсинга добавьте в конфигурацию (`cluster.yaml`) следующую секцию: | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Это первый и единственный раздел про трассировку YDB. Я бы добавил вступление про трассировку вообще.
- предпложение зачем нужна трассировка
- ссылка на то, где с трассировкой можно подробнее ознкомиться (внешний ресурс)
- мы поддерживаем open tracing? написать про это
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Написал небольшое введение про трассировку
- Добавил ссылку на соответствующий раздел opentelemetry
- В следующей секции про минимальную настройку написал, что коллектор должен быть OTEL-совместимым
|
|
||
| {% endnote %} | ||
|
|
||
| ### External throttling |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Не сказано, является ли секция обязательной. Если не обязательна, то какое поведение по умолчанию? Могу я вообще выключить external throttling.
Как в open telemetry называется "external throttling"?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Описал поведение по умолчанию. В open telemetry нет такого понятия, как throttling – только sampling
✅ Documentation buildRevision built successfully |
✅ Documentation buildRevision built successfully |
✅ Documentation buildRevision built successfully |
✅ Documentation buildRevision built successfully |
✅ Documentation buildRevision built successfully |
✅ Documentation buildRevision built successfully |
|
|
||
| Для аггрегации разрозненных спанов в трассы они отправляются в *коллектор*. Это сервис, который аггрегирует и сохраняет получаемые спаны для последующего анализа трасс. {{ ydb-short-name }} не включает в себя этот сервис, администратор должен поднять его самостоятельно. Обычно в качестве коллектора используется [jaeger](https://www.jaegertracing.io/). | ||
|
|
||
| Больше про трассировку вы можете прочитать в разделе [observability primer](https://opentelemetry.io/docs/concepts/observability-primer/) сайта opentelemetry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"сайта opentelemetry" -- это не сайт. В том смысле, что сайт не указан. Я бы перенес ссылку в начало абзаца, а не в конец, там где дается определение трассировки. Там бы и сослался.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Перенес этот абзац в заметку в начало статьи
|
|
||
| ## Полная конфигурация | ||
|
|
||
| Пример более детальной настройки приведен ниже: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Нет явного рассказа про семплинг и external tracing. Это концептуальные вещи, про которые я бы явно писал.
В общем сложно интерпретировать пример без слов, когда видишь первый раз
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Добавил в соответствующие секции краткое описание
| @@ -0,0 +1,10 @@ | |||
| # Передача внешнего trace-id в {{ ydb-short-name }} | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Не понятно из этого документа, мы трассируем любой внеший trace id? Как-то ограниченно?
На мой взгляд странно видеть в документации про трейсинг 2 статьи, одну общую, вторую про внешний трейсинг... Как-то не очень логично для меня...
И очень странно, что про внешний трейсинг в одной статье, а про его настройку в другой... не логично же
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Логика тут такая: первая статья описывает трассировку с точки зрения администратора, вторая описывает трассировку с точки зрения разработчика. Наверное стоит тут сделать отсылку к ограничениям на внешние трассы и написать
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Добавил в статью про передачу trace-id примечание о том, что могут трассироваться не все запросы
✅ Documentation buildRevision built successfully |
✅ Documentation buildRevision built successfully |
✅ Documentation buildRevision built successfully |
✅ Documentation buildRevision built successfully |
✅ Documentation buildRevision built successfully |
✅ Documentation buildRevision built successfully |
✅ Documentation buildRevision built successfully |
❌ Documentation buildRevision build failed Build logsErrors (5)❌ Link is unreachable: ../../concepts/_includes/index/how_it_works.md#ydb-architecture in /ru/reference/observability/tracing/setup.md ❌ Link is unreachable: ../../deploy/configuration/config.md in /ru/reference/observability/tracing/setup.md ❌ Link is unreachable: ../../maintenance/embedded_monitoring/logs.md in /ru/reference/observability/tracing/setup.md ❌ Link is unreachable: ../../maintenance/manual/dynamic-config.md in /ru/reference/observability/tracing/setup.md ❌ Link is unreachable: ../../reference/ydb-sdk/recipes/debug-jaeger.md in /ru/reference/observability/tracing/external-traces.md |
❌ Documentation buildRevision build failed Build logsErrors (1)❌ Link is unreachable: ../../maintenance/manual/dynamic-config.md in /ru/reference/observability/tracing/setup.md |
✅ Documentation buildRevision built successfully |
Changelog entry
Tracing documentation
Changelog category
Additional information
...