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

Update to wasmer 4.2.4 #4581

Merged
merged 1 commit into from
Dec 8, 2023
Merged

Update to wasmer 4.2.4 #4581

merged 1 commit into from
Dec 8, 2023

Conversation

sydhds
Copy link
Contributor

@sydhds sydhds commented Dec 8, 2023

  • document all added functions
  • try in sandbox /simulation/labnet
    • if part of node-launch, checked using the resync_check flag
  • 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

@sydhds sydhds requested review from damip and Leo-Besancon December 8, 2023 10:48
@sydhds sydhds mentioned this pull request Dec 8, 2023
3 tasks
Copy link

codecov bot commented Dec 8, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (a668d77) 71.76% compared to head (addf1b2) 71.76%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #4581   +/-   ##
=======================================
  Coverage   71.76%   71.76%           
=======================================
  Files         291      291           
  Lines       51301    51301           
=======================================
  Hits        36818    36818           
  Misses      14483    14483           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@damip damip merged commit 29f6e6d into main Dec 8, 2023
12 checks passed
@sydhds sydhds deleted the feature/wasmer_4_2_4 branch December 8, 2023 14:30
Leo-Besancon pushed a commit that referenced this pull request Dec 14, 2023
* Update to wasmer 4.2.4 (#4581)

Co-authored-by: sydhds <[email protected]>

* typo fix

* typo fix

* typo fix

* typo fix

* typo fix

---------

Co-authored-by: Sydhds <[email protected]>
Co-authored-by: sydhds <[email protected]>
damip added a commit that referenced this pull request Dec 15, 2023
* Update addresses and public keys

* Add more info to double staking protection's panic message

* fmt

* update version and genesis

* Update version and CI

* Update Cargo.lock

* add community charter

* Add serde json tests (#4579)

* chore: typo fix (#4589)

* Update to wasmer 4.2.4 (#4581)

Co-authored-by: sydhds <[email protected]>

* typo fix

* typo fix

* typo fix

* typo fix

* typo fix

---------

Co-authored-by: Sydhds <[email protected]>
Co-authored-by: sydhds <[email protected]>

* ci: add checksum + tested with ACT (#4590)

* Upgrade-to-latest-toolchain (#4585)

Co-authored-by: Leo-Besancon <[email protected]>

* Update massa-sc-runtime

Signed-off-by: Jean-François <[email protected]>

* Cargo update

* Remove compatibility layer in tracing (#4535)

* Add deferred_credits initial file for test

* Update Genesis and CI

* Push back Genesis

* Same

---------

Signed-off-by: Jean-François <[email protected]>
Co-authored-by: Damir Vodenicarevic <[email protected]>
Co-authored-by: Moncef AOUDIA <[email protected]>
Co-authored-by: Ursulafe <[email protected]>
Co-authored-by: Sydhds <[email protected]>
Co-authored-by: sydhds <[email protected]>
Co-authored-by: Jean-François <[email protected]>
Leo-Besancon pushed a commit that referenced this pull request Dec 15, 2023
Leo-Besancon added a commit that referenced this pull request Dec 15, 2023
* Update addresses and public keys

* Add more info to double staking protection's panic message

* fmt

* update version and genesis

* Update version and CI

* Update Cargo.lock

* add community charter

* Add serde json tests (#4579)

* chore: typo fix (#4589)

* Update to wasmer 4.2.4 (#4581)

Co-authored-by: sydhds <[email protected]>

* typo fix

* typo fix

* typo fix

* typo fix

* typo fix

---------

Co-authored-by: Sydhds <[email protected]>
Co-authored-by: sydhds <[email protected]>

* ci: add checksum + tested with ACT (#4590)

* Upgrade-to-latest-toolchain (#4585)

Co-authored-by: Leo-Besancon <[email protected]>

* Update massa-sc-runtime

Signed-off-by: Jean-François <[email protected]>

* Cargo update

* Remove compatibility layer in tracing (#4535)

* Add deferred_credits initial file for test

* Update Genesis and CI

* Push back Genesis

* Same

---------

Signed-off-by: Jean-François <[email protected]>
Co-authored-by: Damir Vodenicarevic <[email protected]>
Co-authored-by: Moncef AOUDIA <[email protected]>
Co-authored-by: Ursulafe <[email protected]>
Co-authored-by: Sydhds <[email protected]>
Co-authored-by: sydhds <[email protected]>
Co-authored-by: Jean-François <[email protected]>
sydhds added a commit that referenced this pull request Dec 20, 2023
* Update addresses and public keys

* Add more info to double staking protection's panic message

* fmt

* update version and genesis

* Update version and CI

* Update Cargo.lock

* add community charter

* Add serde json tests (#4579)

* chore: typo fix (#4589)

* Update to wasmer 4.2.4 (#4581)

Co-authored-by: sydhds <[email protected]>

* typo fix

* typo fix

* typo fix

* typo fix

* typo fix

---------

Co-authored-by: Sydhds <[email protected]>
Co-authored-by: sydhds <[email protected]>

* ci: add checksum + tested with ACT (#4590)

* Upgrade-to-latest-toolchain (#4585)

Co-authored-by: Leo-Besancon <[email protected]>

* Update massa-sc-runtime

Signed-off-by: Jean-François <[email protected]>

* Cargo update

* Remove compatibility layer in tracing (#4535)

* Add deferred_credits initial file for test

* Update Genesis and CI

* Push back Genesis

* Same

---------

Signed-off-by: Jean-François <[email protected]>
Co-authored-by: Damir Vodenicarevic <[email protected]>
Co-authored-by: Moncef AOUDIA <[email protected]>
Co-authored-by: Ursulafe <[email protected]>
Co-authored-by: Sydhds <[email protected]>
Co-authored-by: sydhds <[email protected]>
Co-authored-by: Jean-François <[email protected]>
sydhds added a commit that referenced this pull request Dec 20, 2023
* Update addresses and public keys

* Add more info to double staking protection's panic message

* fmt

* update version and genesis

* Update version and CI

* Update Cargo.lock

* add community charter

* Add serde json tests (#4579)

* chore: typo fix (#4589)

* Update to wasmer 4.2.4 (#4581)

Co-authored-by: sydhds <[email protected]>

* typo fix

* typo fix

* typo fix

* typo fix

* typo fix

---------

Co-authored-by: Sydhds <[email protected]>
Co-authored-by: sydhds <[email protected]>

* ci: add checksum + tested with ACT (#4590)

* Upgrade-to-latest-toolchain (#4585)

Co-authored-by: Leo-Besancon <[email protected]>

* Update massa-sc-runtime

Signed-off-by: Jean-François <[email protected]>

* Cargo update

* Remove compatibility layer in tracing (#4535)

* Add deferred_credits initial file for test

* Update Genesis and CI

* Push back Genesis

* Same

---------

Signed-off-by: Jean-François <[email protected]>
Co-authored-by: Damir Vodenicarevic <[email protected]>
Co-authored-by: Moncef AOUDIA <[email protected]>
Co-authored-by: Ursulafe <[email protected]>
Co-authored-by: Sydhds <[email protected]>
Co-authored-by: sydhds <[email protected]>
Co-authored-by: Jean-François <[email protected]>
github-merge-queue bot pushed a commit that referenced this pull request Dec 22, 2023
* Add initial chain id impl

* Add chain id to jsonrpc, grpc public & grpc private api

* Add massa-client features sandbox

* Add chain id SC unit test

* Fix unit test + add dummy gas costs

* Update massa-proto-rs dependency

* Update massa-sc-runtime dependency

* Add unit test allowing easy update of massa docs

* Use chain id through config instead of constants directly

* Use chain id everywhere (round 1) (#4594)

* Use chain id everywhere (round 1)

* Use chain id in massa client

---------

Co-authored-by: sydhds <[email protected]>

* Remove confusing chain id

* Fix some unit tests

* Test 27 6 wip (#4582)

* Update addresses and public keys

* Add more info to double staking protection's panic message

* fmt

* update version and genesis

* Update version and CI

* Update Cargo.lock

* add community charter

* Add serde json tests (#4579)

* chore: typo fix (#4589)

* Update to wasmer 4.2.4 (#4581)

Co-authored-by: sydhds <[email protected]>

* typo fix

* typo fix

* typo fix

* typo fix

* typo fix

---------

Co-authored-by: Sydhds <[email protected]>
Co-authored-by: sydhds <[email protected]>

* ci: add checksum + tested with ACT (#4590)

* Upgrade-to-latest-toolchain (#4585)

Co-authored-by: Leo-Besancon <[email protected]>

* Update massa-sc-runtime

Signed-off-by: Jean-François <[email protected]>

* Cargo update

* Remove compatibility layer in tracing (#4535)

* Add deferred_credits initial file for test

* Update Genesis and CI

* Push back Genesis

* Same

---------

Signed-off-by: Jean-François <[email protected]>
Co-authored-by: Damir Vodenicarevic <[email protected]>
Co-authored-by: Moncef AOUDIA <[email protected]>
Co-authored-by: Ursulafe <[email protected]>
Co-authored-by: Sydhds <[email protected]>
Co-authored-by: sydhds <[email protected]>
Co-authored-by: Jean-François <[email protected]>

* Add initial chain id impl

* Use chain id everywhere (round 1) (#4594)

* Use chain id everywhere (round 1)

* Use chain id in massa client

---------

Co-authored-by: sydhds <[email protected]>

* Fix unit tests (final round)

* Minor fix

* Revert some nasty insert from bad rebase

* Auto gen chain id from VERSION

* Cargo clippy && fmt && doc pass

* Update massa-sc-runtime dependency

---------

Signed-off-by: Jean-François <[email protected]>
Co-authored-by: sydhds <[email protected]>
Co-authored-by: Leo-Besancon <[email protected]>
Co-authored-by: Damir Vodenicarevic <[email protected]>
Co-authored-by: Moncef AOUDIA <[email protected]>
Co-authored-by: Ursulafe <[email protected]>
Co-authored-by: Jean-François <[email protected]>
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.

3 participants