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

Compute worker should also submit commitment to BFT roothash #1807

Closed
kostko opened this issue Jun 12, 2019 · 0 comments · Fixed by #3017
Closed

Compute worker should also submit commitment to BFT roothash #1807

kostko opened this issue Jun 12, 2019 · 0 comments · Fixed by #3017
Assignees
Labels
c:roothash Category: root hash service c:runtime/compute Category: runtime compute worker c:security Category: security issues

Comments

@kostko
Copy link
Member

kostko commented Jun 12, 2019

The compute worker currently only submits its commitment to the merge committee. It should also submit it to BFT roothash by using ComputeCommit after a timeout has elapsed and no new blocks have been seen.

This serves the purpose of time-stamping the compute commitment in case the merge committee claims it didn't see it. (Note that this should only happen in case the compute worker is unable to reach any honest merge commitee nodes.)

This requires that we store the write logs in storage and/or use the 3-way merge of roots at the merge committee (e.g., the compute committee cannot just send write logs to the merge committee).

@kostko kostko added c:runtime/compute Category: runtime compute worker c:roothash Category: root hash service c:security Category: security issues labels Jun 12, 2019
@kostko kostko self-assigned this Jun 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c:roothash Category: root hash service c:runtime/compute Category: runtime compute worker c:security Category: security issues
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant