Skip to content
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

[INLONG-11267][Doc] Add signoz doc #1045

Draft
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

sususama
Copy link

@sususama sususama commented Oct 2, 2024

Fixes #11267

Motivation

Add Signoz Doc

Modifications

Verifying this change

  • Make sure that the change passes the CI checks.

(Please pick either of the following options)

  • This change is a trivial rework/code cleanup without any test coverage.

  • This change is already covered by existing tests, such as:
    (please describe tests)

  • This change added tests and can be verified as follows:

Documentation

  • Does this pull request introduce a new feature? (yes / no)
  • If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)
  • If a feature is not applicable for documentation, explain why?
  • If a feature is not documented yet in this PR, please create a followup issue for adding the documentation

@aloyszhang
Copy link
Contributor

aloyszhang commented Oct 9, 2024

docs should be added in https://github.com/apache/inlong-website/tree/master/i18n/zh-CN/docusaurus-plugin-content-docs/current instead of i18n/zh-CN/docusaurus-plugin-content-docs/version-1.13.0

The English document is also required.

@sususama
Copy link
Author

sususama commented Oct 9, 2024

docs should be added in https://github.com/apache/inlong-website/tree/master/i18n/zh-CN/docusaurus-plugin-content-docs/current instead of i18n/zh-CN/docusaurus-plugin-content-docs/version-1.13.0

The English document is also required.

done


首先在自己的电脑中下载 [SigNoz](https://github.com/SigNoz/signoz) 到本地

这里我们可以根据自己的需求,选择使用 InLong 还是 SigNoz项目中的 OpenTelemetry
Copy link
Contributor

Choose a reason for hiding this comment

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

这个描述有问题吧

Copy link
Author

Choose a reason for hiding this comment

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

这个我想表达的是使用哪个项目中配置的OpenTelemetry,我感觉可以理解这句话的意思


### SigNoz中的OpenTelemetry

如果想要使用 SigNoz 中的 OpenTelemetry 则删除 InLong 项目的 `docker/docker-compose/docker-compose.yml` 中的 OpenTelemetry容器
Copy link
Contributor

@vernedeng vernedeng Oct 16, 2024

Choose a reason for hiding this comment

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

不需要删除吧, inlong docker compose引入opentelemetry是可选的

signoz 也提供容器化的选项吧,不要直接下载

…/signoz_doc

# Conflicts:
#	i18n/zh-CN/docusaurus-plugin-content-docs/current/modules/sort/log_reporting.md
@vernedeng vernedeng marked this pull request as draft October 18, 2024 06:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature] Add adaptation to Signoz
3 participants