-
Notifications
You must be signed in to change notification settings - Fork 705
[feat] [observation] trajectory observe #332
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
base: main
Are you sure you want to change the base?
Conversation
(LogID: 20251127120914010091115104657BA09) Co-Authored-By: Coda <coda@bytedance.com>
Change-Id: I276d102667fc5e195d930d8352972f71803c9847
Change-Id: I278ed8d9f51ecadad41074892144e471360f6ef0
Change-Id: I5491314293e069d38f41c6aa6a7975dd22ac0392
…op into feat_trajectory_observe
Change-Id: Ic088c21db3efdc9115ca00b733ae2145f8244bb9
Change-Id: I33d7db20f81e4bc1059c5b8be2113e8826406395
|
Submission history contains excessive meaningless information, please organize it |
mocayo
left a comment
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.
reorganize commit message and resolve conflicts
CozeLoop
left a comment
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.
Summary: Found compile-blocking issue in otel FieldConf map and a missing middleware function reference in router.
- [🚫Must Fix] backend/modules/observability/lib/otel/otel_convert.go: Struct literal for FieldConf uses non-existent field names (
attributeKey,isTag,dataType). Replace withAttributeKey,IsTag,DataType. - [🚨Should Fix] backend/api/router/coze/loop/apis/coze.loop.apis.go: Route references
_gettrajectoryconfigMwbut the stub is not defined inmiddleware.go. Add the missing middleware function.
Please address these to unblock compilation and ensure routing consistency.
backend/modules/observability/infra/mq/producer/span_with_annotation_producer.go
Show resolved
Hide resolved
backend/modules/observability/domain/trace/entity/loop_span/trajectory.go
Show resolved
Hide resolved
backend/modules/observability/infra/mq/producer/span_with_annotation_producer.go
Show resolved
Hide resolved
Change-Id: I923bfa772de801964c637a9741216dcf383eb1ef
…op into feat_trajectory_observe
002b8ef to
1e3d78c
Compare
09c998d to
2f90dfb
Compare
What type of PR is this?
Check the PR title
(Optional) Translate the PR title into Chinese
(Optional) More detailed description for this PR(en: English/zh: Chinese)
en:
zh(optional):
(Optional) Which issue(s) this PR fixes