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

"contract" op code sanity checks #1453

Merged
merged 43 commits into from
Nov 15, 2023
Merged

Conversation

MitchTurner
Copy link
Member

@MitchTurner MitchTurner commented Oct 25, 2023

#1386

block target estimation/contract/bal contract
                        time:   [1.4691 ms 1.4723 ms 1.4758 ms]
block target estimation/contract/bal script
                        time:   [22.914 ms 22.954 ms 22.997 ms]
block target estimation/contract/bhei
                        time:   [2.5669 ms 2.5785 ms 2.5903 ms]
block target estimation/contract/bhsh
                        time:   [3.1837 ms 3.1990 ms 3.2147 ms]
block target estimation/contract/burn
                        time:   [108.90 ms 109.19 ms 109.48 ms]
block target estimation/contract/call 1
                        time:   [71.952 ms 72.085 ms 72.225 ms]
block target estimation/contract/call 10
                        time:   [69.186 ms 69.357 ms 69.526 ms]
block target estimation/contract/call 100
                        time:   [44.121 ms 44.225 ms 44.335 ms]
block target estimation/contract/call 1000
                        time:   [11.294 ms 11.318 ms 11.343 ms]
block target estimation/contract/call 10000
                        time:   [2.8902 ms 2.8962 ms 2.9024 ms]
block target estimation/contract/call 19753
                        time:   [2.4352 ms 2.4473 ms 2.4673 ms]
block target estimation/contract/call 29629
                        time:   [2.1797 ms 2.1898 ms 2.2043 ms]
block target estimation/contract/call 44444
                        time:   [2.1112 ms 2.1173 ms 2.1237 ms]
block target estimation/contract/call 66666
                        time:   [2.0530 ms 2.0610 ms 2.0691 ms]
block target estimation/contract/call 100000
                        time:   [1.9275 ms 1.9322 ms 1.9387 ms]
block target estimation/contract/cb
                        time:   [2.8728 ms 2.8836 ms 2.8946 ms]
block target estimation/contract/ccp 1
                        time:   [3.9833 ms 4.0033 ms 4.0309 ms]
block target estimation/contract/ccp 10
                        time:   [4.2070 ms 4.2235 ms 4.2427 ms]
block target estimation/contract/ccp 100
                        time:   [4.4203 ms 4.4391 ms 4.4574 ms]
block target estimation/contract/ccp 1000
                        time:   [3.7655 ms 3.7747 ms 3.7840 ms]
block target estimation/contract/ccp 10000
                        time:   [4.0406 ms 4.0589 ms 4.0775 ms]
block target estimation/contract/ccp 19753
                        time:   [3.5811 ms 3.5896 ms 3.5981 ms]
block target estimation/contract/ccp 29629
                        time:   [3.4583 ms 3.4839 ms 3.5158 ms]
block target estimation/contract/ccp 44444
                        time:   [3.4880 ms 3.4981 ms 3.5081 ms]
block target estimation/contract/ccp 66666
                        time:   [3.5190 ms 3.5329 ms 3.5514 ms]
block target estimation/contract/ccp 100000
                        time:   [3.4854 ms 3.4964 ms 3.5076 ms]
block target estimation/contract/croo
                        time:   [67.070 ms 67.253 ms 67.441 ms]
block target estimation/contract/csiz 1
                        time:   [3.1927 ms 3.2046 ms 3.2171 ms]
block target estimation/contract/csiz 10
                        time:   [3.3017 ms 3.3142 ms 3.3277 ms]
block target estimation/contract/csiz 100
                        time:   [3.4821 ms 3.4920 ms 3.5020 ms]
block target estimation/contract/csiz 1000
                        time:   [3.0014 ms 3.0087 ms 3.0167 ms]
block target estimation/contract/csiz 10000
                        time:   [3.2728 ms 3.2823 ms 3.2926 ms]
block target estimation/contract/csiz 19753
                        time:   [2.7656 ms 2.7722 ms 2.7794 ms]
block target estimation/contract/csiz 29629
                        time:   [2.6209 ms 2.6371 ms 2.6585 ms]
block target estimation/contract/csiz 44444
                        time:   [2.4669 ms 2.4798 ms 2.4940 ms]
block target estimation/contract/csiz 66666
                        time:   [2.4630 ms 2.4692 ms 2.4758 ms]
block target estimation/contract/csiz 100000
                        time:   [2.4514 ms 2.4616 ms 2.4762 ms]
block target estimation/contract/ldc 1
                        time:   [4.2252 ms 4.2377 ms 4.2506 ms]
block target estimation/contract/ldc 10
                        time:   [4.4222 ms 4.4362 ms 4.4503 ms]
block target estimation/contract/ldc 100
                        time:   [4.4831 ms 4.4936 ms 4.5046 ms]
block target estimation/contract/ldc 1000
                        time:   [4.2607 ms 4.2739 ms 4.2881 ms]
block target estimation/contract/ldc 10000
                        time:   [4.7586 ms 4.7841 ms 4.8157 ms]
block target estimation/contract/ldc 19753
                        time:   [4.6346 ms 4.6492 ms 4.6643 ms]
block target estimation/contract/ldc 29629
                        time:   [4.4694 ms 4.4880 ms 4.5071 ms]
block target estimation/contract/ldc 44444
                        time:   [4.4017 ms 4.4225 ms 4.4500 ms]
block target estimation/contract/ldc 66666
                        time:   [4.3336 ms 4.3425 ms 4.3516 ms]
block target estimation/contract/ldc 100000
                        time:   [4.4381 ms 4.4566 ms 4.4749 ms]
block target estimation/contract/log
                        time:   [10.256 ms 10.303 ms 10.350 ms]
block target estimation/contract/logd 1
                        time:   [4.1894 ms 4.2067 ms 4.2251 ms]
block target estimation/contract/logd 10
                        time:   [4.4084 ms 4.4381 ms 4.4722 ms]
block target estimation/contract/logd 100
                        time:   [5.3502 ms 5.4004 ms 5.4666 ms]
block target estimation/contract/logd 1000
                        time:   [13.050 ms 13.135 ms 13.223 ms]
block target estimation/contract/logd 10000
                        time:   [27.383 ms 27.628 ms 27.887 ms]
block target estimation/contract/logd 19753
                        time:   [29.407 ms 29.729 ms 30.055 ms]
block target estimation/contract/logd 29629
                        time:   [33.715 ms 44.027 ms 57.000 ms]
block target estimation/contract/logd 44444
                        time:   [33.105 ms 42.521 ms 54.599 ms]
block target estimation/contract/logd 66666
                        time:   [30.631 ms 30.949 ms 31.279 ms]
block target estimation/contract/logd 100000
                        time:   [30.155 ms 40.772 ms 54.713 ms]
block target estimation/contract/mint
                        time:   [98.285 ms 99.414 ms 100.69 ms]
block target estimation/contract/ret contract
                        time:   [73.189 ms 73.373 ms 73.580 ms]
block target estimation/contract/retd contract 1
                        time:   [65.356 ms 65.477 ms 65.600 ms]
block target estimation/contract/retd contract 10
                        time:   [66.407 ms 66.503 ms 66.604 ms]
block target estimation/contract/retd contract 100
                        time:   [66.665 ms 66.843 ms 67.024 ms]
block target estimation/contract/retd contract 1000
                        time:   [61.910 ms 62.100 ms 62.305 ms]
block target estimation/contract/retd contract 10000
                        time:   [49.114 ms 49.293 ms 49.474 ms]
block target estimation/contract/retd contract 19753
                        time:   [41.567 ms 41.799 ms 42.039 ms]
block target estimation/contract/retd contract 29629
                        time:   [40.203 ms 45.726 ms 56.567 ms]
block target estimation/contract/retd contract 44444
                        time:   [43.955 ms 58.338 ms 75.298 ms]
block target estimation/contract/retd contract 66666
                        time:   [36.076 ms 36.297 ms 36.520 ms]
block target estimation/contract/retd contract 100000
                        time:   [33.281 ms 33.629 ms 34.039 ms]
block target estimation/contract/smo 1
                        time:   [65.119 ms 65.588 ms 66.177 ms]
block target estimation/contract/smo 10
                        time:   [63.433 ms 63.720 ms 64.141 ms]
block target estimation/contract/smo 100
                        time:   [61.177 ms 61.465 ms 61.839 ms]
block target estimation/contract/smo 1000
                        time:   [59.890 ms 60.007 ms 60.134 ms]
block target estimation/contract/smo 10000
                        time:   [51.135 ms 51.444 ms 51.793 ms]
block target estimation/contract/smo 19753
                        time:   [45.648 ms 45.967 ms 46.336 ms]
block target estimation/contract/smo 29629
                        time:   [55.496 ms 69.138 ms 84.636 ms]
block target estimation/contract/smo 44444
                        time:   [50.462 ms 65.771 ms 83.605 ms]
block target estimation/contract/smo 66666
                        time:   [37.880 ms 38.247 ms 38.722 ms]
block target estimation/contract/smo 100000
                        time:   [33.433 ms 33.628 ms 33.823 ms]
block target estimation/contract/srw
                        time:   [73.681 ms 73.943 ms 74.347 ms]
block target estimation/contract/sww
                        time:   [111.88 ms 112.64 ms 113.41 ms]
block target estimation/contract/time
                        time:   [64.179 ms 64.539 ms 64.899 ms]
block target estimation/contract/tr
                        time:   [128.13 ms 128.49 ms 128.94 ms]

@MitchTurner MitchTurner self-assigned this Oct 30, 2023
@MitchTurner
Copy link
Member Author

MitchTurner commented Nov 2, 2023

I've left the sequential op codes unbenched because the one I did wasn't priced correctly.

Other than that, all that is left is smo and tro. Both are partially written but require some additional setup to pass.

@MitchTurner MitchTurner marked this pull request as ready for review November 3, 2023 18:00
@MitchTurner MitchTurner requested a review from a team November 8, 2023 08:06
bvrooman
bvrooman previously approved these changes Nov 10, 2023
Comment on lines 196 to 203
let state_size = std::env::var_os("STATE_SIZE")
.map(|value| {
let value = value.to_str().unwrap();
let value = value.parse::<u64>().unwrap();
println!("Overriding state size with {}", value);
value
})
.unwrap_or(STATE_SIZE);
Copy link
Contributor

Choose a reason for hiding this comment

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

[nit] Accepting state_size as a parameter to this function, rather than determining it here, would be a better separation of concerns. You could calculate state_size in this way at the call site and pass it in.

Comment on lines +161 to +164
let rt = tokio::runtime::Builder::new_current_thread()
.enable_all()
.build()
.unwrap();
Copy link
Contributor

Choose a reason for hiding this comment

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

Note: Mitch and I talked about this pattern, where the service and runtime are coupled together. It is not ideal and we can investigate ways to clean this up, either as part of this PR, or in a follow up issue.

benches/benches/block_target_gas_set/contract.rs Outdated Show resolved Hide resolved
// VmBench::contract(rng, op::ret(RegId::ONE)).unwrap(),
// );

// TODO: Is `rvrt` even possible to bench?
Copy link
Contributor

Choose a reason for hiding this comment

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

Perhaps we can create a separate ticket to investigate benchmarks for rvrt. Depending on the results of that investigation, we can continue from there.

Copy link
Member Author

Choose a reason for hiding this comment

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

Yeah. By its nature it is going to revert everything you've done. It's not important to bench in that case, since there is no way to spam it. We can just cost it something reasonable.

Copy link
Collaborator

@xgreenx xgreenx left a comment

Choose a reason for hiding this comment

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

Maybe it would be nice to use a builder pattern, but it is up to you=)

let mut instructions = setup_instructions();
instructions.extend(vec![op::call(0x10, RegId::ZERO, 0x11, 0x12)]);

replace_contract_in_service(&mut service, &contract_id, contract_instructions);
Copy link
Collaborator

Choose a reason for hiding this comment

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

You can move this step to run_with_service function and pass only the contract set of instructions


// burn
{
let contract = vec![op::burn(RegId::ONE, RegId::HP), op::jmpb(RegId::ZERO, 0)];
Copy link
Collaborator

Choose a reason for hiding this comment

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

Do we have enough balance to perform all burns?

Copy link
Member Author

Choose a reason for hiding this comment

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

The benchmark is completing without an error. Is that what you mean?

{
let contract = vec![op::burn(RegId::ONE, RegId::HP), op::jmpb(RegId::ZERO, 0)];
let mut instructions = setup_instructions();
// TODO: I don't know why we need these extra ops
Copy link
Collaborator

Choose a reason for hiding this comment

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

We do it because we want to have a zeroed sub id for the asset. You allocated 32 zero bytes and after you started to use it as a sub id

{
let contract = vec![
op::gtf_args(0x16, 0x00, GTFArgs::ScriptData),
op::movi(0x15, 2000),
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why do we need to allocate 2000? It seems 32 should be enough


// mint
{
let contract = vec![op::mint(RegId::ONE, RegId::ZERO), op::ret(RegId::ZERO)];
Copy link
Collaborator

Choose a reason for hiding this comment

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

Do we want to return here? I thought we wanted to jump back=)

op::addi(0x16, 0x15, 32), // data ppinter
op::movi(0x17, i), // data length
op::smo(0x15, 0x16, 0x17, 0x18),
op::ret(RegId::ZERO),
Copy link
Collaborator

Choose a reason for hiding this comment

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

You can pass a lot of the assets during the first call and jump backward here. Also, you can create a message with 1 coins instead of 10

// sww

{
let contract = vec![op::sww(RegId::ZERO, 0x29, RegId::ONE), op::ret(RegId::ZERO)];
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why do you want to return here? backward jump looks good=)

Copy link
Collaborator

Choose a reason for hiding this comment

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

Also, it would be nice to write different values into the storage to generate different hash. You can increment some value by 1 each iteration

// tr

{
let contract = vec![op::tr(0x15, 0x14, 0x15), op::ret(RegId::ZERO)];
Copy link
Collaborator

Choose a reason for hiding this comment

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

You can send 1 token with tr opcode. In this case you can pass huge value like 100000 into the call instruction and use backward jump instead.


// tro

// The `tro` benchmark is disabled because it would require many, many outputs, because each
Copy link
Collaborator

Choose a reason for hiding this comment

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

Actually, we can still run this benchmark that will fail later because of the 255 limit to get at least some results. You can customize the run function to allow other error

Copy link
Member Author

@MitchTurner MitchTurner Nov 14, 2023

Choose a reason for hiding this comment

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

I've addressed all the concerns except this one. My tro code is failing with the error InvalidInstruction.

This implies to me that I'm jumping to invalid memory or something.

Copy link
Collaborator

Choose a reason for hiding this comment

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

We can leave it for later=)

Copy link
Collaborator

@xgreenx xgreenx left a comment

Choose a reason for hiding this comment

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

=)

Copy link
Collaborator

@xgreenx xgreenx left a comment

Choose a reason for hiding this comment

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

Thank you for huge work! I like the builder for contract opcodes, very easy to read benchmarks right now!=)


// tro

// The `tro` benchmark is disabled because it would require many, many outputs, because each
Copy link
Collaborator

Choose a reason for hiding this comment

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

We can leave it for later=)

@MitchTurner MitchTurner enabled auto-merge (squash) November 15, 2023 22:21
@MitchTurner MitchTurner merged commit 053c260 into master Nov 15, 2023
23 checks passed
@MitchTurner MitchTurner deleted the contract-op-code-sanity-check branch November 15, 2023 22:29
bvrooman pushed a commit that referenced this pull request Nov 24, 2023
## Version 0.21.0

This release focuses on preparing `fuel-core` for the mainnet
environment.
- Most of the changes improved the security and stability of the node.
- The gas model was reworked to cover all aspects of execution.
- The benchmarking system was significantly enhanced, covering worst
scenarios.
- A new set of benchmarks was added to track the accuracy of gas prices.
- Optimized heavy operations and removed/replaced exploitable
functionality.

Besides that, there are more concrete changes:
- Unified naming conventions for all CLI arguments. Added dependencies
between related fields to avoid misconfiguration in case of missing
arguments. Added `--debug` flag that enables additional functionality
like a debugger.
- Improved telemetry to cover the internal work of services and added
support for the Pyroscope, allowing it to generate real-time flamegraphs
to track performance.
- Improved stability of the P2P layer and adjusted the updating of
reputation. The speed of block synchronization was significantly
increased.
- The node is more stable and resilient. Improved DoS resistance and
resource management. Fixed critical bugs during state transition.
- Reworked the `Mint` transaction to accumulate the fee from block
production inside the contract defined by the block producer.

FuelVM received a lot of safety and stability improvements:
- The audit helped identify some bugs and errors that have been
successfully fixed.
- Updated the gas price model to charge for resources used during the
transaction lifecycle.
- Added `no_std` and 32 bit system support. This opens doors for fraud
proving in the future.
- Removed the `ChainId` from the `PredicateId` calculation, allowing the
use of predicates cross-chain.
- Improvements in the performance of some storage-related opcodes.
- Support the `ECAL` instruction that allows adding custom functionality
to the VM. It can be used to create unique rollups or advanced indexers
in the future.
- Support of [transaction
policies](https://github.com/FuelLabs/fuel-vm/blob/master/CHANGELOG.md#version-0420)
provides additional safety for the user.
It also allows the implementation of a multi-dimensional price model in
the future, making the transaction execution cheaper and allowing more
transactions that don't affect storage.
- Refactored errors, returning more detailed errors to the user,
simplifying debugging.

### Added

- [#1503](#1503): Add `gtf`
opcode sanity check.
- [#1502](#1502): Added price
benchmark for `vm_initialization`.
- [#1501](#1501): Add a CLI
command for generating a fee collection contract.
- [#1492](#1492): Support
backward iteration in the RocksDB. It allows backward queries that were
not allowed before.
- [#1490](#1490): Add push and
pop benchmarks.
- [#1485](#1485): Prepare rc
release of fuel core v0.21
- [#1476](#1453): Add the
majority of the "other" benchmarks for contract opcodes.
- [#1473](#1473): Expose
fuel-core version as a constant
- [#1469](#1469): Added
support of bloom filter for RocksDB tables and increased the block
cache.
- [#1465](#1465): Improvements
for keygen cli and crates
- [#1642](#1462): Added
benchmark to measure the performance of contract state and contract ID
calculation; use for gas costing.
- [#1457](#1457): Fixing
incorrect measurement for fast(µs) opcodes.
- [#1456](#1456): Added
flushing of the RocksDB during a graceful shutdown.
- [#1456](#1456): Added more
logs to track the service lifecycle.
- [#1453](#1453): Add the
majority of the "sanity" benchmarks for contract opcodes.
- [#1452](#1452): Added
benchmark to measure the performance of contract root calculation when
utilizing the maximum contract size; used for gas costing of contract
root during predicate owner validation.
- [#1449](#1449): Fix coin
pagination in e2e test client.
- [#1447](#1447): Add timeout
for continuous e2e tests
- [#1444](#1444): Add "sanity"
benchmarks for memory opcodes.
- [#1437](#1437): Add some
transaction throughput tests for basic transfers.
- [#1436](#1436): Add a github
action to continuously test beta-4.
- [#1433](#1433): Add "sanity"
benchmarks for flow opcodes.
- [#1432](#1432): Add a new
`--api-request-timeout` argument to control TTL for GraphQL requests.
- [#1430](#1430): Add "sanity"
benchmarks for crypto opcodes.
- [#1426](#1426) Split keygen
into a create and a binary.
- [#1419](#1419): Add
additional "sanity" benchmarks for arithmetic op code instructions.
- [#1411](#1411): Added WASM
and `no_std` compatibility.
- [#1405](#1405): Use correct
names for service metrics.
- [#1400](#1400): Add releasy
beta to fuel-core so that new commits to fuel-core master triggers
fuels-rs.
- [#1371](#1371): Add new
client function for querying the `MessageStatus` for a specific message
(by `Nonce`).
- [#1356](#1356): Add peer
reputation reporting to heartbeat code.
- [#1355](#1355): Added new
metrics related to block importing, such as tps, sync delays etc.
- [#1339](#1339): Adds
`baseAssetId` to `FeeParameters` in the GraphQL API.
- [#1331](#1331): Add peer
reputation reporting to block import code.
- [#1324](#1324): Added
pyroscope profiling to fuel-core, intended to be used by a secondary
docker image that has debug symbols enabled.
- [#1309](#1309): Add
documentation for running debug builds with CLion and Visual Studio
Code.
- [#1308](#1308): Add support
for loading .env files when compiling with the `env` feature. This
allows users to conveniently supply CLI arguments in a secure and
IDE-agnostic way.
- [#1304](#1304): Implemented
`submit_and_await_commit_with_receipts` method for `FuelClient`.
- [#1286](#1286): Include
readable names for test cases where missing.
- [#1274](#1274): Added tests
to benchmark block synchronization.
- [#1263](#1263): Add gas
benchmarks for `ED19` and `ECR1` instructions.

### Changed

- [#1512](#1512): Internally
simplify merkle_contract_state_range.
- [#1507](#1507): Updated
chain configuration to be ready for beta 5 network. It includes opcode
prices from the latest benchmark and contract for the block producer.
- [#1477](#1477): Upgraded the
Rust version used in CI and containers to 1.73.0. Also includes
associated Clippy changes.
- [#1469](#1469): Replaced
usage of `MemoryTransactionView` by `Checkpoint` database in the
benchmarks.
- [#1468](#1468): Bumped
version of the `fuel-vm` to `v0.40.0`. It brings some breaking changes
into consensus parameters API because of changes in the underlying
types.
- [#1466](#1466): Handling
overflows during arithmetic operations.
- [#1460](#1460): Change
tracking branch from main to master for releasy tests.
- [#1454](#1454): Update gas
benchmarks for opcodes that append receipts.
- [#1440](#1440): Don't report
reserved nodes that send invalid transactions.
- [#1439](#1439): Reduced
memory BMT consumption during creation of the header.
- [#1434](#1434): Continue
gossiping transactions to reserved peers regardless of gossiping
reputation score.
- [#1408](#1408): Update gas
benchmarks for storage opcodes to use a pre-populated database to get
more accurate worst-case costs.
- [#1399](#1399): The Relayer
now queries Ethereum for its latest finalized block instead of using a
configurable "finalization period" to presume finality.
- [#1397](#1397): Improved
keygen. Created a crate to be included from forc plugins and upgraded
internal library to drop requirement of protoc to build
- [#1395](#1395): Add
DependentCost benchmarks for `k256`, `s256` and `mcpi` instructions.
- [#1393](#1393): Increase
heartbeat timeout from `2` to `60` seconds, as suggested in [this
issue](#1330).
- [#1392](#1392): Fixed an
overflow in `message_proof`.
- [#1390](#1390): Up the
`ethers` version to `2` to fix an issue with `tungstenite`.
- [#1383](#1383): Disallow
usage of `log` crate internally in favor of `tracing` crate.
- [#1380](#1380): Add
preliminary, hard-coded config values for heartbeat peer reputation,
removing `todo`.
- [#1377](#1377): Remove
`DiscoveryEvent` and use `KademliaEvent` directly in
`DiscoveryBehavior`.
- [#1366](#1366): Improve
caching during docker builds in CI by replacing gha
- [#1358](#1358): Upgraded the
Rust version used in CI to 1.72.0. Also includes associated Clippy
changes.
- [#1349](#1349): Updated
peer-to-peer transactions API to support multiple blocks in a single
request, and updated block synchronization to request multiple blocks
based on the configured range of headers.
- [#1342](#1342): Add error
handling for P2P requests to return `None` to requester and log error.
- [#1318](#1318): Modified
block synchronization to use asynchronous task execution when retrieving
block headers.
- [#1314](#1314): Removed
`types::ConsensusParameters` in favour of `fuel_tx:ConsensusParameters`.
- [#1302](#1302): Removed the
usage of flake and building of the bridge contract ABI.
It simplifies the maintenance and updating of the events, requiring only
putting the event definition into the codebase of the relayer.
- [#1293](#1293):
Parallelized the `estimate_predicates` endpoint to utilize all available
threads.
- [#1270](#1270): Modify the
way block headers are retrieved from peers to be done in batches.

#### Breaking
- [#1506](#1506): Added
validation of the coin's fields during block production and validation.
Before, it was possible to submit a transaction that didn't match the
coin's values in the database, allowing printing/using unavailable
assets.
- [#1491](#1491): Removed
unused request and response variants from the Gossipsub implementation,
as well as related definitions and tests. Specifically, this removes
gossiping of `ConsensusVote` and `NewBlock` events.
- [#1472](#1472): Upgraded
`fuel-vm` to `v0.42.0`. It introduces transaction policies that changes
layout of the transaction. FOr more information check the
[v0.42.0](FuelLabs/fuel-vm#635) release.
- [#1470](#1470): Divide
`DependentCost` into "light" and "heavy" operations.
- [#1464](#1464): Avoid
possible truncation of higher bits. It may invalidate the code that
truncated higher bits causing different behavior on 32-bit vs. 64-bit
systems. The change affects some endpoints that now require lesser
integers.
- [#1432](#1432): All
subscriptions and requests have a TTL now. So each subscription
lifecycle is limited in time. If the subscription is closed because of
TTL, it means that you subscribed after your transaction had been
dropped by the network.
- [#1407](#1407): The
recipient is a `ContractId` instead of `Address`. The block producer
should deploy its contract to receive the transaction fee. The collected
fee is zero until the recipient contract is set.
- [#1407](#1407): The `Mint`
transaction is reworked with new fields to support the account-base
model. It affects serialization and deserialization of the transaction
and also affects GraphQL schema.
- [#1407](#1407): The `Mint`
transaction is the last transaction in the block instead of the first.
- [#1374](#1374): Renamed
`base_chain_height` to `da_height` and return current relayer height
instead of latest Fuel block height.
- [#1367](#1367): Update to
the latest version of fuel-vm.
- [#1363](#1363): Change
message_proof api to take `nonce` instead of `message_id`
- [#1355](#1355): Removed the
`metrics` feature flag from the fuel-core crate, and metrics are now
included by default.
- [#1339](#1339): Added a new
required field called `base_asset_id` to the `FeeParameters` definition
in `ConsensusParameters`, as well as default values for `base_asset_id`
in the `beta` and `dev` chain specifications.
- [#1322](#1322):
The `debug` flag is added to the CLI. The flag should be used for local
development only. Enabling debug mode:
      - Allows GraphQL Endpoints to arbitrarily advance blocks.
      - Enables debugger GraphQL Endpoints.
      - Allows setting `utxo_validation` to `false`.
- [#1318](#1318): Removed the
`--sync-max-header-batch-requests` CLI argument, and renamed
`--sync-max-get-txns` to `--sync-block-stream-buffer-size` to better
represent the current behavior in the import.
- [#1290](#1290): Standardize
CLI args to use `-` instead of `_`.
- [#1279](#1279): Added a new
CLI flag to enable the Relayer service `--enable-relayer`, and disabled
the Relayer service by default. When supplying the `--enable-relayer`
flag, the `--relayer` argument becomes mandatory, and omitting it is an
error. Similarly, providing a `--relayer` argument without the
`--enable-relayer` flag is an error. Lastly, providing the `--keypair`
or `--network` arguments will also produce an error if the
`--enable-p2p` flag is not set.
- [#1262](#1262): The
`ConsensusParameters` aggregates all configuration data related to the
consensus. It contains many fields that are segregated by the usage. The
API of some functions was affected to use lesser types instead the whole
`ConsensusParameters`. It is a huge breaking change requiring
repetitively monotonically updating all places that use the
`ConsensusParameters`. But during updating, consider that maybe you can
use lesser types. Usage of them may simplify signatures of methods and
make them more user-friendly and transparent.

### Removed

#### Breaking
- [#1484](#1484): Removed
`--network` CLI argument. Now the name of the network is fetched form
chain configuration.
- [#1399](#1399): Removed
`relayer-da-finalization` parameter from the relayer CLI.
- [#1338](#1338): Updated
GraphQL client to use `DependentCost` for `k256`, `mcpi`, `s256`,
`scwq`, `swwq` opcodes.
- [#1322](#1322): The
`manual_blocks_enabled` flag is removed from the CLI. The analog is a
`debug` flag.

## What's Changed
* Added changelog in the same way as we did for `fuel-vm` by @xgreenx in
#1287
* Decompose consensus params struct by @MitchTurner in
#1262
* Replace all instances of `_` in CLI `long` arguments with `-` by
@MitchTurner in #1290
* [rpc] Parallelise estimate_predicates endpoint by @YusongWang in
#1293
* Add readable test names to test cases by @MitchTurner in
#1286
* ED19 and ECR1 benchmarks by @Dentosal in
#1263
* feat: Add `enable-relayer` flag to `fuel-core` CLI by @bvrooman in
#1279
* Small nits found during deploying and benchmarking by @xgreenx in
#1303
* Implemented `submit_and_await_commit_with_receipts` method by @xgreenx
in #1304
* Removed the usage of flake and building of the bridge contract ABI by
@xgreenx in #1302
* Download range of headers during syncing by @MitchTurner in
#1270
* feat: Enable .env file loading using `env` feature by @bvrooman in
#1308
* chore: Remove types::ConsensusParameters by @bvrooman in
#1314
* docs: Debugging instructions using IDEs by @bvrooman in
#1309
* test: Block Import benchmarks and test helpers by @bvrooman in
#1274
* Fix audit CI for `webpki` by @xgreenx in
#1319
* chore: Single buffer block sync by @bvrooman in
#1318
* Run unit tests without default features only for specific crates by
@xgreenx in #1321
* Added support for `debug` CLI flag. by @xgreenx in
#1322
* Report Peers that give bad Block Info by @MitchTurner in
#1331
* Pyroscope profiling by @Voxelot in
#1324
* Add error handling for P2P requests to return `None` to requester and
log error by @MitchTurner in
#1342
* Importer Metrics by @Voxelot in
#1355
* chore: Upgrade to Rust 1.72.0 in CI by @bvrooman in
#1358
* chore: Upgrade `fuel-vm` to 0.37.0 by @bvrooman in
#1338
* docs: remove stale schema by @calldelegation in
#1362
* Replace message_id with nonce in `MessageProof` query by @MitchTurner
in #1363
* feat: Configurable base asset by @bvrooman in
#1339
* avoid gha caching for docker builds by @Voxelot in
#1366
* Add tracking for peer heartbeats by @MitchTurner in
#1356
* docs: fix broken link in README by @PaulRBerg in
#1357
* Use right level of error for `.env` file and duplicated service by
@xgreenx in #1372
* Add query and handling for `MessageStatus` by @MitchTurner in
#1371
* Renamed `base_chain_height` to `da_height` and return current relayer
height instead of latest Fuel block height by @xgreenx in
#1374
* Remove `DiscoveryEvent` and use `KademliaEvent` instead by
@MitchTurner in #1377
* Remove todo from code and provide some preliminary peer review config
values by @MitchTurner in
#1380
* Clippy lint to disallow logging crate macros in favor of tracing by
@Dentosal in #1383
* fuel-vm nostd update by @Dentosal in
#1367
* Up the `ethers` version to `2` to fix an issue with `tungstenite` by
@xgreenx in #1390
* Increase heartbeat timeout to one minute by @Dentosal in
#1393
* feat: batch transactions import by @bvrooman in
#1349
* Fix an interger overflow in message_proof when looking beyond genesis
by @Dentosal in #1392
* Improve keygen by @cr-fuel in
#1397
* Benchmark {k256, s256, mcpi} opcodes using DependentCost by @Dentosal
in #1395
* Use correct names for service metrics by @xgreenx in
#1405
* feat: Retrieve finalized blocks from Ethereum by @bvrooman in
#1399
* Updated CI to run `cargo update` weekly by @xgreenx in
#1415
* CI `cargo update` weekly by @xgreenx in
#1416
* Weekly `cargo update` by @github-actions in
#1417
* Fix typos by @GoodDaisy in
#1424
* feat: Support no_std and WASM compilation for `fuel-core` crates by
@bvrooman in #1411
* Create sanity benchmark checks for all the arithmetic op codes by
@MitchTurner in #1419
* Update benchmarking and collecting for `scwq`, `swwq` and `srwq`
opcodes by @xgreenx in #1427
* Account-base fee collection by @xgreenx in
#1407
* Rollback modification of the chain specification by @xgreenx in
#1429
* Tx subscription cleanup (#1422) by @xgreenx in
#1432
* Mark reserved peers as explicit for gossipsub to avoid reputation
decreasing (#1423) by @xgreenx in
#1434
* Weekly `cargo update` by @github-actions in
#1438
* ci: add beta releasy to fuel-core by @kayagokalp in
#1400
* setup a cron job to test beta-4 continuously by @Voxelot in
#1436
* chore: Use MerkleRootCalculator when only BMT root is needed by
@bvrooman in #1439
* Crypto op code sanity checks by @MitchTurner in
#1430
* Update gas benchmarks for some storage opcodes by @Dentosal in
#1408
* Split keygen into a create and a binary by @cr-fuel in
#1426
* Basic Transfer TPS Benchmarking by @Voxelot in
#1437
* "flow" op code sanity benchmarks by @MitchTurner in
#1433
* Applying `#1435` to `master` by @xgreenx in
#1440
* "memory" opcode sanity checks by @MitchTurner in
#1444
* Add timeout to E2E test by @Voxelot in
#1447
* fix owns_coins to properly paginate by @Voxelot in
#1449
* Fixing incorrect measurement for fast(µs) opcodes by @xgreenx in
#1457
* Call `flush` during end of the `FuelService` by @xgreenx in
#1456
* chore: fix releasy test tracking branch from `main` to `master` by
@kayagokalp in #1460
* Weekly `cargo update` by @github-actions in
#1461
* `DB::repair` breaks the table for checkpoints by @xgreenx in
#1463
* Avoid possible truncation of higher bits by @xgreenx in
#1464
* Bumping `fuel-vm` to `v0.40.0` by @xgreenx in
#1468
* Improvements for keygen cli and crates by @cr-fuel in
#1465
* Handling overflows during arithmetic operations by @xgreenx in
#1466
* test: Add contract code root benchmark by @bvrooman in
#1452
* test: Add state root and contract id benchmarks by @bvrooman in
#1462
* Avoid storage caching in benchmarks by @xgreenx in
#1469
* Weekly `cargo update` by @github-actions in
#1471
* Expose fuel-core version as a constant by @Br1ght0ne in
#1473
* Upgrade ed25519_dalek 1.0.1 -> 2.0.0 by @Dentosal in
#1475
* chore: Upgrade Rust 1.73 by @bvrooman in
#1477
* Decrease number of iterations for sequential benchmarks by @xgreenx in
#1480
* Add mldv bench by @Dentosal in
#1481
* feat: Separate light and heavy gas costs (core) by @bvrooman in
#1483
* chore: Remove `network` parameter from CLI by @bvrooman in
#1484
* Transaction policies support by @xgreenx in
#1472
* stage release candidate for beta-5 by @Voxelot in
#1485
* setup docker auth for binary publishing by @Voxelot in
#1488
* Fix macos binary publishing by @xgreenx in
#1489
* Add documentation for our calculation of a _reasonable_ gas fee for
storage by @MitchTurner in
#1478
* "contract" op code sanity checks by @MitchTurner in
#1453
* "other" op code sanity check by @MitchTurner in
#1476
* Add push and pop benches by @MitchTurner in
#1490
* chore: Remove unused gossipsub items by @bvrooman in
#1491
* Adjusting of the prices benchmarks after sanity check benchmarks by
@xgreenx in #1494
* Weekly `cargo update` by @github-actions in
#1498
* Update benches that are affected by receipts by @Dentosal in
#1454
* Support backward iteration in the RocksDB by @xgreenx in
#1492
* Added validation of the coin's fields during block production and
validation by @xgreenx in
#1506
* Added price benchmark for `vm_initialization` by @xgreenx in
#1502
* Sanity-check `gtf` opcode by @MitchTurner in
#1503
* Tool for generating fee collection contract by @Voxelot in
#1501
* Chain config for beta 5 network by @xgreenx in
#1507
* Remove usage of iter_all_filtered from merkle_contract_state_range by
@Dentosal in #1512

## New Contributors
* @YusongWang made their first contribution in
#1293
* @calldelegation made their first contribution in
#1362
* @PaulRBerg made their first contribution in
#1357
* @cr-fuel made their first contribution in
#1397
* @github-actions made their first contribution in
#1417
* @GoodDaisy made their first contribution in
#1424

**Full Changelog**:
v0.20.8...v0.21.0

---------

Co-authored-by: Brandon Vrooman <[email protected]>
crypto523 added a commit to crypto523/fuel-core that referenced this pull request Oct 7, 2024
## Version 0.21.0

This release focuses on preparing `fuel-core` for the mainnet
environment.
- Most of the changes improved the security and stability of the node.
- The gas model was reworked to cover all aspects of execution.
- The benchmarking system was significantly enhanced, covering worst
scenarios.
- A new set of benchmarks was added to track the accuracy of gas prices.
- Optimized heavy operations and removed/replaced exploitable
functionality.

Besides that, there are more concrete changes:
- Unified naming conventions for all CLI arguments. Added dependencies
between related fields to avoid misconfiguration in case of missing
arguments. Added `--debug` flag that enables additional functionality
like a debugger.
- Improved telemetry to cover the internal work of services and added
support for the Pyroscope, allowing it to generate real-time flamegraphs
to track performance.
- Improved stability of the P2P layer and adjusted the updating of
reputation. The speed of block synchronization was significantly
increased.
- The node is more stable and resilient. Improved DoS resistance and
resource management. Fixed critical bugs during state transition.
- Reworked the `Mint` transaction to accumulate the fee from block
production inside the contract defined by the block producer.

FuelVM received a lot of safety and stability improvements:
- The audit helped identify some bugs and errors that have been
successfully fixed.
- Updated the gas price model to charge for resources used during the
transaction lifecycle.
- Added `no_std` and 32 bit system support. This opens doors for fraud
proving in the future.
- Removed the `ChainId` from the `PredicateId` calculation, allowing the
use of predicates cross-chain.
- Improvements in the performance of some storage-related opcodes.
- Support the `ECAL` instruction that allows adding custom functionality
to the VM. It can be used to create unique rollups or advanced indexers
in the future.
- Support of [transaction
policies](https://github.com/FuelLabs/fuel-vm/blob/master/CHANGELOG.md#version-0420)
provides additional safety for the user.
It also allows the implementation of a multi-dimensional price model in
the future, making the transaction execution cheaper and allowing more
transactions that don't affect storage.
- Refactored errors, returning more detailed errors to the user,
simplifying debugging.

### Added

- [#1503](FuelLabs/fuel-core#1503): Add `gtf`
opcode sanity check.
- [#1502](FuelLabs/fuel-core#1502): Added price
benchmark for `vm_initialization`.
- [#1501](FuelLabs/fuel-core#1501): Add a CLI
command for generating a fee collection contract.
- [#1492](FuelLabs/fuel-core#1492): Support
backward iteration in the RocksDB. It allows backward queries that were
not allowed before.
- [#1490](FuelLabs/fuel-core#1490): Add push and
pop benchmarks.
- [#1485](FuelLabs/fuel-core#1485): Prepare rc
release of fuel core v0.21
- [#1476](FuelLabs/fuel-core#1453): Add the
majority of the "other" benchmarks for contract opcodes.
- [#1473](FuelLabs/fuel-core#1473): Expose
fuel-core version as a constant
- [#1469](FuelLabs/fuel-core#1469): Added
support of bloom filter for RocksDB tables and increased the block
cache.
- [#1465](FuelLabs/fuel-core#1465): Improvements
for keygen cli and crates
- [#1642](FuelLabs/fuel-core#1462): Added
benchmark to measure the performance of contract state and contract ID
calculation; use for gas costing.
- [#1457](FuelLabs/fuel-core#1457): Fixing
incorrect measurement for fast(µs) opcodes.
- [#1456](FuelLabs/fuel-core#1456): Added
flushing of the RocksDB during a graceful shutdown.
- [#1456](FuelLabs/fuel-core#1456): Added more
logs to track the service lifecycle.
- [#1453](FuelLabs/fuel-core#1453): Add the
majority of the "sanity" benchmarks for contract opcodes.
- [#1452](FuelLabs/fuel-core#1452): Added
benchmark to measure the performance of contract root calculation when
utilizing the maximum contract size; used for gas costing of contract
root during predicate owner validation.
- [#1449](FuelLabs/fuel-core#1449): Fix coin
pagination in e2e test client.
- [#1447](FuelLabs/fuel-core#1447): Add timeout
for continuous e2e tests
- [#1444](FuelLabs/fuel-core#1444): Add "sanity"
benchmarks for memory opcodes.
- [#1437](FuelLabs/fuel-core#1437): Add some
transaction throughput tests for basic transfers.
- [#1436](FuelLabs/fuel-core#1436): Add a github
action to continuously test beta-4.
- [#1433](FuelLabs/fuel-core#1433): Add "sanity"
benchmarks for flow opcodes.
- [#1432](FuelLabs/fuel-core#1432): Add a new
`--api-request-timeout` argument to control TTL for GraphQL requests.
- [#1430](FuelLabs/fuel-core#1430): Add "sanity"
benchmarks for crypto opcodes.
- [#1426](FuelLabs/fuel-core#1426) Split keygen
into a create and a binary.
- [#1419](FuelLabs/fuel-core#1419): Add
additional "sanity" benchmarks for arithmetic op code instructions.
- [#1411](FuelLabs/fuel-core#1411): Added WASM
and `no_std` compatibility.
- [#1405](FuelLabs/fuel-core#1405): Use correct
names for service metrics.
- [#1400](FuelLabs/fuel-core#1400): Add releasy
beta to fuel-core so that new commits to fuel-core master triggers
fuels-rs.
- [#1371](FuelLabs/fuel-core#1371): Add new
client function for querying the `MessageStatus` for a specific message
(by `Nonce`).
- [#1356](FuelLabs/fuel-core#1356): Add peer
reputation reporting to heartbeat code.
- [#1355](FuelLabs/fuel-core#1355): Added new
metrics related to block importing, such as tps, sync delays etc.
- [#1339](FuelLabs/fuel-core#1339): Adds
`baseAssetId` to `FeeParameters` in the GraphQL API.
- [#1331](FuelLabs/fuel-core#1331): Add peer
reputation reporting to block import code.
- [#1324](FuelLabs/fuel-core#1324): Added
pyroscope profiling to fuel-core, intended to be used by a secondary
docker image that has debug symbols enabled.
- [#1309](FuelLabs/fuel-core#1309): Add
documentation for running debug builds with CLion and Visual Studio
Code.
- [#1308](FuelLabs/fuel-core#1308): Add support
for loading .env files when compiling with the `env` feature. This
allows users to conveniently supply CLI arguments in a secure and
IDE-agnostic way.
- [#1304](FuelLabs/fuel-core#1304): Implemented
`submit_and_await_commit_with_receipts` method for `FuelClient`.
- [#1286](FuelLabs/fuel-core#1286): Include
readable names for test cases where missing.
- [#1274](FuelLabs/fuel-core#1274): Added tests
to benchmark block synchronization.
- [#1263](FuelLabs/fuel-core#1263): Add gas
benchmarks for `ED19` and `ECR1` instructions.

### Changed

- [#1512](FuelLabs/fuel-core#1512): Internally
simplify merkle_contract_state_range.
- [#1507](FuelLabs/fuel-core#1507): Updated
chain configuration to be ready for beta 5 network. It includes opcode
prices from the latest benchmark and contract for the block producer.
- [#1477](FuelLabs/fuel-core#1477): Upgraded the
Rust version used in CI and containers to 1.73.0. Also includes
associated Clippy changes.
- [#1469](FuelLabs/fuel-core#1469): Replaced
usage of `MemoryTransactionView` by `Checkpoint` database in the
benchmarks.
- [#1468](FuelLabs/fuel-core#1468): Bumped
version of the `fuel-vm` to `v0.40.0`. It brings some breaking changes
into consensus parameters API because of changes in the underlying
types.
- [#1466](FuelLabs/fuel-core#1466): Handling
overflows during arithmetic operations.
- [#1460](FuelLabs/fuel-core#1460): Change
tracking branch from main to master for releasy tests.
- [#1454](FuelLabs/fuel-core#1454): Update gas
benchmarks for opcodes that append receipts.
- [#1440](FuelLabs/fuel-core#1440): Don't report
reserved nodes that send invalid transactions.
- [#1439](FuelLabs/fuel-core#1439): Reduced
memory BMT consumption during creation of the header.
- [#1434](FuelLabs/fuel-core#1434): Continue
gossiping transactions to reserved peers regardless of gossiping
reputation score.
- [#1408](FuelLabs/fuel-core#1408): Update gas
benchmarks for storage opcodes to use a pre-populated database to get
more accurate worst-case costs.
- [#1399](FuelLabs/fuel-core#1399): The Relayer
now queries Ethereum for its latest finalized block instead of using a
configurable "finalization period" to presume finality.
- [#1397](FuelLabs/fuel-core#1397): Improved
keygen. Created a crate to be included from forc plugins and upgraded
internal library to drop requirement of protoc to build
- [#1395](FuelLabs/fuel-core#1395): Add
DependentCost benchmarks for `k256`, `s256` and `mcpi` instructions.
- [#1393](FuelLabs/fuel-core#1393): Increase
heartbeat timeout from `2` to `60` seconds, as suggested in [this
issue](FuelLabs/fuel-core#1330).
- [#1392](FuelLabs/fuel-core#1392): Fixed an
overflow in `message_proof`.
- [#1390](FuelLabs/fuel-core#1390): Up the
`ethers` version to `2` to fix an issue with `tungstenite`.
- [#1383](FuelLabs/fuel-core#1383): Disallow
usage of `log` crate internally in favor of `tracing` crate.
- [#1380](FuelLabs/fuel-core#1380): Add
preliminary, hard-coded config values for heartbeat peer reputation,
removing `todo`.
- [#1377](FuelLabs/fuel-core#1377): Remove
`DiscoveryEvent` and use `KademliaEvent` directly in
`DiscoveryBehavior`.
- [#1366](FuelLabs/fuel-core#1366): Improve
caching during docker builds in CI by replacing gha
- [#1358](FuelLabs/fuel-core#1358): Upgraded the
Rust version used in CI to 1.72.0. Also includes associated Clippy
changes.
- [#1349](FuelLabs/fuel-core#1349): Updated
peer-to-peer transactions API to support multiple blocks in a single
request, and updated block synchronization to request multiple blocks
based on the configured range of headers.
- [#1342](FuelLabs/fuel-core#1342): Add error
handling for P2P requests to return `None` to requester and log error.
- [#1318](FuelLabs/fuel-core#1318): Modified
block synchronization to use asynchronous task execution when retrieving
block headers.
- [#1314](FuelLabs/fuel-core#1314): Removed
`types::ConsensusParameters` in favour of `fuel_tx:ConsensusParameters`.
- [#1302](FuelLabs/fuel-core#1302): Removed the
usage of flake and building of the bridge contract ABI.
It simplifies the maintenance and updating of the events, requiring only
putting the event definition into the codebase of the relayer.
- [#1293](FuelLabs/fuel-core#1293):
Parallelized the `estimate_predicates` endpoint to utilize all available
threads.
- [#1270](FuelLabs/fuel-core#1270): Modify the
way block headers are retrieved from peers to be done in batches.

#### Breaking
- [#1506](FuelLabs/fuel-core#1506): Added
validation of the coin's fields during block production and validation.
Before, it was possible to submit a transaction that didn't match the
coin's values in the database, allowing printing/using unavailable
assets.
- [#1491](FuelLabs/fuel-core#1491): Removed
unused request and response variants from the Gossipsub implementation,
as well as related definitions and tests. Specifically, this removes
gossiping of `ConsensusVote` and `NewBlock` events.
- [#1472](FuelLabs/fuel-core#1472): Upgraded
`fuel-vm` to `v0.42.0`. It introduces transaction policies that changes
layout of the transaction. FOr more information check the
[v0.42.0](FuelLabs/fuel-vm#635) release.
- [#1470](FuelLabs/fuel-core#1470): Divide
`DependentCost` into "light" and "heavy" operations.
- [#1464](FuelLabs/fuel-core#1464): Avoid
possible truncation of higher bits. It may invalidate the code that
truncated higher bits causing different behavior on 32-bit vs. 64-bit
systems. The change affects some endpoints that now require lesser
integers.
- [#1432](FuelLabs/fuel-core#1432): All
subscriptions and requests have a TTL now. So each subscription
lifecycle is limited in time. If the subscription is closed because of
TTL, it means that you subscribed after your transaction had been
dropped by the network.
- [#1407](FuelLabs/fuel-core#1407): The
recipient is a `ContractId` instead of `Address`. The block producer
should deploy its contract to receive the transaction fee. The collected
fee is zero until the recipient contract is set.
- [#1407](FuelLabs/fuel-core#1407): The `Mint`
transaction is reworked with new fields to support the account-base
model. It affects serialization and deserialization of the transaction
and also affects GraphQL schema.
- [#1407](FuelLabs/fuel-core#1407): The `Mint`
transaction is the last transaction in the block instead of the first.
- [#1374](FuelLabs/fuel-core#1374): Renamed
`base_chain_height` to `da_height` and return current relayer height
instead of latest Fuel block height.
- [#1367](FuelLabs/fuel-core#1367): Update to
the latest version of fuel-vm.
- [#1363](FuelLabs/fuel-core#1363): Change
message_proof api to take `nonce` instead of `message_id`
- [#1355](FuelLabs/fuel-core#1355): Removed the
`metrics` feature flag from the fuel-core crate, and metrics are now
included by default.
- [#1339](FuelLabs/fuel-core#1339): Added a new
required field called `base_asset_id` to the `FeeParameters` definition
in `ConsensusParameters`, as well as default values for `base_asset_id`
in the `beta` and `dev` chain specifications.
- [#1322](FuelLabs/fuel-core#1322):
The `debug` flag is added to the CLI. The flag should be used for local
development only. Enabling debug mode:
      - Allows GraphQL Endpoints to arbitrarily advance blocks.
      - Enables debugger GraphQL Endpoints.
      - Allows setting `utxo_validation` to `false`.
- [#1318](FuelLabs/fuel-core#1318): Removed the
`--sync-max-header-batch-requests` CLI argument, and renamed
`--sync-max-get-txns` to `--sync-block-stream-buffer-size` to better
represent the current behavior in the import.
- [#1290](FuelLabs/fuel-core#1290): Standardize
CLI args to use `-` instead of `_`.
- [#1279](FuelLabs/fuel-core#1279): Added a new
CLI flag to enable the Relayer service `--enable-relayer`, and disabled
the Relayer service by default. When supplying the `--enable-relayer`
flag, the `--relayer` argument becomes mandatory, and omitting it is an
error. Similarly, providing a `--relayer` argument without the
`--enable-relayer` flag is an error. Lastly, providing the `--keypair`
or `--network` arguments will also produce an error if the
`--enable-p2p` flag is not set.
- [#1262](FuelLabs/fuel-core#1262): The
`ConsensusParameters` aggregates all configuration data related to the
consensus. It contains many fields that are segregated by the usage. The
API of some functions was affected to use lesser types instead the whole
`ConsensusParameters`. It is a huge breaking change requiring
repetitively monotonically updating all places that use the
`ConsensusParameters`. But during updating, consider that maybe you can
use lesser types. Usage of them may simplify signatures of methods and
make them more user-friendly and transparent.

### Removed

#### Breaking
- [#1484](FuelLabs/fuel-core#1484): Removed
`--network` CLI argument. Now the name of the network is fetched form
chain configuration.
- [#1399](FuelLabs/fuel-core#1399): Removed
`relayer-da-finalization` parameter from the relayer CLI.
- [#1338](FuelLabs/fuel-core#1338): Updated
GraphQL client to use `DependentCost` for `k256`, `mcpi`, `s256`,
`scwq`, `swwq` opcodes.
- [#1322](FuelLabs/fuel-core#1322): The
`manual_blocks_enabled` flag is removed from the CLI. The analog is a
`debug` flag.

## What's Changed
* Added changelog in the same way as we did for `fuel-vm` by @xgreenx in
FuelLabs/fuel-core#1287
* Decompose consensus params struct by @MitchTurner in
FuelLabs/fuel-core#1262
* Replace all instances of `_` in CLI `long` arguments with `-` by
@MitchTurner in FuelLabs/fuel-core#1290
* [rpc] Parallelise estimate_predicates endpoint by @YusongWang in
FuelLabs/fuel-core#1293
* Add readable test names to test cases by @MitchTurner in
FuelLabs/fuel-core#1286
* ED19 and ECR1 benchmarks by @Dentosal in
FuelLabs/fuel-core#1263
* feat: Add `enable-relayer` flag to `fuel-core` CLI by @bvrooman in
FuelLabs/fuel-core#1279
* Small nits found during deploying and benchmarking by @xgreenx in
FuelLabs/fuel-core#1303
* Implemented `submit_and_await_commit_with_receipts` method by @xgreenx
in FuelLabs/fuel-core#1304
* Removed the usage of flake and building of the bridge contract ABI by
@xgreenx in FuelLabs/fuel-core#1302
* Download range of headers during syncing by @MitchTurner in
FuelLabs/fuel-core#1270
* feat: Enable .env file loading using `env` feature by @bvrooman in
FuelLabs/fuel-core#1308
* chore: Remove types::ConsensusParameters by @bvrooman in
FuelLabs/fuel-core#1314
* docs: Debugging instructions using IDEs by @bvrooman in
FuelLabs/fuel-core#1309
* test: Block Import benchmarks and test helpers by @bvrooman in
FuelLabs/fuel-core#1274
* Fix audit CI for `webpki` by @xgreenx in
FuelLabs/fuel-core#1319
* chore: Single buffer block sync by @bvrooman in
FuelLabs/fuel-core#1318
* Run unit tests without default features only for specific crates by
@xgreenx in FuelLabs/fuel-core#1321
* Added support for `debug` CLI flag. by @xgreenx in
FuelLabs/fuel-core#1322
* Report Peers that give bad Block Info by @MitchTurner in
FuelLabs/fuel-core#1331
* Pyroscope profiling by @Voxelot in
FuelLabs/fuel-core#1324
* Add error handling for P2P requests to return `None` to requester and
log error by @MitchTurner in
FuelLabs/fuel-core#1342
* Importer Metrics by @Voxelot in
FuelLabs/fuel-core#1355
* chore: Upgrade to Rust 1.72.0 in CI by @bvrooman in
FuelLabs/fuel-core#1358
* chore: Upgrade `fuel-vm` to 0.37.0 by @bvrooman in
FuelLabs/fuel-core#1338
* docs: remove stale schema by @calldelegation in
FuelLabs/fuel-core#1362
* Replace message_id with nonce in `MessageProof` query by @MitchTurner
in FuelLabs/fuel-core#1363
* feat: Configurable base asset by @bvrooman in
FuelLabs/fuel-core#1339
* avoid gha caching for docker builds by @Voxelot in
FuelLabs/fuel-core#1366
* Add tracking for peer heartbeats by @MitchTurner in
FuelLabs/fuel-core#1356
* docs: fix broken link in README by @PaulRBerg in
FuelLabs/fuel-core#1357
* Use right level of error for `.env` file and duplicated service by
@xgreenx in FuelLabs/fuel-core#1372
* Add query and handling for `MessageStatus` by @MitchTurner in
FuelLabs/fuel-core#1371
* Renamed `base_chain_height` to `da_height` and return current relayer
height instead of latest Fuel block height by @xgreenx in
FuelLabs/fuel-core#1374
* Remove `DiscoveryEvent` and use `KademliaEvent` instead by
@MitchTurner in FuelLabs/fuel-core#1377
* Remove todo from code and provide some preliminary peer review config
values by @MitchTurner in
FuelLabs/fuel-core#1380
* Clippy lint to disallow logging crate macros in favor of tracing by
@Dentosal in FuelLabs/fuel-core#1383
* fuel-vm nostd update by @Dentosal in
FuelLabs/fuel-core#1367
* Up the `ethers` version to `2` to fix an issue with `tungstenite` by
@xgreenx in FuelLabs/fuel-core#1390
* Increase heartbeat timeout to one minute by @Dentosal in
FuelLabs/fuel-core#1393
* feat: batch transactions import by @bvrooman in
FuelLabs/fuel-core#1349
* Fix an interger overflow in message_proof when looking beyond genesis
by @Dentosal in FuelLabs/fuel-core#1392
* Improve keygen by @cr-fuel in
FuelLabs/fuel-core#1397
* Benchmark {k256, s256, mcpi} opcodes using DependentCost by @Dentosal
in FuelLabs/fuel-core#1395
* Use correct names for service metrics by @xgreenx in
FuelLabs/fuel-core#1405
* feat: Retrieve finalized blocks from Ethereum by @bvrooman in
FuelLabs/fuel-core#1399
* Updated CI to run `cargo update` weekly by @xgreenx in
FuelLabs/fuel-core#1415
* CI `cargo update` weekly by @xgreenx in
FuelLabs/fuel-core#1416
* Weekly `cargo update` by @github-actions in
FuelLabs/fuel-core#1417
* Fix typos by @GoodDaisy in
FuelLabs/fuel-core#1424
* feat: Support no_std and WASM compilation for `fuel-core` crates by
@bvrooman in FuelLabs/fuel-core#1411
* Create sanity benchmark checks for all the arithmetic op codes by
@MitchTurner in FuelLabs/fuel-core#1419
* Update benchmarking and collecting for `scwq`, `swwq` and `srwq`
opcodes by @xgreenx in FuelLabs/fuel-core#1427
* Account-base fee collection by @xgreenx in
FuelLabs/fuel-core#1407
* Rollback modification of the chain specification by @xgreenx in
FuelLabs/fuel-core#1429
* Tx subscription cleanup (#1422) by @xgreenx in
FuelLabs/fuel-core#1432
* Mark reserved peers as explicit for gossipsub to avoid reputation
decreasing (#1423) by @xgreenx in
FuelLabs/fuel-core#1434
* Weekly `cargo update` by @github-actions in
FuelLabs/fuel-core#1438
* ci: add beta releasy to fuel-core by @kayagokalp in
FuelLabs/fuel-core#1400
* setup a cron job to test beta-4 continuously by @Voxelot in
FuelLabs/fuel-core#1436
* chore: Use MerkleRootCalculator when only BMT root is needed by
@bvrooman in FuelLabs/fuel-core#1439
* Crypto op code sanity checks by @MitchTurner in
FuelLabs/fuel-core#1430
* Update gas benchmarks for some storage opcodes by @Dentosal in
FuelLabs/fuel-core#1408
* Split keygen into a create and a binary by @cr-fuel in
FuelLabs/fuel-core#1426
* Basic Transfer TPS Benchmarking by @Voxelot in
FuelLabs/fuel-core#1437
* "flow" op code sanity benchmarks by @MitchTurner in
FuelLabs/fuel-core#1433
* Applying `#1435` to `master` by @xgreenx in
FuelLabs/fuel-core#1440
* "memory" opcode sanity checks by @MitchTurner in
FuelLabs/fuel-core#1444
* Add timeout to E2E test by @Voxelot in
FuelLabs/fuel-core#1447
* fix owns_coins to properly paginate by @Voxelot in
FuelLabs/fuel-core#1449
* Fixing incorrect measurement for fast(µs) opcodes by @xgreenx in
FuelLabs/fuel-core#1457
* Call `flush` during end of the `FuelService` by @xgreenx in
FuelLabs/fuel-core#1456
* chore: fix releasy test tracking branch from `main` to `master` by
@kayagokalp in FuelLabs/fuel-core#1460
* Weekly `cargo update` by @github-actions in
FuelLabs/fuel-core#1461
* `DB::repair` breaks the table for checkpoints by @xgreenx in
FuelLabs/fuel-core#1463
* Avoid possible truncation of higher bits by @xgreenx in
FuelLabs/fuel-core#1464
* Bumping `fuel-vm` to `v0.40.0` by @xgreenx in
FuelLabs/fuel-core#1468
* Improvements for keygen cli and crates by @cr-fuel in
FuelLabs/fuel-core#1465
* Handling overflows during arithmetic operations by @xgreenx in
FuelLabs/fuel-core#1466
* test: Add contract code root benchmark by @bvrooman in
FuelLabs/fuel-core#1452
* test: Add state root and contract id benchmarks by @bvrooman in
FuelLabs/fuel-core#1462
* Avoid storage caching in benchmarks by @xgreenx in
FuelLabs/fuel-core#1469
* Weekly `cargo update` by @github-actions in
FuelLabs/fuel-core#1471
* Expose fuel-core version as a constant by @Br1ght0ne in
FuelLabs/fuel-core#1473
* Upgrade ed25519_dalek 1.0.1 -> 2.0.0 by @Dentosal in
FuelLabs/fuel-core#1475
* chore: Upgrade Rust 1.73 by @bvrooman in
FuelLabs/fuel-core#1477
* Decrease number of iterations for sequential benchmarks by @xgreenx in
FuelLabs/fuel-core#1480
* Add mldv bench by @Dentosal in
FuelLabs/fuel-core#1481
* feat: Separate light and heavy gas costs (core) by @bvrooman in
FuelLabs/fuel-core#1483
* chore: Remove `network` parameter from CLI by @bvrooman in
FuelLabs/fuel-core#1484
* Transaction policies support by @xgreenx in
FuelLabs/fuel-core#1472
* stage release candidate for beta-5 by @Voxelot in
FuelLabs/fuel-core#1485
* setup docker auth for binary publishing by @Voxelot in
FuelLabs/fuel-core#1488
* Fix macos binary publishing by @xgreenx in
FuelLabs/fuel-core#1489
* Add documentation for our calculation of a _reasonable_ gas fee for
storage by @MitchTurner in
FuelLabs/fuel-core#1478
* "contract" op code sanity checks by @MitchTurner in
FuelLabs/fuel-core#1453
* "other" op code sanity check by @MitchTurner in
FuelLabs/fuel-core#1476
* Add push and pop benches by @MitchTurner in
FuelLabs/fuel-core#1490
* chore: Remove unused gossipsub items by @bvrooman in
FuelLabs/fuel-core#1491
* Adjusting of the prices benchmarks after sanity check benchmarks by
@xgreenx in FuelLabs/fuel-core#1494
* Weekly `cargo update` by @github-actions in
FuelLabs/fuel-core#1498
* Update benches that are affected by receipts by @Dentosal in
FuelLabs/fuel-core#1454
* Support backward iteration in the RocksDB by @xgreenx in
FuelLabs/fuel-core#1492
* Added validation of the coin's fields during block production and
validation by @xgreenx in
FuelLabs/fuel-core#1506
* Added price benchmark for `vm_initialization` by @xgreenx in
FuelLabs/fuel-core#1502
* Sanity-check `gtf` opcode by @MitchTurner in
FuelLabs/fuel-core#1503
* Tool for generating fee collection contract by @Voxelot in
FuelLabs/fuel-core#1501
* Chain config for beta 5 network by @xgreenx in
FuelLabs/fuel-core#1507
* Remove usage of iter_all_filtered from merkle_contract_state_range by
@Dentosal in FuelLabs/fuel-core#1512

## New Contributors
* @YusongWang made their first contribution in
FuelLabs/fuel-core#1293
* @calldelegation made their first contribution in
FuelLabs/fuel-core#1362
* @PaulRBerg made their first contribution in
FuelLabs/fuel-core#1357
* @cr-fuel made their first contribution in
FuelLabs/fuel-core#1397
* @github-actions made their first contribution in
FuelLabs/fuel-core#1417
* @GoodDaisy made their first contribution in
FuelLabs/fuel-core#1424

**Full Changelog**:
FuelLabs/fuel-core@v0.20.8...v0.21.0

---------

Co-authored-by: Brandon Vrooman <[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.

5 participants