docs: correct MAX_REORG_DEPTH default (512 → 96) - #22068
Merged
Merged
Conversation
The configuring-erigon reference listed MAX_REORG_DEPTH's default as 512, but
the code default is 96 (common/dbg/experiments.go: MaxReorgDepth =
EnvUint("MAX_REORG_DEPTH", 96)). The 512 value only appears in test fixtures.
Regenerated the llms.txt artifacts.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Contributor
There was a problem hiding this comment.
Pull request overview
This PR corrects the documented default value for the MAX_REORG_DEPTH environment variable in the Erigon configuration reference to match the actual runtime default used in release/3.5 (96), and updates the generated LLM documentation artifacts accordingly.
Changes:
- Update
MAX_REORG_DEPTHdocumented default from512to96inconfiguring-erigon.mdx. - Regenerate/update
llms-full.txtartifacts to reflect the corrected default.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
docs/site/docs/fundamentals/configuring-erigon.mdx |
Fixes the MAX_REORG_DEPTH default shown in the env-var reference to 96. |
llms-full.txt |
Updates generated LLM artifact content to match the corrected doc default. |
docs/site/static/llms-full.txt |
Updates the docs-site published LLM artifact to match the corrected doc default. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
yperbasis
approved these changes
Jun 29, 2026
pull Bot
pushed a commit
to Dustin4444/erigon
that referenced
this pull request
Jun 29, 2026
The `configuring-erigon` env-var reference documents `MAX_REORG_DEPTH`
with `default: 512`, but the actual code default on `main` is **96**:
```
common/dbg/experiments.go:39
MaxReorgDepth = EnvUint("MAX_REORG_DEPTH", 96)
```
The `512` value only appears in test fixtures. Corrects the documented
default and regenerates the `llms.txt` / `llms-full.txt` artifacts.
Verified against `origin/main` source. `docs/site` build is green;
`generate-llms.py --check` passes. The matching `release/3.5` fix is
erigontech#22068 (dual-commit rule).
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-authored-by: Bloxster <gianni.morselli@erigon.tech>
Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
yperbasis
added a commit
that referenced
this pull request
Jul 10, 2026
Adds the **v3.5.1** release notes to `ChangeLog.md`, following the v3.4.1–v3.4.4 point-release format (codename **Tidal Tails**, kept from 3.5.0; date left as `TBD` pending the release). Scope: user-facing **Bugfixes** and **Improvements** merged to `release/3.5` since the `v3.5.0` tag. Each entry cites the `[r3.5]` PR that landed on the branch and credits the original fix author, matching the existing point-release style. The six issues closed in [milestone 3.5.1](https://github.com/erigontech/erigon/milestone/78) are all covered: #21992 & #22101 (commitment consistency), #22013 (`stage_custom_trace` prune), #22275 (snapshot reset), #22337 (Bellatrix reconstruction loop), #22351 (forkchoice `GetHead` stall). Deliberately omitted as minor/internal/tooling/docs-only: - `cmd/integration` / `seg` build tweaks (#22137, #22262), QA sync-wait bump (#22280), docs & changelog commits (#22062, #22068, #22090), downloader concurrency hardening (#22319). - The **net-zero** rcache pair — #22047 (bump rcache domain to v3.1) and its revert #22207 — cancels out, so it produces no 3.5.0→3.5.1 change. Opened as a **draft**: fill in the release date on tag, and confirm nothing else is expected to land in 3.5.1 before finalizing. _No `make lint` run — the diff is markdown-only (`ChangeLog.md`), which golangci-lint does not scan._
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The
configuring-erigonenv-var reference documentsMAX_REORG_DEPTHwithdefault: 512, but the actual code default onrelease/3.5is 96:The
512value only appears in test fixtures. This corrects the documented default and regenerates thellms.txt/llms-full.txtartifacts.Verified against
origin/release/3.5source.docs/sitebuild is green;generate-llms.py --checkpasses. A matching fix formainis in a separate PR (dual-commit rule).🤖 Generated with Claude Code