Skip to content

refactor(observability): new observability infra based on OTeL#464

Merged
FantasyRL merged 18 commits into
west2-online:mainfrom
Becks723:refactor/tracing
Apr 26, 2026
Merged

refactor(observability): new observability infra based on OTeL#464
FantasyRL merged 18 commits into
west2-online:mainfrom
Becks723:refactor/tracing

Conversation

@Becks723

Copy link
Copy Markdown
Contributor

自查 PR 结构

  • PR 标题符合这个格式: <type>(optional scope): <description>

  • 此 PR 标题的描述以用户为导向,足够清晰,其他人可以理解。

  • 我已经对所有 commit 提供了签名(GPG 密钥签名、SSH 密钥签名)

  • 这个 PR 属于强制变更/破坏性更改

如果是,请在 PR 标题中添加 BREAKING CHANGE 前缀,并在 PR 描述中详细说明。

这个 PR 的类型是什么?

这个 PR 做了什么 / 我们为什么需要这个 PR?

一个可以算作 feature 的可观测性基础设施,支持导出为 Otel traces & logs

(可选)这个 PR 解决了哪个/些 issue?

对 Reviewer 预留的一些提醒

@ozline

ozline commented Apr 22, 2026

Copy link
Copy Markdown
Member

可以先修一下 CI

@Becks723

Copy link
Copy Markdown
Contributor Author

unit test 没过好像不是我的问题:\

@FantasyRL

Copy link
Copy Markdown
Member

unit test 没过好像不是我的问题:\

现在没过这单测也能合就是了

@renbaoshuo

Copy link
Copy Markdown
Member

merge 一下主分支,那边单测是好的了现在

Signed-off-by: Cai <13110818005@qq.com>
@codecov

codecov Bot commented Apr 24, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 49.22280% with 98 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
pkg/tracing/provider.go 20.40% 38 Missing and 1 partial ⚠️
pkg/logger/context.go 35.71% 35 Missing and 1 partial ⚠️
pkg/base/client/cache.go 0.00% 5 Missing ⚠️
internal/user/service/bind_friend.go 0.00% 4 Missing ⚠️
pkg/base/client/db.go 0.00% 4 Missing ⚠️
pkg/logger/logger.go 71.42% 4 Missing ⚠️
pkg/tracing/shutdown.go 66.66% 1 Missing and 1 partial ⚠️
internal/academic/service/get_scores.go 0.00% 1 Missing ⚠️
pkg/base/client/rpc.go 0.00% 1 Missing ⚠️
pkg/base/server/config.go 0.00% 1 Missing ⚠️
... and 1 more
@@            Coverage Diff             @@
##             main     #464      +/-   ##
==========================================
- Coverage   62.95%   62.71%   -0.24%     
==========================================
  Files         224      230       +6     
  Lines        5852     6027     +175     
==========================================
+ Hits         3684     3780      +96     
- Misses       2051     2126      +75     
- Partials      117      121       +4     
Flag Coverage Δ
unittest 62.71% <49.22%> (-0.24%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
api/handler/trace_ping.go 100.00% <100.00%> (ø)
internal/oa/service/feedback_service.go 100.00% <100.00%> (ø)
internal/user/service/get_friend_config.go 100.00% <100.00%> (ø)
internal/user/service/reorder_friend.go 100.00% <100.00%> (ø)
pkg/logger/otellog.go 100.00% <100.00%> (ø)
pkg/logger/tracing.go 100.00% <100.00%> (ø)
internal/academic/service/get_scores.go 82.50% <0.00%> (ø)
pkg/base/client/rpc.go 0.00% <0.00%> (ø)
pkg/base/server/config.go 0.00% <0.00%> (ø)
pkg/taskqueue/queue.go 32.72% <92.85%> (+32.72%) ⬆️
... and 7 more

... and 2 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@FantasyRL FantasyRL left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

TL;DR

@linrongda

Copy link
Copy Markdown
Member

为什么这个 docker-compose 是单独开的,而不是合并在 /docker/docker-compose.yml 里面

@FantasyRL FantasyRL merged commit dcce14f into west2-online:main Apr 26, 2026
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants