Skip to content

add support for core dumps#107

Merged
ch4r10t33r merged 6 commits intomainfrom
crash-dump
Feb 11, 2026
Merged

add support for core dumps#107
ch4r10t33r merged 6 commits intomainfrom
crash-dump

Conversation

@chetanyb
Copy link
Contributor

@chetanyb chetanyb commented Feb 6, 2026

Summary

Adds opt-in core dump support for debugging crashed nodes.

  • New --coreDumps flag for local and Ansible deployments
  • Granular targeting: "all", specific nodes (zeam_0,ream_0), or client types (zeam,qlean)
  • Core dumps persist in <data_dir>/<node_name>/

Changes

Local deployment (parse-env.sh, spin-node.sh):

  • Parse --coreDumps flag
  • Add should_enable_core_dumps() helper for matching logic
  • Apply --init --ulimit core=-1 --workdir /data to matching containers

Ansible deployment (run-ansible.sh, 7 role files):

  • Pass --coreDumps through as enable_core_dumps extra var
  • Add Jinja2 conditional to all client roles (zeam, ream, qlean, lighthouse, lantern, grandine, ethlambda)

Usage

Local

NETWORK_DIR=local-devnet ./spin-node.sh --node all --coreDumps all

Ansible

NETWORK_DIR=local-devnet ./spin-node.sh --node all --deploymentMode ansible --coreDumps zeam,ream

@chetanyb chetanyb marked this pull request as ready for review February 11, 2026 14:37
Copy link
Contributor

@ch4r10t33r ch4r10t33r left a comment

Choose a reason for hiding this comment

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

Are we only looking to support coredump for the docker images only (and not binaries)? If so, can we update the README.md to reflect this?

Copy link
Contributor

@ch4r10t33r ch4r10t33r left a comment

Choose a reason for hiding this comment

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

Looks good.

@ch4r10t33r ch4r10t33r merged commit db058b9 into main Feb 11, 2026
4 checks passed
@ch4r10t33r ch4r10t33r deleted the crash-dump branch February 11, 2026 16:35
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.

2 participants