massa icon indicating copy to clipboard operation
massa copied to clipboard

Keep old testnet final ledger.

Open AurelienFT opened this issue 2 years ago • 5 comments

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

Need to be tested, don't think it's a good thing to keep it for testnet 17.

AurelienFT avatar Nov 23 '22 23:11 AurelienFT

That being said, most of the SCs from 16 won't work on 17

damip avatar Nov 23 '22 23:11 damip

That being said, most of the SCs from 16 won't work on 17

Yes I think we will implement it in testnet 18.

AurelienFT avatar Nov 24 '22 08:11 AurelienFT

note: this has impacts on the deploy scripts for labnet/testnet

damip avatar Nov 24 '22 15:11 damip

note: this has impacts on the deploy scripts for labnet/testnet

yes we need to remove the storage folder.

AurelienFT avatar Nov 24 '22 16:11 AurelienFT

This PR is included in a more massive work about version and I think we should have some brainstorming before doing things like this. Moving it to TEST.19

AurelienFT avatar Dec 27 '22 07:12 AurelienFT

This PR is tightly linked to the https://github.com/massalabs/massa/issues/3611 issue, so I think we should coordinate based on what we want to / can deliver for testnet_21, and what we want in the future.

In PR https://github.com/massalabs/massa/pull/3624, I made the assumption that we only keep the ledger if we want to restart from a snapshot, which is the case for only 1 node. (The other nodes will bootstrap the ledger from it as usual).

If we include the network restart capabilities in testnet_21, I would argue to keep the same approach: only have one node load the ledger with a CLI flag, and the other bootstrap from it. It's more expensive for the servers, but prevents syncing issues between them.

If we don't include the network restart capabilities, I think the current approach is fine for testnet_21. Maybe just add a --remove-ledger flag in sandbox mode to avoid having to delete it manually on every launch. (maybe even have it as the default?)

Leo-Besancon avatar Mar 28 '23 11:03 Leo-Besancon

Sorry @Leo-Besancon I have forgotten to commit a file...

AurelienFT avatar Mar 28 '23 13:03 AurelienFT