Skip to content

Conversation

@domwst
Copy link
Contributor

@domwst domwst commented Jan 26, 2024

Changelog entry

Tracing documentation

Changelog category

  • Documentation (changelog entry is not required)

Additional information

...

@domwst domwst requested a review from a team as a code owner January 26, 2024 08:27
@github-actions github-actions bot added the documentation Improvements or additions to documentation label Jan 26, 2024
@domwst domwst marked this pull request as draft January 26, 2024 08:27
@github-actions
Copy link

✅ Documentation build

Revision builded successful
Revision preview link

Build logs

Warnings (1)

⚠️ Incorrect syntax for notes, file /ru/administration/tracing/external-traces.md

@github-actions
Copy link

✅ Documentation build

Revision builded successful
Revision preview link

Build logs

Warnings (1)

⚠️ Incorrect syntax for notes, file /ru/administration/tracing/external-traces.md

@github-actions
Copy link

github-actions bot commented Feb 8, 2024

✅ Documentation build

Revision built successfully
Revision preview link

Build logs

Warnings (1)

⚠️ Incorrect syntax for notes, file /ru/administration/tracing/setup.md

@github-actions
Copy link

github-actions bot commented Feb 8, 2024

✅ Documentation build

Revision built successfully
Revision preview link

Build logs

Warnings (1)

⚠️ Incorrect syntax for notes, file /ru/administration/tracing/external-traces.md

@github-actions
Copy link

github-actions bot commented Feb 8, 2024

✅ Documentation build

Revision built successfully
Revision preview link

@github-actions
Copy link

github-actions bot commented Feb 8, 2024

✅ Documentation build

Revision built successfully
Revision preview link

@github-actions
Copy link

github-actions bot commented Feb 8, 2024

✅ Documentation build

Revision built successfully
Revision preview link

@github-actions
Copy link

github-actions bot commented Feb 9, 2024

✅ Documentation build

Revision built successfully
Revision preview link

@github-actions
Copy link

github-actions bot commented Feb 9, 2024

❌ Documentation build

Revision build failed

Build logs

Errors (1)

❌ No such file or has no access to /ru/administration/tracing/sampling.md

@github-actions
Copy link

github-actions bot commented Feb 9, 2024

✅ Documentation build

Revision built successfully
Revision preview link

@github-actions
Copy link

✅ Documentation build

Revision built successfully
Revision preview link

@github-actions
Copy link

✅ Documentation build

Revision built successfully
Revision preview link

@@ -0,0 +1,155 @@
# Настройка трассировки {{ ydb-short-name }}

Для настройки трейсинга добавьте в конфигурацию (`cluster.yaml`) следующую секцию:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Это первый и единственный раздел про трассировку YDB. Я бы добавил вступление про трассировку вообще.

  • предпложение зачем нужна трассировка
  • ссылка на то, где с трассировкой можно подробнее ознкомиться (внешний ресурс)
  • мы поддерживаем open tracing? написать про это

Copy link
Contributor Author

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
Copy link
Member

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"?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Описал поведение по умолчанию. В open telemetry нет такого понятия, как throttling – только sampling

@domwst domwst linked an issue Feb 20, 2024 that may be closed by this pull request
@github-actions
Copy link

✅ Documentation build

Revision built successfully
Revision preview link

@github-actions
Copy link

✅ Documentation build

Revision built successfully
Revision preview link

@github-actions
Copy link

✅ Documentation build

Revision built successfully
Revision preview link

@github-actions
Copy link

✅ Documentation build

Revision built successfully
Revision preview link

@github-actions
Copy link

✅ Documentation build

Revision built successfully
Revision preview link

@domwst domwst marked this pull request as ready for review February 28, 2024 08:27
@github-actions
Copy link

✅ Documentation build

Revision built successfully
Revision preview link


Для аггрегации разрозненных спанов в трассы они отправляются в *коллектор*. Это сервис, который аггрегирует и сохраняет получаемые спаны для последующего анализа трасс. {{ ydb-short-name }} не включает в себя этот сервис, администратор должен поднять его самостоятельно. Обычно в качестве коллектора используется [jaeger](https://www.jaegertracing.io/).

Больше про трассировку вы можете прочитать в разделе [observability primer](https://opentelemetry.io/docs/concepts/observability-primer/) сайта opentelemetry.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"сайта opentelemetry" -- это не сайт. В том смысле, что сайт не указан. Я бы перенес ссылку в начало абзаца, а не в конец, там где дается определение трассировки. Там бы и сослался.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Перенес этот абзац в заметку в начало статьи


## Полная конфигурация

Пример более детальной настройки приведен ниже:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Нет явного рассказа про семплинг и external tracing. Это концептуальные вещи, про которые я бы явно писал.
В общем сложно интерпретировать пример без слов, когда видишь первый раз

Copy link
Contributor Author

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 }}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Не понятно из этого документа, мы трассируем любой внеший trace id? Как-то ограниченно?
На мой взгляд странно видеть в документации про трейсинг 2 статьи, одну общую, вторую про внешний трейсинг... Как-то не очень логично для меня...
И очень странно, что про внешний трейсинг в одной статье, а про его настройку в другой... не логично же

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Логика тут такая: первая статья описывает трассировку с точки зрения администратора, вторая описывает трассировку с точки зрения разработчика. Наверное стоит тут сделать отсылку к ограничениям на внешние трассы и написать

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Добавил в статью про передачу trace-id примечание о том, что могут трассироваться не все запросы

@github-actions
Copy link

github-actions bot commented Mar 5, 2024

✅ Documentation build

Revision built successfully
Revision preview link

@github-actions
Copy link

github-actions bot commented Mar 6, 2024

✅ Documentation build

Revision built successfully
Revision preview link

@github-actions
Copy link

github-actions bot commented Mar 6, 2024

✅ Documentation build

Revision built successfully
Revision preview link

@github-actions
Copy link

github-actions bot commented Mar 6, 2024

✅ Documentation build

Revision built successfully
Revision preview link

@github-actions
Copy link

github-actions bot commented Apr 3, 2024

✅ Documentation build

Revision built successfully
Revision preview link

@github-actions
Copy link

github-actions bot commented Apr 3, 2024

✅ Documentation build

Revision built successfully
Revision preview link

@github-actions
Copy link

github-actions bot commented Apr 3, 2024

✅ Documentation build

Revision built successfully
Revision preview link

@github-actions
Copy link

github-actions bot commented Apr 3, 2024

❌ Documentation build

Revision build failed

Build logs

Errors (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

@github-actions
Copy link

github-actions bot commented Apr 3, 2024

❌ Documentation build

Revision build failed

Build logs

Errors (1)

❌ Link is unreachable: ../../maintenance/manual/dynamic-config.md in /ru/reference/observability/tracing/setup.md

@github-actions
Copy link

github-actions bot commented Apr 3, 2024

✅ Documentation build

Revision built successfully
Revision preview link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Write documentation on using tracing.

3 participants