Skip to content

Update docker compose using Jaeger v2#240

Merged
carole-lavillonniere merged 3 commits into
mainfrom
update-jaeger-docker-compose
May 12, 2026
Merged

Update docker compose using Jaeger v2#240
carole-lavillonniere merged 3 commits into
mainfrom
update-jaeger-docker-compose

Conversation

@joe4dev
Copy link
Copy Markdown
Member

@joe4dev joe4dev commented May 11, 2026

Motivation

The previously used jaegertracing/all-in-one:latest image (Jaeger v1) is deprecated and now fails to start, so make otel is broken.
Screenshot 2026-05-08 at 11 57 43

Changes

Upgrades docker-compose.tracing.yaml to Jaeger v2 (jaegertracing/jaeger:latest).

Screenshot 2026-05-08 at 12 10 25
  • Switches to the Jaeger v2 image with Badger embedded storage (7-day span TTL)
  • Adds a named Docker volume (jaeger-data) so traces persist across container restarts
  • Adds restart: unless-stopped and an explicit container_name

joe4dev and others added 2 commits May 11, 2026 11:54
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@joe4dev
Copy link
Copy Markdown
Member Author

joe4dev commented May 11, 2026

I'd love some feedback on https://github.com/localstack/localstack-pro/pull/6934, introducing OTEL=1 in LocalStack together with this fix 🙏 . You can check out feature/otel-flag and test Docker mode together with lstk.

FYI: I provide three tracing tool alternatives in this PR https://github.com/localstack/appinspector-workspace/pull/5

@joe4dev joe4dev marked this pull request as ready for review May 11, 2026 10:03
@joe4dev joe4dev mentioned this pull request May 11, 2026
Copy link
Copy Markdown
Collaborator

@carole-lavillonniere carole-lavillonniere left a comment

Choose a reason for hiding this comment

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

Thanks @joe4dev for the fix and improvement 👏
I just added one minor comment.

I am thinking we should also handle LOCALSTACK_OTEL and treat it as an alias to LSTK_OTEL since it is the name of the env var the emulator is using, so it can ultimately work across all LS products when set in the shell (not in the scope of your PR)

Will test together with your PR soon.

Comment thread docker-compose.tracing.yaml Outdated
@carole-lavillonniere carole-lavillonniere merged commit f43da8d into main May 12, 2026
12 checks passed
@carole-lavillonniere carole-lavillonniere deleted the update-jaeger-docker-compose branch May 12, 2026 10:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants