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

Testnet 17 #3181

Merged
merged 191 commits into from
Nov 30, 2022
Merged

Testnet 17 #3181

merged 191 commits into from
Nov 30, 2022

Conversation

modship and others added 18 commits November 28, 2022 15:01
* Check post-bootstrap hashes in tests

* Add test_cycle_info_hash_computation

* Fix cycle info post-bootstrap hash dif issue

* Move rng_seed len check back to its original place

* Fix async pool bootstrap test issue
* Check post-bootstrap hashes in tests

* Add test_cycle_info_hash_computation

* Fix cycle info post-bootstrap hash dif issue

* Move rng_seed len check back to its original place

* Fix async pool bootstrap test issue

* Improve test_cycle_info_hash_computation

* Fix production stats hash computing
Co-authored-by: Damir Vodenicarevic <damipator@gmail.com>
bors bot added a commit to massalabs/massa-sc-runtime that referenced this pull request Nov 30, 2022
161: Working branch for testnet 17 r=AurelienFT a=gterzian

Part of massalabs/massa#3181

Co-authored-by: gterzian <2792687+gterzian@users.noreply.github.com>
Co-authored-by: sydhds <sydhds@massa.net>
Co-authored-by: AurelienFT <aurelien.foucault@epitech.eu>
Co-authored-by: sydh <sydhds@massa.net>
Co-authored-by: Greg Terzian <2792687+gterzian@users.noreply.github.com>
Co-authored-by: Thomas Plisson <thomas.plisson@epitech.eu>
* Initial code for get_keys SC abi

* Rework get_keys()

* Avoid hardcoding key repr & value size

* Code cleanup

* Add get_keys_for()

* Rework get_keys() for speculative ledger

* Remove get_keys() in ledger_changes.rs

* Return None if entry does not exist

* Share read lock

* Move entry_exists() check inside get_datastore_keys()

Co-authored-by: sydhds <sydhds@massa.net>
@AurelienFT AurelienFT marked this pull request as ready for review November 30, 2022 14:06
@AurelienFT
Copy link
Contributor

bors merge

@bors
Copy link
Contributor

bors bot commented Nov 30, 2022

Build succeeded:

@bors bors bot merged commit 5044f28 into main Nov 30, 2022
@gterzian
Copy link
Contributor Author

gterzian commented Dec 2, 2022

@massalabs/core-team FYI tested the gas price replacement by fee on the labnet, it seems to work fine as:

  1. Fee is added to block reward, split among endorser(right?)
  2. Transaction is rejected if fee is too high(although it's not 100% clear, since one doesn't get feedback other than that the transaction "dissapears", as noted in Add information about operation execution in CLI #3295)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants