Skip to content

chore: upgrade jaegertracing to v2.8.0 #2389

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 7 commits into
base: main
Choose a base branch
from
Draft

chore: upgrade jaegertracing to v2.8.0 #2389

wants to merge 7 commits into from

Conversation

erka
Copy link

@erka erka commented Jul 23, 2025 •
edited
Loading

Changes

Upgrade Jaeger to v2.8.0

Merge Requirements

For new features contributions, please make sure you have completed the following
essential items:

  • CHANGELOG.md updated to document new feature additions
  • Appropriate documentation updates in the [docs][]
  • Appropriate Helm chart updates in the [helm-charts][]

@linux-foundation-easycla Linux Foundation: EasyCLA
Copy link

linux-foundation-easycla bot commented Jul 23, 2025 •
edited
Loading

CLA Signed

The committers listed above are authorized under a signed CLA.

@github-actions github-actions bot added the helm-update-required Requires an update to the Helm chart when released label Jul 23, 2025
julianocosta89
julianocosta89 reviewed Jul 24, 2025
View reviewed changes
Copy link
Member

@julianocosta89 julianocosta89 left a comment

Choose a reason for hiding this comment

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

Hey @erka thx for that!
I know that the PR is still in draft but I wanted to take a look and left some comments.

Let me know when that is ready and I can take a closer look and test it out!

@erka
Copy link
Author

erka commented Jul 24, 2025

@julianocosta89 While it works fine with docker compose, the Helm chart for Jaeger v2 hasn’t been officially released yet. I’m not sure what the best workaround is to use the jaegertracing/helm-charts@v2 branch with helm in this case. Do you have any advice?

@julianocosta89
Copy link
Member

@julianocosta89 While it works fine with docker compose, the Helm chart for Jaeger v2 hasn’t been officially released yet. I’m not sure what the best workaround is to use the jaegertracing/helm-charts@v2 branch with helm in this case. Do you have any advice?

good question!

@puckpuck any ideas?

@github-actions GitHub Actions
Copy link

github-actions bot commented Aug 1, 2025

This PR was marked stale due to lack of activity. It will be closed in 7 days.

@github-actions github-actions bot added the Stale label Aug 1, 2025
@github-actions GitHub Actions
Copy link

github-actions bot commented Aug 9, 2025

This PR was marked stale due to lack of activity. It will be closed in 7 days.

@github-actions github-actions bot added the Stale label Aug 9, 2025
@github-actions github-actions bot removed the Stale label Aug 13, 2025
@github-actions GitHub Actions
Copy link

This PR was marked stale due to lack of activity. It will be closed in 7 days.

@github-actions github-actions bot added the Stale label Aug 21, 2025
@github-actions GitHub Actions
Copy link

github-actions bot commented Sep 2, 2025

This PR was marked stale due to lack of activity. It will be closed in 7 days.

@github-actions github-actions bot added the Stale label Sep 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@julianocosta89 julianocosta89 julianocosta89 left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
helm-update-required Requires an update to the Helm chart when released Stale
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@erka @julianocosta89