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

Network restart followup #3783

Merged
merged 140 commits into from
Jun 1, 2023
Merged

Network restart followup #3783

merged 140 commits into from
Jun 1, 2023

Conversation

Leo-Besancon
Copy link
Collaborator

@Leo-Besancon Leo-Besancon commented Apr 6, 2023

  • document all added functions
  • try in sandbox /simulation/labnet
  • unit tests on the added/changed features
    • make tests compile
    • make tests pass
  • add logs allowing easy debugging in case the changes caused problems
  • if the API has changed, update the API specification

Tested in labnet:

  • Basic network behaviour (send_tx, buy/sell rolls, etc.)
  • Restart the network and keep roll information
  • Bootstrap on initial network and after network restart
  • Send 1500 txps

@Leo-Besancon Leo-Besancon linked an issue Apr 6, 2023 that may be closed by this pull request
@AurelienFT AurelienFT changed the base branch from testnet_21 to testnet_22 April 7, 2023 09:49
@Leo-Besancon Leo-Besancon self-assigned this Apr 7, 2023
Leo-Besancon and others added 23 commits April 12, 2023 10:34
This does not build, I have to add the logic of apply_change to the given ledger_batch
- Created massa-db package for easier access to rocks_db in final_state
- Put the AsyncPool in rocks_db
- Re-designed the speculative async_pool to avoid querying rocks_db too much

/!\ Tests fail! Need to investigate
But now the execution tests are not in a deadlock state.
Iteration per message > Iteration per key
@Leo-Besancon Leo-Besancon marked this pull request as ready for review May 2, 2023 12:25
@Leo-Besancon Leo-Besancon requested a review from sydhds May 30, 2023 07:23
* Add mip_store in final_state

* Add reset_db in MIP store

* Fix bootstrap test - added the case of empty Versioning bootstrap

We have to rethink of how the cursors work, see follow up issue #4000

* Rework shutdown start & shutdown end computation for final state check

* Rework delete_prefix

* Rework extend_from_db for MipStore

---------

Co-authored-by: sydhds <sd@massa.net>
Co-authored-by: Leo-Besancon <leoleo38@live.fr>
@Leo-Besancon Leo-Besancon requested a review from litchipi May 30, 2023 16:11
@Leo-Besancon Leo-Besancon merged commit 2e4efb0 into testnet_23 Jun 1, 2023
@Eitu33
Copy link
Contributor

Eitu33 commented Jun 1, 2023

Congrats 🥳

@AurelienFT
Copy link
Contributor

Yeahhhhh !

@modship
Copy link
Member

modship commented Jun 1, 2023

👏🏻

@sydhds sydhds deleted the network_restart_followup branch June 1, 2023 09:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
6 participants