Skip to content

Conversation

@Garandor
Copy link
Contributor

@Garandor Garandor commented Jan 8, 2026

What

Improve error handling during node startup for PeerState creation and TSS state initialization.

Why

Lack of detailed error information when critical components fail to initialize makes it difficult to diagnose startup issues.

Background

We wasted too much time figuring out why a newly recreated node wasn't able to join the network. If it's due to the AW registration failing, it should now be obvious from the logs

Risks

  • No functional changes to the code logic, only improved error handling
  • The changes maintain the same behavior of panicking on critical initialization failures, but with better error reporting

Copy link
Contributor Author

Garandor commented Jan 8, 2026

This stack of pull requests is managed by Graphite. Learn more about stacking.

@Garandor Garandor marked this pull request as ready for review January 8, 2026 23:21
@Garandor Garandor self-assigned this Jan 8, 2026
@Garandor Garandor changed the title more helpful logs if attested wallet registration fails c Jan 8, 2026
@Garandor Garandor changed the title c Improve error handling for PeerState and TssState initialization failures Jan 8, 2026
@Garandor Garandor changed the title Improve error handling for PeerState and TssState initialization failures Better logging on node init failure due to AW or TSS creation failing Jan 8, 2026
@github-actions
Copy link

github-actions bot commented Jan 8, 2026

PASS [ 43.171s] (2/3) lit_node::test toxiproxy::perf_tests::load_with_50ms_latency_single_link
PASS [ 43.205s] (3/3) lit_node::test toxiproxy::perf_tests::load_with_no_latency
PASS [ 88.342s] (1/3) lit_node::test toxiproxy::perf_tests::load_with_50ms_latency_all_links

Copy link
Contributor Author

Garandor commented Jan 10, 2026

Merge activity

  • Jan 10, 2:13 AM UTC: Graphite rebased this pull request as part of a merge.
  • Jan 10, 2:38 AM UTC: @Garandor merged this pull request with Graphite.

@Garandor Garandor merged commit c697c92 into master Jan 10, 2026
31 checks passed
@Garandor Garandor deleted the inf-935 branch January 10, 2026 02:38
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.

3 participants