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

[Prover] ProvingBroker resolves conflicts when same jobs is worked on by two agents #9531

Closed
Tracked by #8495
alexghr opened this issue Oct 29, 2024 · 0 comments · Fixed by #9400
Closed
Tracked by #8495

[Prover] ProvingBroker resolves conflicts when same jobs is worked on by two agents #9531

alexghr opened this issue Oct 29, 2024 · 0 comments · Fixed by #9400
Assignees
Milestone

Comments

@alexghr
Copy link
Contributor

alexghr commented Oct 29, 2024

In case two agents are working on the same job, the broker must be able to resolve conflicts:

  • it should accept the first proof that it receives
  • it should inform the other agent that the job is no longer available the next time the agent checks in

The only way the system should end up in this situation is if the broker goes down while a proof is in progress and when the broker comes back online it hands off the same request to a different agent.

@alexghr alexghr changed the title ProvingBroker resolves conflicts when same jobs is worked on by two agents [Prover] ProvingBroker resolves conflicts when same jobs is worked on by two agents Oct 29, 2024
@alexghr alexghr added this to A3 Nov 12, 2024
@github-project-automation github-project-automation bot moved this to Todo in A3 Nov 12, 2024
@alexghr alexghr added this to the TestNet milestone Nov 12, 2024
@alexghr alexghr self-assigned this Nov 12, 2024
@alexghr alexghr closed this as completed Nov 18, 2024
@github-project-automation github-project-automation bot moved this from Todo to Done in A3 Nov 18, 2024
@alexghr alexghr linked a pull request Nov 18, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

1 participant