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

Releases/v2.1.0 #1263

Draft
wants to merge 16 commits into
base: main
Choose a base branch
from
Draft

Releases/v2.1.0 #1263

wants to merge 16 commits into from

Conversation

Yashk767
Copy link
Contributor

@Yashk767 Yashk767 commented Jan 2, 2025

Description

This is a release PR for release v2.1.0

chore: merge main to develop post Releases/v2.0.0
* chore: updated config variable default and limits for 5 min epoch (#1232)

* feat: added timeout to retry mechanism using context

* refactor: called stateBuffer once and used in other functions

* refactor: added context to RPC calls used in vote function

* refactor: added context instance to all the dependent functions

* refactor: fixed tests for cmd package

* refactor: fixed tests for utils package

* refactor: fixed additional tests in cmd package

* refactor: fixed benchmarks in cmd package

* refactor: reduced number of retry attempts

* refactor: added tests for context deadline exceeded for retry case

* refactor: rebased retry params with current main branch v2.0.0 release
… retry function (#1252)

* refactor: replaced retry implementation with genernic retry function

* refactor: used generic retry function instead of individual retry implementation

* refactor: reverted nil with exisiting 0 big integer when error occurs in fetchBalance
…c retry function to make build successful (#1256)

* refactor: used generic retry function instead of individual retry implementation

* refactor: reverted nil with exisiting 0 big integer when error occurs in fetchBalance

* fix: added context to function parameters foe successfull build

* refactor: fixed tests

* refactor: fixed benchmarks
* refactor: replaced SKL with sFuel as a gas token in logs

* refactor: lowered the value for minimum sFuel required to warn the staker to add more

* refactor: rename sFuel to sFUEL
…r removing waitForBlockCompletion check for voting transactions (#1254)

* refactor: added verification layer on commit data from memory

* refactor: removed waitForBlockCompletion from propose and reveal

* fix: returned err instead of nil in GetConfirmedBlock

* reafctor: fixed tests

* refactor: fixed lint errors

* fix: returned supported struct type and added a condition to attempt confirmBlock when node just started voting

* refactor: fixed tests after rebasing

* refactor: removed commented unwanted tests

* refactor: Fetched selected commit data first and than applied commitment verification

* feat: saved commitment in the memory/file along with commit data and reused it in reveal
)

* chore: updated parameters to check transaction mined status

* feat: added timeout over WaitForBlockCompletion
* chore: updated geth and go version

* chore: update go and geth version for docker and workflows

* ci: fixed ci by updating dependencies

* ci: fix line change error
* feature: introduced RPC manager module

* feat: Implemented RPCParameters for all contract calls

* refactor: moved RPCParameters struct to RPC module

* fix: added assetId parameter to getActiveStatus retry call

* refactor: passed rpcParameters to Batch call functions instead of client

* fix: path to assets.json and client parameter index fixed

* refactor: tests/updated mocks

* refactor: revert mainnet addresses change

* fix: added ctx parameter in CheckTransactionReceipt

* fix: revert chain Id and contract addresses change

* fix: refreshed RPC list after confirm state every epoch

* fix: added disputes contract call to the blockManager struct

* refactor: fixed lint log error

* reafctor: fixed tests

* fix: calculated dynamically the path to endpoints.json file

* fix: endpoints.json file to be picked from .razor directory instaed of repo

* refactor: set up temp endpoints.json file for tests

* feat: added importEndpoints command

* refactor: removed alternateProvider functions and flag as its not required now

* fix: switch to next client only if the connection is successful

* feat: added state block check by introducing blockMonitor (#1262)

* feat: introduced block monitor to keep track of blocks

* refactor: removed global variables used in logger and added blockMonitor as a field

* refactor: used refactored logger module

* fix: fixed logger global instance

* refactor: exported logrus instance field from logger struct

* refactor: fixed tests

* refactor: removed  unwanted return variables in tests

* refactor: added log for current best endpoint URL after every refresh

* fix: added endpoint validation while switching endpoints

* fix: added switched returned type while switching endpoints and removed staleBlockCallback field

* refactor: used BestRPCClient from BestEndpoint

* refactor: renamed RPC module to rpc

* refactor: removed unwanted logs

* refactor: corrected importEndpoints command info
* refactor: removed fatal logs from GetTxnOpts()

* refactor: removed fatal log when address not found

* reafctor: fixed tests
Copy link

qodo-merge-pro-for-open-source bot commented Feb 20, 2025

CI Feedback 🧐

(Feedback updated until commit c12a45a)

A test triggered by this PR failed. Here is an AI-generated analysis of the failure:

Action: test

Failed stage: Set up job [❌]

Failure summary:

The action failed because it's using deprecated versions of two GitHub Actions:

  • actions/download-artifact@v3
  • actions/upload-artifact@v3
    These versions have been deprecated and GitHub is automatically failing
    workflows that use them. The workflow needs to be updated to use newer versions of these actions.

  • Relevant error logs:
    1:  ##[group]Operating System
    2:  Ubuntu
    ...
    
    32:  ##[endgroup]
    33:  ##[group]Download immutable action package 'actions/setup-go@v2'
    34:  Version: 2.2.0
    35:  Digest: sha256:decc21c9a7275ee14541d6c57e7999c727f6a61d50f7a0f2fb11dfb5c54506c2
    36:  Source commit SHA: bfdd3570ce990073878bf10f6b2d79082de49492
    37:  ##[endgroup]
    38:  Download action repository 'bissolli/gh-action-persist-workspace@v1' (SHA:d8cc100f79b61715d23f0669abedc29f81715a38)
    39:  Getting action download info
    40:  ##[error]This request has been automatically failed because it uses a deprecated version of `actions/download-artifact: v3`. Learn more: https://github.blog/changelog/2024-04-16-deprecation-notice-v3-of-the-artifact-actions/. This request has been automatically failed because it uses a deprecated version of `actions/upload-artifact: v3`. Learn more: https://github.blog/changelog/2024-04-16-deprecation-notice-v3-of-the-artifact-actions/
    

    * refactor: added tests for RPC module
    
    * Revert "refactor: added tests for RPC module"
    
    This reverts commit bac5b3b.
    
    * reafctor: added tests for rpc module using httpstest servers
    
    * ci: updated artifact version for binary downloads and uploads
    
    * ci: fixed ci by updateing action versions
    
    * reafctor: replaced ts.cancel() with  t.Cleanup(func() { ts1.Close() })
    …er (#1264)
    
    * fix: switch RPC if block number is less than last recorded block number
    
    * refactor: added tests for block module
    …ocess (#1269)
    
    * refactor: fetched block number from block monitor in voting
    
    * refactor: fixed tests
    
    * fix: added nil check for header from block monitor
    
    * refactor: replaced empty block monitor reference with nil in case of error
    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.

    1 participant