chore: cleanup env vars and self hosting docs by mnafees · Pull Request #4420 · hatchet-dev/hatchet

mnafees · 2026-07-14T16:36:47Z

Description

Gets rid of older env vars that are no longer references anywhere. Updates self hosting docs accordingly for errors.

Specifically these vars have been removed across all references:

SERVER_FLUSH_STRATEGY
SERVER_LIMITS_DEFAULT_CRON_LIMIT
SERVER_TLS_SERVER_NAME
SERVER_LIMITS_DEFAULT_CRON_ALARM_LIMIT
SERVER_TASK_OPERATION_LIMITS_TIMEOUT_LIMIT
SERVER_LIMITS_DEFAULT_SCHEDULE_LIMIT
SERVER_TASK_OPERATION_LIMITS_REASSIGN_LIMIT
SERVER_LIMITS_DEFAULT_SCHEDULE_ALARM_LIMIT
SERVER_TASK_OPERATION_LIMITS_RETRY_QUEUE_LIMIT
SERVER_WAIT_FOR_FLUSH
SERVER_TASK_OPERATION_LIMITS_DURABLE_SLEEP_LIMIT
SERVER_MAX_CONCURRENT
SERVER_DEFAULT_ENGINE_VERSION
SERVER_FLUSH_PERIOD_MILLISECONDS
SERVER_REQUEUE_LIMIT
SERVER_FLUSH_ITEMS_THRESHOLD
SERVER_UPDATE_HASH_FACTOR
DATABASE_MAX_QUEUE_CONNS
SERVER_UPDATE_CONCURRENT_FACTOR
DATABASE_MIN_QUEUE_CONNS
SEED_DEVELOPMENT

Type of change

Documentation change (pure documentation change)
Chore (changes which are not directly related to any business logic)

Checklist

Changes have been:

Tested (unit, integration, or manually with steps specified)
Linted and formatted
Documented (where applicable)

🤖 AI Disclosure

I acknowledge that an LLM was used in the creation of this Pull Request, in accordance with Hatchet's AI_POLICY.md.

Details: [e.g. generating tests, writing docs]

vercel · 2026-07-14T16:36:53Z

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
hatchet-docs	Ready	Preview, Comment	Jul 17, 2026 2:20pm

Copilot

Pull request overview

This PR removes several legacy server/database/limits configuration fields and their environment variable bindings, and updates self-hosting documentation to reflect the current configuration surface (Kubernetes Helm and Docker Compose guidance, plus the configuration options reference).

Changes:

Removed unused config fields and BindEnv mappings from pkg/config/server and pkg/config/database.
Removed obsolete limits fields from pkg/config/limits.
Updated self-hosting docs (Helm quickstart/config, Docker Compose networking note, and the env-var reference page).

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
pkg/config/server/server.go	Removes legacy runtime fields and env var bindings from server config.
pkg/config/limits/limits.go	Removes unused limits fields from the limits config struct.
pkg/config/database/config.go	Removes legacy DB pool/seed env config fields and bindings.
frontend/docs/pages/self-hosting/kubernetes-quickstart.mdx	Updates Helm install command and quickstart job explanation.
frontend/docs/pages/self-hosting/kubernetes-helm-configuration.mdx	Updates Helm values example for seeding and shared config.
frontend/docs/pages/self-hosting/docker-compose.mdx	Updates guidance for gRPC broadcast address port inside the Docker network.
frontend/docs/pages/self-hosting/configuration-options.mdx	Updates and trims the env-var reference list and examples.

github-actions · 2026-07-14T16:48:49Z

⚠️ Optional test failure: The load-deadlock job failed on this PR (optimistic-scheduling=true). This check is non-mandatory and does not block merging, but may be worth investigating. View logs

github-actions · 2026-07-14T16:49:11Z

⚠️ Optional test failure: The load-deadlock job failed on this PR (optimistic-scheduling=false). This check is non-mandatory and does not block merging, but may be worth investigating. View logs

Copilot

Pull request overview

Copilot reviewed 13 out of 13 changed files in this pull request and generated 4 comments.

github-actions · 2026-07-14T16:57:45Z

⚠️ Optional test failure: The load-deadlock job failed on this PR (optimistic-scheduling=false). This check is non-mandatory and does not block merging, but may be worth investigating. View logs

github-actions · 2026-07-14T19:30:30Z

⚠️ Optional test failure: The load-deadlock job failed on this PR (optimistic-scheduling=false). This check is non-mandatory and does not block merging, but may be worth investigating. View logs

abelanger5 · 2026-07-15T13:45:02Z

Two notes:

It looks like we're missing the buffer configuration settings for the mq sub buffer / mq pub buffer

It looks like we don't have the dual writes environment variables here either, and importantly we should probably be disabling the dual writes by default (will need to check with @mrkaye97 on that)

This is just on first glance, but I suspect we haven't caught other cases if I caught this with just a quick look.

If we're disabling dual writes by default, can we just remove those code paths instead? It's effectively the same at this point, and I suspect nobody is manually enabling them because they don't know to do so, and it's been a long time of running with support for that at this point

I am open to removing those code paths but that may severely bloat the PR and its intended outcome. Happy to open a followup PR later though.

cc @abelanger5

That’s fine, follow up PR works for me if we’re good with going that route

abelanger5 · 2026-07-15T13:46:02Z


 ```yaml
-SERVER_GRPC_BROADCAST_ADDRESS: "hatchet-engine:7077"
+SERVER_GRPC_BROADCAST_ADDRESS: "hatchet-engine:7070"


We've tested this? I remember some back and forth on this somewhere, I wonder if it has to do with the networking mode of the docker instance, or perhaps it's just incorrect 🤷‍♂️

Yup, tested this locally

github-actions · 2026-07-16T13:16:46Z

⚠️ Optional test failure: The load-deadlock job failed on this PR (optimistic-scheduling=false). This check is non-mandatory and does not block merging, but may be worth investigating. View logs

github-actions · 2026-07-16T13:16:46Z

⚠️ Optional test failure: The load-deadlock job failed on this PR (optimistic-scheduling=true). This check is non-mandatory and does not block merging, but may be worth investigating. View logs

mnafees added 2 commits July 14, 2026 18:15

cleanup env vars and self hosting docs

27d5ad0

correction

a0b572a

mnafees self-assigned this Jul 14, 2026

github-actions Bot added documentation Improvements or additions to documentation engine Related to the core Hatchet engine labels Jul 14, 2026

mnafees requested review from abelanger5 and Copilot July 14, 2026 16:37

Copilot started reviewing on behalf of mnafees July 14, 2026 16:37 View session

vercel Bot deployed to Preview July 14, 2026 16:39 View deployment

Copilot AI reviewed Jul 14, 2026

View reviewed changes

Comment thread pkg/config/server/server.go

Comment thread frontend/docs/pages/self-hosting/configuration-options.mdx Outdated

deprecate SERVER_DEFAULT_ENGINE_VERSION

b1a568e

github-actions Bot added the github_actions Pull requests that update GitHub Actions code label Jul 14, 2026

mnafees requested a review from Copilot July 14, 2026 16:45

Copilot started reviewing on behalf of mnafees July 14, 2026 16:46 View session

vercel Bot deployed to Preview July 14, 2026 16:49 View deployment

Copilot AI reviewed Jul 14, 2026

View reviewed changes

mnafees added 2 commits July 14, 2026 21:14

remove all env var references

edced17

fmt

e93a589

vercel Bot deployed to Preview July 14, 2026 19:21 View deployment

abelanger5 requested changes Jul 15, 2026

View reviewed changes

mnafees added 2 commits July 16, 2026 14:38

Merge branch 'main' into nafees/docs+env-cleanup

b876f7e

resolve comments

5c0c112

vercel Bot deployed to Preview July 16, 2026 13:07 View deployment

mnafees added 2 commits July 17, 2026 15:00

Merge branch 'main' into nafees/docs+env-cleanup

e4a91c0

remove dual writes

e0a1716

vercel Bot deployed to Preview July 17, 2026 13:22 View deployment

old v1 docs

95fd038

vercel Bot deployed to Preview July 17, 2026 13:31 View deployment

add k8s env missing

016f23c

vercel Bot deployed to Preview July 17, 2026 13:40 View deployment

doc updates

65eb236

mnafees requested review from abelanger5 and mrkaye97 July 17, 2026 14:18

vercel Bot deployed to Preview July 17, 2026 14:20 View deployment

Uh oh!

Conversation

mnafees commented Jul 14, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Type of change

Checklist

Uh oh!

vercel Bot commented Jul 14, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

github-actions Bot commented Jul 14, 2026

Uh oh!

github-actions Bot commented Jul 14, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

github-actions Bot commented Jul 14, 2026

Uh oh!

github-actions Bot commented Jul 14, 2026

Uh oh!

abelanger5 Jul 15, 2026

Choose a reason for hiding this comment

Uh oh!

mrkaye97 Jul 16, 2026

Choose a reason for hiding this comment

Uh oh!

mnafees Jul 16, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

mrkaye97 Jul 16, 2026

Choose a reason for hiding this comment

Uh oh!

abelanger5 Jul 15, 2026

Choose a reason for hiding this comment

Uh oh!

mnafees Jul 16, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

github-actions Bot commented Jul 16, 2026

Uh oh!

github-actions Bot commented Jul 16, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

mnafees commented Jul 14, 2026 •

edited

Loading

vercel Bot commented Jul 14, 2026 •

edited

Loading

mnafees Jul 16, 2026 •

edited

Loading