Skip to content

Bump tracing-app chart version to 0.2.6#530

Merged
vitramir merged 2 commits into
mainfrom
fix-tracing-app-chart-version
May 25, 2026
Merged

Bump tracing-app chart version to 0.2.6#530
vitramir merged 2 commits into
mainfrom
fix-tracing-app-chart-version

Conversation

@rowan-stein
Copy link
Copy Markdown
Collaborator

@rowan-stein rowan-stein commented May 25, 2026

Closes #528.

Bumps tracing_app_chart_version to 0.2.8 (no separate image tag pin) so the provisioned E2E stack runs the tracing-app release containing agynio/tracing-app#45.

Release: https://github.com/agynio/tracing-app/releases/tag/v0.2.8

@rowan-stein rowan-stein requested a review from a team as a code owner May 25, 2026 18:37
Copy link
Copy Markdown

@noa-lucent noa-lucent left a comment

Choose a reason for hiding this comment

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

Review in progress.

Copy link
Copy Markdown

@noa-lucent noa-lucent left a comment

Choose a reason for hiding this comment

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

Thanks for the focused update. I found one blocking issue: the selected tracing-app version does not include the PR #45 fix required by issue #528.

Comment thread stacks/platform/variables.tf Outdated
Copy link
Copy Markdown

@noa-lucent noa-lucent left a comment

Choose a reason for hiding this comment

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

Re-reviewed the update. The prior blocking issue is resolved: tracing_app_chart_version now points to 0.2.8, and the v0.2.8 tracing-app tag/release resolves exactly to the agynio/tracing-app#45 merge commit. terraform fmt -check -recursive passes.

@rowan-stein
Copy link
Copy Markdown
Collaborator Author

CI is green and Noa approved. Merge is blocked by base branch policy (auto-merge disabled); please merge when able.

@vitramir vitramir merged commit 70a52b3 into main May 25, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Tracing-app chart/image update needed for E2E: message deeplink empty state fix

3 participants