Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Handle critical errors #181

Merged
merged 14 commits into from
Feb 18, 2025
Merged

Handle critical errors #181

merged 14 commits into from
Feb 18, 2025

Conversation

idky137
Copy link
Contributor

@idky137 idky137 commented Feb 4, 2025

Adds automatic server restart for critical errors.

  • propagates errors from child processes to Indexer.
  • Updates Indexer::serve and Indexer::start to return IndexerErrors properly.
  • Updates main to restart on Critical errors.

PR details:

@idky137 idky137 added the ZGM2 Issues that need to be resolved for the completion of the Zaino dev grant milestone 2 label Feb 4, 2025
@idky137 idky137 changed the title Critical errors Handle critical errors Feb 6, 2025
@idky137 idky137 marked this pull request as ready for review February 6, 2025 15:54
@idky137 idky137 marked this pull request as draft February 13, 2025 14:33
zaino-state/src/fetch.rs Outdated Show resolved Hide resolved
zaino-state/src/fetch.rs Outdated Show resolved Hide resolved
@idky137 idky137 marked this pull request as ready for review February 15, 2025 17:48
@idky137 idky137 mentioned this pull request Feb 17, 2025
@idky137
Copy link
Contributor Author

idky137 commented Feb 18, 2025

Helper fn merged, Current test failures in zaino_testutils, zaino_state::fetch and integration_tests::wallet_to_validator are fixed in #186.

@AloeareV AloeareV merged commit 091b8c0 into zingolabs:dev Feb 18, 2025
4 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ZGM2 Issues that need to be resolved for the completion of the Zaino dev grant milestone 2
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Task 2.2.3: Add automatic server restart / shutdown for critical errors.
2 participants