Skip to content

fix: bump threads chart to 0.1.2 for multi-arch image#115

Merged
vitramir merged 2 commits intomainfrom
noa/issue-6
Mar 17, 2026
Merged

fix: bump threads chart to 0.1.2 for multi-arch image#115
vitramir merged 2 commits intomainfrom
noa/issue-6

Conversation

@casey-brooks
Copy link
Contributor

@casey-brooks casey-brooks commented Mar 17, 2026

Summary

  • revert threads_chart_version to 0.1.1 (published chart)
  • pin threads_image_tag to v0.1.2 for the multi-arch image override

Testing

  • terraform -chdir=stacks/k8s apply -input=false -auto-approve
  • terraform -chdir=stacks/system apply -input=false -auto-approve
  • terraform -chdir=stacks/routing apply -input=false -auto-approve
  • terraform -chdir=stacks/data apply -input=false -auto-approve
  • terraform -chdir=stacks/platform apply -input=false -auto-approve

Related: agynio/threads#6

@casey-brooks casey-brooks requested a review from a team as a code owner March 17, 2026 23:33
@casey-brooks
Copy link
Contributor Author

Summary

  • bumped threads_chart_version to 0.1.2 to pick up the multi-arch threads chart

Testing

  • terraform -chdir=stacks/k8s apply -input=false -auto-approve
  • terraform -chdir=stacks/system apply -input=false -auto-approve
  • terraform -chdir=stacks/routing apply -input=false -auto-approve
  • terraform -chdir=stacks/data apply -input=false -auto-approve
  • terraform -chdir=stacks/platform apply -input=false -auto-approve

Lint

  • not configured (no lint checks found)

@casey-brooks
Copy link
Contributor Author

Test & Lint Summary

Tests (passed/failed/skipped): 5/0/0

Commands:

  • terraform -chdir=stacks/k8s apply -input=false -auto-approve
  • terraform -chdir=stacks/system apply -input=false -auto-approve
  • terraform -chdir=stacks/routing apply -input=false -auto-approve
  • terraform -chdir=stacks/data apply -input=false -auto-approve
  • terraform -chdir=stacks/platform apply -input=false -auto-approve

Lint: not configured (no lint checks found)

@casey-brooks
Copy link
Contributor Author

Test & Lint Summary

Tests (passed/failed/skipped): 5/0/0

Commands:

  • terraform -chdir=stacks/k8s apply -input=false -auto-approve
  • terraform -chdir=stacks/system apply -input=false -auto-approve
  • terraform -chdir=stacks/routing apply -input=false -auto-approve
  • terraform -chdir=stacks/data apply -input=false -auto-approve
  • terraform -chdir=stacks/platform apply -input=false -auto-approve

Lint: not configured (no lint checks found)

@rowan-stein
Copy link
Collaborator

Ready for Review

Context: agynio/threads#6 — The threads:v0.1.1 image was built without arm64 support, causing pod pull failures on arm64 k3d nodes. Chat also fails since it depends on threads.

Fix: Override the threads image tag to v0.1.2, which was rebuilt with both linux/amd64 and linux/arm64 support. The Helm chart stays at 0.1.1 (already published); only the container image tag is updated.

Verification: The v0.1.2 image manifest was verified to include both architectures during the release workflow run.

Copy link

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

Clean, minimal fix. The override mechanism (threads_image_tagresolved_threads_image_tag) works correctly: the non-empty tag takes precedence over the chart-version fallback, so the chart stays at 0.1.1 while the deployed image moves to the multi-arch v0.1.2. LGTM.

@vitramir vitramir merged commit 25153b3 into main Mar 17, 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.

4 participants