Skip to content

chore(helm): bump appVersion to v4.4.4#3432

Merged
nicktrn merged 2 commits intomainfrom
devin/1776950524-helm-bump-v4.4.4
Apr 23, 2026
Merged

chore(helm): bump appVersion to v4.4.4#3432
nicktrn merged 2 commits intomainfrom
devin/1776950524-helm-bump-v4.4.4

Conversation

@devin-ai-integration
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot commented Apr 23, 2026

✅ Checklist

  • I have followed every step in the contributing guide
  • The PR title follows the convention.
  • I ran and tested the code works

Testing

Chart lint / template

  • helm dependency build + helm lint pass cleanly.
  • helm template renders successfully; the only diff vs. the 4.0.6 render is the chart labels and triggerdotdev/trigger.dev:v4.4.4 / triggerdotdev/supervisor:v4.4.4 image refs — no structural differences.

Fresh install on a local kind cluster

Ran hosting/k8s/setup-kind.shhelm install trigger ./hosting/k8s/helm -n trigger on a single-node kind cluster (with a light resources override so everything fits on 2 CPU / 8 GB).

  • All 8 pods reach Running / Ready: webapp (2/2), supervisor, postgres, redis, clickhouse, electric, minio, minio-console.
  • Webapp init ran cleanly:
    • 757 Prisma migrations applied ("All migrations have been successfully applied.").
    • goose up on ClickHouse succeeded to schema version 28.
  • RUN_REPLICATION_ENABLED=1 established the logical-replication slot task_runs_to_clickhouse_v1 against the deployed Postgres automatically.
  • Supervisor registered against the webapp's bootstrap worker group (managed token synced via the token-syncer sidecar), WS connected to platform, failed-pod handler + pod-cleaner informers healthy.
  • All 6 helm test suites pass: test-clickhouse, test-electric, test-postgres, test-redis, test-s3, test-webapp (webapp /healthcheck green).

App-level compatibility audit (v4.0.4 → v4.4.4, 532 commits, 272 touching webapp/supervisor)

  • Webapp env audit (apps/webapp/app/env.server.ts): every newly added variable is either .optional() or has a default. Only CLICKHOUSE_URL is unconditionally required, and the chart already wires it via include "trigger-v4.clickhouse.url". Cross-checked in the reverse direction: no env var the chart currently sets has been removed or renamed (APP_LOG_LEVEL / DATABASE_HOST are still read via process.env, and CLICKHOUSE_PASSWORD is shell-substituted into CLICKHOUSE_URL when using an external-secret ClickHouse).
  • Supervisor env audit (apps/supervisor/src/env.ts): all new settings have defaults. The new conditional requirements in the superRefine (TRIGGER_METADATA_URL, TRIGGER_WORKLOAD_API_DOMAIN) only trigger when COMPUTE_SNAPSHOTS_ENABLED=true, which defaults off; TRIGGER_WORKLOAD_API_DOMAIN is already set by the chart unconditionally.
  • Supervisor ↔ webapp wire protocol (packages/core/src/v3/runEngineWorker/supervisor/ + apps/webapp/app/routes/engine.v1.worker-actions.*): only additive changes. The new ComputeWorkloadManager path is gated by the hasComputeAccess feature flag and COMPUTE_SNAPSHOTS_ENABLED, both off by default.
  • Run queue / Redis (fix(run-queue): prevent concurrency keys from bloating master queue shards #3219, fix(run-queue): prevent concurrency keys from bloating master queue shards): commit message explicitly calls out "Old-format entries drain naturally during rollout — no migration step needed, single deploy." No keyspace migration required.
  • ClickHouse schema: +22 new goose migrations between 4.0.4 and 4.4.4, all applied automatically on webapp container startup via docker/scripts/entrypoint.sh (no manual step).
  • Postgres schema: ~30 new Prisma migrations, applied automatically via db:migrate:deploy in the entrypoint. Includes one FK-drop migration (metadata-only, fast) and a covering index created CONCURRENTLY (non-blocking).
  • Electric: already pinned to 1.2.4 in the existing chart values.yaml; the webapp's Electric client was upgraded in-step.
  • Object Storage (Object Storage seamless migration #3275, seamless multi-provider migration): fully backwards-compatible. All existing OBJECT_STORE_* env vars remain optional with unchanged names. OBJECT_STORE_DEFAULT_PROTOCOL is an optional opt-in for zero-downtime migrations to new providers. Users who were working on v4.0.4 continue to work unchanged on v4.4.4.

Upgrade notes

None required. No manual migration, no breaking values/env-var change, and no new required configuration for existing deployments. First webapp rollout after upgrade will spend extra time applying ~30 Prisma + 22 ClickHouse migrations on startup; beyond the longer init window there's no operator action to take.


Changelog

Bump the Helm chart appVersion and version from v4.0.4 / 4.0.6 to v4.4.4 / 4.4.4 so users pull the latest released webapp and supervisor images by default, with chart version kept in sync with appVersion. No values.yaml changes are required — all new env vars introduced between 4.0.4 and 4.4.4 are optional or covered by existing chart defaults, and ClickHouse remains a required dependency wired end-to-end.

Link to Devin session: https://app.devin.ai/sessions/742fb351b3df4811b15ebaf1963669f4

Co-Authored-By: nick <55853254+nicktrn@users.noreply.github.com>
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Apr 23, 2026

⚠️ No Changeset found

Latest commit: 19ff45d

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 23, 2026

🧭 Helm Chart Prerelease Published

Version: 4.4.4-pr3432.19ff45d

Install:

helm upgrade --install trigger \
  oci://ghcr.io/triggerdotdev/charts/trigger \
  --version "4.4.4-pr3432.19ff45d"

⚠️ This is a prerelease for testing. Do not use in production.

@devin-ai-integration
Copy link
Copy Markdown
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

devin-ai-integration[bot]

This comment was marked as resolved.

Co-Authored-By: nick <55853254+nicktrn@users.noreply.github.com>
@devin-ai-integration devin-ai-integration Bot force-pushed the devin/1776950524-helm-bump-v4.4.4 branch from 7f33ffb to 19ff45d Compare April 23, 2026 13:59
@nicktrn nicktrn added the ready label Apr 23, 2026
@nicktrn nicktrn enabled auto-merge (squash) April 23, 2026 16:20
@nicktrn
Copy link
Copy Markdown
Collaborator

nicktrn commented Apr 23, 2026

ready

@nicktrn nicktrn merged commit cbb1f35 into main Apr 23, 2026
9 checks passed
@nicktrn nicktrn deleted the devin/1776950524-helm-bump-v4.4.4 branch April 23, 2026 16:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants