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

[CX_HARDENING] - Delete Legacy Consensus #3640

Open
jparr721 opened this issue Sep 3, 2024 · 2 comments · Fixed by #3755
Open

[CX_HARDENING] - Delete Legacy Consensus #3640

jparr721 opened this issue Sep 3, 2024 · 2 comments · Fixed by #3755
Assignees
Labels
cappuccino cappuccino-sprint2 consensus-hardening Hardening the consensus protocol and adaquate testing.

Comments

@jparr721
Copy link
Contributor

jparr721 commented Sep 3, 2024

What is this task and why do we need to work on it?

This is straightforward. Go in and delete the consensus module and clean up the compiler errors. This should include a check of the duplicated functions, however, before removal. In particular

  • update_view
  • handle_quorum_proposal_validated
  • create_and_send_proposal

The purpose of these checks is to make sure that things like upgrade logic etc is included in the new code paths. We also need to rename consensus2 to consensus.

What work will need to be done to complete this task?

No response

Are there any other details to include?

No response

What are the acceptance criteria to close this issue?

  • HotShot only has one consensus module.
  • The dependency tasks are stable and deployed in all relevant environments.

Branch work will be merged to (if not the default branch)

No response

@pls148
Copy link
Contributor

pls148 commented Oct 11, 2024

#3755 erroneously closed this issue, reopening as there will be additional PRs landing here.

@pls148 pls148 reopened this Oct 11, 2024
@pls148
Copy link
Contributor

pls148 commented Oct 11, 2024

Keeping the remaining work organized here:

  • DCE: pick out the now unused or misused types/functions and types in task-impl helpers (and other places)
  • Rename consensus2->consensus
  • Clean up any other bits relating to dependency-tasks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cappuccino cappuccino-sprint2 consensus-hardening Hardening the consensus protocol and adaquate testing.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants