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

Bump fuel-vm to 0.35.0 version #1256

Merged
merged 5 commits into from
Jul 14, 2023
Merged

Bump fuel-vm to 0.35.0 version #1256

merged 5 commits into from
Jul 14, 2023

Conversation

xgreenx
Copy link
Collaborator

@xgreenx xgreenx commented Jul 14, 2023

The PR uses the latest fuel-vm 0.35.0. It affects the fuel-core in the next way:

  • Removes support for OpaqueReceipt transaction
  • Adds new opcodes eck1, ecr1, ed19
  • Adds new Receipt::Mint and Receipt::Burn

@xgreenx xgreenx self-assigned this Jul 14, 2023
@xgreenx xgreenx requested a review from a team July 14, 2023 16:54
@@ -20,9 +20,9 @@ pub fn run(c: &mut Criterion) {
let signature = Signature::sign(&secret, &message);

run_group_ref(
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should we also setup benchmarks for ecr1 and ed19?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is an issue for that=) #1254

Co-authored-by: Brandon Kite <brandonkite92@gmail.com>
@@ -76,8 +76,7 @@ async fn test_tx_gossiping() {
.next()
.expect("Producer is initialized with one transaction");

let response = client_one.transaction(&tx_id).await.unwrap();
assert!(response.is_some());
let _ = client_one.await_transaction_commit(&tx_id).await.unwrap();
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is this a workaround for the flakey test?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It should fix it properly =)

@xgreenx xgreenx requested review from Voxelot and a team July 14, 2023 19:46
@xgreenx xgreenx merged commit ed82fb4 into master Jul 14, 2023
22 checks passed
@xgreenx xgreenx deleted the feature/fuel-vm-0.35.0 branch July 14, 2023 21:17
@xgreenx xgreenx mentioned this pull request Jul 16, 2023
xgreenx added a commit that referenced this pull request Jul 17, 2023
## Release v0.20.0

The release brings a couple of new breaking changes from the [`fuel-vm
0.35.0`](https://github.com/FuelLabs/fuel-vm/releases/tag/v0.35.0) with
bugfixes. Check the description of the VM release for more details.

The `fuel-core` release mostly improved the internal codebase but also
brought some breaking changes:
- Removed `Trigger::Hybrid` PoA block trigger mode. Only
`Trigger::Instante` and `Trigger::Interval` are available for block
production now. The main mode for testnets and mainnet will be
`Interval`.
- Removed support for `OpaqueReceipt` and the `Receipt` type doesn't
have the `raw_payload` field anymore.
- A `Receipt` type got two new variants: `Mint` and `Burn`. The
corresponding opcodes emit these new events.
- The `AssetId` is derived from `ContractId` and additional nonce. So
the `ContractId` and `AssetId` can't be the same anymore.

## What's Changed
* bump rocksdb to enable compiling with GCC 13 by @segfault-magnet in
#1219
* setting peer reputation params by @leviathanbeak in
#1202
* Take into account the actually used gas by the transactions and fetch
more transaction by @xgreenx in
#1223
* Use production configuration for `fuel-core` during benches by
@xgreenx in #1227
* Speedup and stabilize unit and integration tests by @xgreenx in
#1231
* test: State benchmarks by @bvrooman in
#1226
* Remove hybrid PoA block trigger mode by @Dentosal in
#1232
* test: Benchmark contract state insertions with DB vs. DB transactions
by @bvrooman in #1230
* multiplatform docker builds by @Voxelot in
#1233
* Fix typo in architecture.md by @eltociear in
#1241
* Expose gas cost in chain info by @MitchTurner in
#1244
* Reuse calculated tx id in executor by @MitchTurner in
#1248
* Fix multi-platform images by @Voxelot in
#1251
* Add logging of the long GraphQL queries for future debug by
@MitchTurner in #1250
* Reused `CheckedTransaction` from transaction pool in the executor by
@xgreenx in #1249
* Bump `fuel-vm` to `0.35.0` version by @xgreenx in
#1256

## New Contributors
* @segfault-magnet made their first contribution in
#1219
* @eltociear made their first contribution in
#1241
* @MitchTurner made their first contribution in
#1244

**Full Changelog**:
v0.19.1...v0.20.0
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.

None yet

3 participants