Skip to content
New issue

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

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

Already on GitHub? # to your account

feat(host): impl API "/admin/pause" #440

Merged
merged 1 commit into from
Dec 27, 2024

Conversation

keroro520
Copy link
Contributor

@keroro520 keroro520 commented Dec 24, 2024

Add HTTP API "/admin/pause".

Clear pending tasks and cancel all running tasks and aggregation tasks when "/admin/pause" called.

  • Add HTTP endpoint "/admin/pause" and "/admin/unpause"
  • Add a pause flag Prover.pause_flag
  • Handle pause message inside ProofActor
  • Add tests covered HTTP endpoint "/admin/pause"
  • Add tests covered ProofActor.handle_system_pause()
  • Reject request if pause flag is true

TODOs in the future:

Copy link
Contributor

@smtmfft smtmfft left a comment

Choose a reason for hiding this comment

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

LGTM

@keroro520 keroro520 added this pull request to the merge queue Dec 27, 2024
Merged via the queue into taikoxyz:main with commit ddba6b0 Dec 27, 2024
11 checks passed
@keroro520 keroro520 deleted the feat-admin-pause branch December 27, 2024 09:13
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants