OriginTrail Testnet Release v6.0.17 #2748
Merged
Conversation
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
OriginTrail Mainnet Prerelease v6.0.16
OriginTrail Mainnet Release v6.0.16
* Transform telemtry-service to module * Remove old telemetry files * Remove unneeded logging * Add telemetry module migration * Add telemetry module enabled check before startin it * Add ot-telemetry config * Rename telemetry migration class and file * Remove async from startTelemetryModule * Rename telemetry-service to ot-telemetry * Refine migration to only update user configuration telemetry settings * Remove old telemetry configuration * Fix version * Use getModuleConfiguration to get implementation config * Refactor sendTelemetryData to return a boolean for response success * Remove unneeded enable field * Fix onEvenetRecived -> onEventReceived typo * Fix ot-telemetry configuration for testnet and mainnet * Add unit test * Use getModuleConfiguration without argument * Revert package-lock changes * Update tests * Update tests * Format files using Prettier * Set better name for test --------- Co-authored-by: Mihajlo Pavlovic <[email protected]> Co-authored-by: Mihajlo Pavlovic <[email protected]> Co-authored-by: Nikola Todorovic <[email protected]>
…g when proof is empty, removed duplicated SA record update, increased tx polling timeout to 120 seconds
Fix/transactions timeout
Branches leveling
OriginTrail Devnet Prerelease v6.0.17
OriginTrail Devnet Release v6.0.17
OriginTrail Testnet Prerelease v6.0.17
u-hubar
added
bug
u-hubar
requested review from
branarakic,
djordjekovac,
NZT48 and
zeroxbt
as code owners
brkagithub
approved these changes
Nov 3, 2023
NZT48
approved these changes
Nov 3, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Description
The 6.0.17 release brings several noteworthy improvements aiming to fix issues when the ot-node submits erroneous transactions, which will reduce the number of failed transactions on the blockchain. The transaction timeout is increased to 120 seconds from 50s. Additional error handling has been added to commits and proofs commands, and a patch has been added to ensure nodes do not send empty proofs. The commit and proof retries have been improved and the gas is increased properly between retries. Removed a redundant service agreement record update in the operational database. The telemetry service has been upgraded to a module.
🌟 Improvements
Increased txs polling timeout to 120 seconds (#2741)
Added error handling for commits/proofs commands (#2741)
Added check if calculated proof isn't empty (#2741)
Improved commits/proofs commands retries (#2741)
Removed duplicated SA record update in the operational db (#2741)
🧰 Maintenance
Transform telemetry service to module (#2737)
#TraceOn