Release v1.7.14 #55
+130
−0
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,122 @@ | ||
| [comment]: <> (tags/v1.7.15) | ||
|
|
||
| # Contents | ||
|
|
||
| This document explains the contents of the rc/v1.7.next1 release codenamed Interim. It is split in 2 sections: | ||
| - the **features** list containing detailed insights of the feature along with the external impact and the relevant | ||
| pull requests list | ||
| - the **smaller features and fixes** area contains the one-pull request small features or fixes along with the | ||
| external impact details | ||
|
|
||
| This documentation is relevant for the `tags/v1.7.15` tag release. | ||
|
|
||
| # Features | ||
|
|
||
| ## 1. Relayed v3 [#5741](https://github.com/multiversx/mx-chain-go/pull/5741) | ||
|
There was a problem hiding this comment. Maybe link towards docs, as well? There was a problem hiding this comment. added. it was not pushed yet when I initially opened the PR. |
||
|
|
||
| Relayed v3 feature brings a cheaper, improved version of relayed transactions, that can hold multiple inner transactions. | ||
| The inner transactions will be full transactions. | ||
|
|
||
| With this feature, a fix on the base cost of relayed transactions was implemented, to follow the minimum gas rule. | ||
|
There was a problem hiding this comment. Additionally. Also, maybe it should be specified that this additional fix affects all types of relayed transactions? |
||
|
|
||
| ### Impact: | ||
|
There was a problem hiding this comment. Maybe we should split the impact section into more sections - impact on validator, on developers, on integrators etc.? There was a problem hiding this comment. Additional note for integrators that read blockchain data - they need to handle relayed V3 transactions. There was a problem hiding this comment. added a note that v1 and v2 will be deprecated, should be pretty clear that this is a new type of tx that must be handled separately |
||
| * There are new enable epochs definitions for this feature, called `RelayedTransactionsV3EnableEpoch`, `FixRelayedBaseCostEnableEpoch` | ||
| * There are changes to the configuration options in the `config.toml` file, a new section called `[RelayedTransactionConfig]` was added | ||
| * There is a new field in the `Transaction` structure, `InnerTransactions`, that can hold multiple other `Transaction` as part of relayed v3 | ||
| * No binary flags changes | ||
| * No API endpoints changes | ||
|
|
||
| ### Relevant PRs: | ||
| - [#5572](https://github.com/multiversx/mx-chain-go/pull/5572) - New approach for relayed V3, transaction structure changed | ||
| - [#5627](https://github.com/multiversx/mx-chain-go/pull/5627) - Separate fee handling for inner tx of type move balance | ||
| - [#5981](https://github.com/multiversx/mx-chain-go/pull/5981) - Added api check for recursive relayed v3 + fixed interceptor | ||
| - [#6252](https://github.com/multiversx/mx-chain-go/pull/6252) - RelayedV3 fixes | ||
| - [#6176](https://github.com/multiversx/mx-chain-go/pull/6176) - Append log events for all inner transactions failure | ||
| - [#6274](https://github.com/multiversx/mx-chain-go/pull/6274) - Rename FixRelayedMoveBalanceFlag to FixRelayedBaseCostFlag + proper fix | ||
| - [#6278](https://github.com/multiversx/mx-chain-go/pull/6278) - Fixed processTxFee for inner tx after base cost fix | ||
| - [#6280](https://github.com/multiversx/mx-chain-go/pull/6280) - Relayedv3 further fixes | ||
| - [#6297](https://github.com/multiversx/mx-chain-go/pull/6297) - Added more integration tests for non-executable inner tx + small fix on logs append | ||
| - [#6302](https://github.com/multiversx/mx-chain-go/pull/6302) - Change receivers ids relayed v3 and multi transfer integration | ||
| - [#6328](https://github.com/multiversx/mx-chain-go/pull/6328) - Fixed tests by using real FailedTxLogsAccumulator | ||
| - [#6147](https://github.com/multiversx/mx-chain-go/pull/6147) - Extra tests and coverage for relayed v3 with multiple inner transactions | ||
|
|
||
| ## 2. ESDT improvements [#5821](https://github.com/multiversx/mx-chain-go/pull/5821) | ||
|
|
||
| The improvements implemented on ESDTs enable the dynamic NFT functionality. | ||
|
There was a problem hiding this comment. Maybe link towards docs, as well? There was a problem hiding this comment. not yet merged, will be updated once it is done |
||
|
|
||
| ### Impact: | ||
|
There was a problem hiding this comment. Maybe mention the new available transactions, or why was this feature implemented? There was a problem hiding this comment. right, added them |
||
| * There is a new enable epoch definition for this feature, called `DynamicESDTEnableEpoch` | ||
| * There are changes to the configuration options in the gas schedule files, new values were added for `ESDTModifyRoyalties`, `ESDTModifyCreator`,`ESDTNFTRecreate`,`ESDTNFTUpdate`, `ESDTNFTSetNewURIs` | ||
| * No binary flags changes | ||
|
There was a problem hiding this comment. Here, and below, "binary flags" can be renamed to "Node CLI arguments". |
||
| * API endpoints `/:address/nft/:tokenIdentifier/nonce/:nonce` and `/:address/esdt` will now return token type as well | ||
|
|
||
| ### Relevant PRs: | ||
| - [#5806](https://github.com/multiversx/mx-chain-go/pull/5806) - ESDT improvements integration | ||
| - [#5682](https://github.com/multiversx/mx-chain-go/pull/5682) - ESDT v2 implementations on SystemSC - dynamic tokens and tokenType | ||
| - [#6284](https://github.com/multiversx/mx-chain-go/pull/6284) - ESDT Improvements integration tests | ||
| - [#6194](https://github.com/multiversx/mx-chain-go/pull/6194) - ESDT improvements chain simulator tests and fixes | ||
| - [#6262](https://github.com/multiversx/mx-chain-go/pull/6262) - ESDT improvements integration tests part2 + fixes | ||
| - [#6250](https://github.com/multiversx/mx-chain-go/pull/6250) - Do not allow NFTs to be upgraded to dynamic | ||
| - [#6263](https://github.com/multiversx/mx-chain-go/pull/6263) - Add more testing scenarios | ||
| - [#6242](https://github.com/multiversx/mx-chain-go/pull/6242) - Add esdt token type api | ||
| - [#6220](https://github.com/multiversx/mx-chain-go/pull/6220) - Token type in altered accounts | ||
| - [#6350](https://github.com/multiversx/mx-chain-go/pull/6350) - Fix missing role + Use setSpecialRole function in tests | ||
|
|
||
| ## 3. Crypto API, new Opcodes and EGLD in MultiESDTTransfer [#6139](https://github.com/multiversx/mx-chain-go/pull/6139) | ||
|
|
||
| With this feature, new opcodes were enabled for developers, among with new crypto endpoints(`VerifySecp256r1`, `VerifyBLSSignatureShare`, `VerifyBLSMultiSig`). | ||
|
There was a problem hiding this comment. We can specify that these endpoints are VM endpoints / VM EEI additions - since the word "endpoint" is a bit ambiguous. |
||
| Another improvement was EGLD as part of MultiESDTTransfer (EGLD-000000). | ||
|
There was a problem hiding this comment. It's not obvious that since this upgrade, one can send native tokens along with custom ones by means of a MultiESDTTransfer. There was a problem hiding this comment. updated + added note |
||
|
|
||
| ### Impact: | ||
|
There was a problem hiding this comment. Additional note for integrators that read blockchain data - they need to handle EGLD transfers within MultiESDTTransfer. |
||
| * There are new enable epochs definitions for this feature, called `EGLDInMultiTransferEnableEpoch` and `CryptoOpcodesV2EnableEpoch` | ||
| * There are changes to the configuration options in the gas schedule files, new values were added for `VerifySecp256r1`, `VerifyBLSSignatureShare`,`VerifyBLSMultiSig` | ||
| * No binary flags changes | ||
| * No API endpoints changes | ||
|
There was a problem hiding this comment. In this context, API is ambiguous: VM API vs. HTTP API. Perhaps say Node HTTP API (here and above)? |
||
|
|
||
| ### Relevant PRs: | ||
| - [#6334](https://github.com/multiversx/mx-chain-go/pull/6334) - Multi transfer execute by user flag | ||
| - [#6313](https://github.com/multiversx/mx-chain-go/pull/6313) - Added egld with multi transfer test scenario | ||
|
|
||
| ## 4. Refactor persister factory [#6001](https://github.com/multiversx/mx-chain-go/pull/6001) | ||
|
|
||
| The storage unit package was refactored, adding the option of using static and non-static storers. | ||
|
|
||
| ### Impact: | ||
| * No activation epoch | ||
| * No configuration files changes | ||
| * No binary flags changes | ||
| * No API endpoints changes | ||
|
There was a problem hiding this comment. Any changes within the database structure that is worth mentioning? Alternatively, replace section with a simple statement e.g. no impact on validators / developers etc.? |
||
|
|
||
| ### Relevant PRs: | ||
| - [#5797](https://github.com/multiversx/mx-chain-go/pull/5797) - Use create with retries in persister factory | ||
| - [#5995](https://github.com/multiversx/mx-chain-go/pull/5995) - Persister tmp file path | ||
| - [#6010](https://github.com/multiversx/mx-chain-go/pull/6010) - Update to use latest storage version | ||
|
|
||
| # Smaller features or fixes | ||
|
|
||
| - [#5851](https://github.com/multiversx/mx-chain-go/pull/5851) - Overridable config structs | ||
| - [#6116](https://github.com/multiversx/mx-chain-go/pull/6116) - Added map support for overridable configs | ||
| - [#5897](https://github.com/multiversx/mx-chain-go/pull/5897) - Added the possibility to define more protocol IDs for p2p networks | ||
| - [#6186](https://github.com/multiversx/mx-chain-go/pull/6186) - Proposer optimisation | ||
| - [#6225](https://github.com/multiversx/mx-chain-go/pull/6225) - Update enable epochs flags | ||
| - [#6184](https://github.com/multiversx/mx-chain-go/pull/6184) - Chain simulator improvements | ||
| - [#6182](https://github.com/multiversx/mx-chain-go/pull/6182) - Chain simulator custom transactions sender component | ||
| - [#6173](https://github.com/multiversx/mx-chain-go/pull/6173) - Chain simulator force change of epoch | ||
| - [#6182](https://github.com/multiversx/mx-chain-go/pull/6182) - Chain simulator custom transactions sender component | ||
| - [#6150](https://github.com/multiversx/mx-chain-go/pull/6150) - Chain simulator updates for sovereign shards | ||
| - [#6217](https://github.com/multiversx/mx-chain-go/pull/6217) - Chain simulator tests refactor | ||
| - [#6320](https://github.com/multiversx/mx-chain-go/pull/6320) - Chain simulator extra parameter for delay between vm queries | ||
| - [#6268](https://github.com/multiversx/mx-chain-go/pull/6268) - Chain simulator verify tx on send | ||
| - [#6248](https://github.com/multiversx/mx-chain-go/pull/6248) - Chain simulator remove consensus group size | ||
| - [#6018](https://github.com/multiversx/mx-chain-go/pull/6018) - TrieRecreate refactor | ||
| - [#6238](https://github.com/multiversx/mx-chain-go/pull/6238) - FIX: Destination shard id in chain simulator for meta chain addresses | ||
| - [#6292](https://github.com/multiversx/mx-chain-go/pull/6292) - Extend delegation log events for multi claim and multi redelegate rewards | ||
| - [#6117](https://github.com/multiversx/mx-chain-go/pull/6117) - Integrate new vm-common: events for "claim developer rewards" | ||
| - [#6039](https://github.com/multiversx/mx-chain-go/pull/6039) - Added missing fields on the transaction/pool by-sender request | ||
| - [#6162](https://github.com/multiversx/mx-chain-go/pull/6162) - Adjust workflow runners (MacOS) | ||
| - [#6247](https://github.com/multiversx/mx-chain-go/pull/6247) - Fixed error on withKeys | ||
| - [#6161](https://github.com/multiversx/mx-chain-go/pull/6161) - Cleanup proposal misses | ||
| - [#6322](https://github.com/multiversx/mx-chain-go/pull/6322) - Optimize "DisplayProcessTxDetails": Early exit if log level is not TRACE | ||
| - [#6333](https://github.com/multiversx/mx-chain-go/pull/6333) - Fix node/termui dockerfiles | ||
| - [#6281](https://github.com/multiversx/mx-chain-go/pull/6281) - Update Dockerfiles | ||
| - [#6275](https://github.com/multiversx/mx-chain-go/pull/6275) - Added action for building keygenerator docker images | ||
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Codename should be a star name.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Decided with @schimih that this one would be called Interim for now.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
https://dictionary.cambridge.org/dictionary/english/interim