Skip to content

[pkg/translator/jaeger] migrate semantic convention from v1.25.0 to v1.40.0#49301

Merged
songy23 merged 17 commits into
open-telemetry:mainfrom
singhvibhanshu:migrateHTTP-45036
Jul 27, 2026
Merged

[pkg/translator/jaeger] migrate semantic convention from v1.25.0 to v1.40.0#49301
songy23 merged 17 commits into
open-telemetry:mainfrom
singhvibhanshu:migrateHTTP-45036

Conversation

@singhvibhanshu

@singhvibhanshu singhvibhanshu commented Jun 24, 2026

Copy link
Copy Markdown
Member

Fixes #45036

http.response.status_code is the replacement atttribute for http.status_code
Here is the reference link: https://opentelemetry.io/docs/specs/semconv/registry/attributes/http/#http-status-code

Authorship

  • I, a human, wrote this pull request description myself.

singhvibhanshu and others added 3 commits June 23, 2026 22:01
@singhvibhanshu
migrate HTTP attributes for jaeger
cdab7d6 
Signed-off-by: singhvibhanshu <find.vibhanshu@gmail.com>
@singhvibhanshu
Merge branch 'main' into migrateHTTP-45036
2b6d0aa
@singhvibhanshu
fix CI
97d2b6a 
Signed-off-by: singhvibhanshu <find.vibhanshu@gmail.com>
@singhvibhanshu
singhvibhanshu marked this pull request as ready for review June 24, 2026 05:40
@singhvibhanshu
singhvibhanshu requested a review from a team as a code owner June 24, 2026 05:40
@singhvibhanshu
singhvibhanshu requested a review from mx-psi June 24, 2026 05:40
@github-actions
github-actions Bot requested a review from frzifus June 24, 2026 05:40
@singhvibhanshu
fixed changelog file
a7a806e 
Signed-off-by: singhvibhanshu <find.vibhanshu@gmail.com>
@singhvibhanshu

singhvibhanshu commented Jun 24, 2026

Copy link
Copy Markdown
Member Author

/rerun

@singhvibhanshu
singhvibhanshu marked this pull request as draft June 24, 2026 12:27
@singhvibhanshu

singhvibhanshu commented Jun 24, 2026

Copy link
Copy Markdown
Member Author

I ran make update-codeowners, but nothing happened.

@singhvibhanshu
singhvibhanshu marked this pull request as ready for review June 26, 2026 07:44
@singhvibhanshu
singhvibhanshu marked this pull request as draft June 26, 2026 07:45
@singhvibhanshu
resolved merge conflicts and updates from version
6598c2f 
Signed-off-by: singhvibhanshu <find.vibhanshu@gmail.com>
@singhvibhanshu
singhvibhanshu marked this pull request as ready for review July 7, 2026 14:09
@singhvibhanshu

singhvibhanshu commented Jul 7, 2026

Copy link
Copy Markdown
Member Author

@mx-psi, @frzifus
Kindly take a look at it as a code-owner.
I am marking this PR as draft due to limitations on number of PRs open at a time, but feel free to give it a look.
Thanks!

cc: @open-telemetry/collector-contrib-approvers

@singhvibhanshu
singhvibhanshu marked this pull request as draft July 7, 2026 15:01
@singhvibhanshu singhvibhanshu mentioned this pull request Jul 8, 2026
Merged
@opentelemetry-pr-dashboard

opentelemetry-pr-dashboard Bot commented Jul 18, 2026 •
edited
Loading

Copy link
Copy Markdown

Pull request dashboard status

Status last refreshed: 2026-07-27 15:38:36 UTC.

  • Status: Merged.

This automated status or its linked feedback items may be incorrect. If something looks wrong, please report it with the result you expected.

@singhvibhanshu
resolved merge conflicts
7f2276e 
Signed-off-by: singhvibhanshu <find.vibhanshu@gmail.com>
@singhvibhanshu

singhvibhanshu commented Jul 19, 2026 •
edited
Loading

Copy link
Copy Markdown
Member Author

I don't understand why CI is explaining? I wrote pkg/jaeger in the changelog file but it is complaining pkg/translator/jaeger is not a valid component. IDK, what's wrong here?
Resolved.

@singhvibhanshu
Merge branch 'main' into migrateHTTP-45036
539a57d
@singhvibhanshu
resolved merge conflicts
94d07f3 
Signed-off-by: singhvibhanshu <find.vibhanshu@gmail.com>
@singhvibhanshu
restored config.yaml
c1d9eeb 
Signed-off-by: singhvibhanshu <find.vibhanshu@gmail.com>
@singhvibhanshu
updated from version
064aa36 
Signed-off-by: singhvibhanshu <find.vibhanshu@gmail.com>
@singhvibhanshu
singhvibhanshu marked this pull request as ready for review July 27, 2026 10:35
@opentelemetry-pr-dashboard opentelemetry-pr-dashboard Bot mentioned this pull request Jul 27, 2026
Open
@singhvibhanshu
updated changelog
78389cc 
Signed-off-by: singhvibhanshu <find.vibhanshu@gmail.com>
frzifus
frzifus approved these changes Jul 27, 2026
View reviewed changes

@frzifus frzifus 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.

LGTM, just one thing on the release note.

Comment thread .chloggen/migrateHTTP-45036.yaml Outdated
@singhvibhanshu
added a note for a future breaking change
25b11db 
Signed-off-by: singhvibhanshu <find.vibhanshu@gmail.com>
@singhvibhanshu

singhvibhanshu commented Jul 27, 2026

Copy link
Copy Markdown
Member Author

Could someone from @open-telemetry/collector-contrib-approvers please take a look at it? 🙏🏻

@singhvibhanshu singhvibhanshu added ready to merge Code review completed; ready to merge by maintainers and removed waiting-for-code-owners labels Jul 27, 2026
@songy23
songy23 merged commit 41d4b00 into open-telemetry:main Jul 27, 2026
158 checks passed
@github-actions github-actions Bot mentioned this pull request Jul 27, 2026
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@iblancasa iblancasa iblancasa approved these changes
@mx-psi mx-psi Awaiting requested review from mx-psi mx-psi is a code owner automatically assigned from open-telemetry/collector-contrib-approvers
+1 more reviewer
@frzifus frzifus frzifus approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

@atoulme atoulme

Labels

pkg/translator/jaeger ready to merge Code review completed; ready to merge by maintainers

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[pkg/translator/jaeger] update semconv from 1.25.0

5 participants

@singhvibhanshu @iblancasa @frzifus @atoulme @songy23