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

Migration for CQT network v1.5 (Staking on Ethereum) #178

Merged
merged 10 commits into from
Jan 16, 2024

Conversation

noslav
Copy link
Member

@noslav noslav commented Jan 5, 2024

This only applies to testing on moonbase proofchain for the current iteration (pre covenet/ ethereum-wayback-machine)

Signed-off-by: Pranay Valson <pranay.valson@gmail.com>
Signed-off-by: Pranay Valson <pranay.valson@gmail.com>
Signed-off-by: Pranay Valson <pranay.valson@gmail.com>
Copy link

codecov bot commented Jan 9, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (86e01d9) 36.34% compared to head (65a1718) 39.17%.
Report is 5 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #178      +/-   ##
===========================================
+ Coverage    36.34%   39.17%   +2.82%     
===========================================
  Files           26       25       -1     
  Lines          886      822      -64     
===========================================
  Hits           322      322              
+ Misses         564      500      -64     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Signed-off-by: Pranay Valson <pranay.valson@gmail.com>
@noslav noslav force-pushed the feature/migration-v1.5-mbase branch from d622752 to 5007fd9 Compare January 9, 2024 23:56
@noslav
Copy link
Member Author

noslav commented Jan 10, 2024

Jan 10 00:28:58 : rudder       | [info] found 1 bsps to process
Jan 10 00:28:58 : rudder       | [info] processing specimen 1_18972940_46a40a368d18f3578a3799c3abc7e5605d068b5f2de62b1f2704ee73aa54c9d8_22c18b3e1463389e817de05b338c3741273c69334f2181a19a6d84b208a9e881
Jan 10 00:28:58 : ipfs-pinner  | 2024/01/10 00:28:58 unixfsApi.Get: getting the cid: bafybeihnbc2tq6qrj7yx4q6vueinvqmyqotl6td4yrfq4l5zq3hfcwftlq
Jan 10 00:28:58 : ipfs-pinner  | 2024/01/10 00:28:58 trying out https://w3s.link/ipfs/bafybeihnbc2tq6qrj7yx4q6vueinvqmyqotl6td4yrfq4l5zq3hfcwftlq
Jan 10 00:28:58 : rudder       | [info] synced to latest; waiting for 5862592 to be mined
Jan 10 00:29:03 : ipfs-pinner  | 2024/01/10 00:29:03 got the content!
Jan 10 00:29:03 : rudder       | [info] Counter for ipfs_metrics - [fetch: 2]
Jan 10 00:29:03 : rudder       | [info] LastValue for ipfs_metrics - [fetch_last_exec_time: 0.004628]
Jan 10 00:29:03 : rudder       | [info] Counter for bsp_metrics - [decode: 2]
Jan 10 00:29:03 : rudder       | [info] LastValue for bsp_metrics - [decode_last_exec_time: 0.0]
Jan 10 00:29:03 : rudder       | [info] submitting 18972940 to evm http server...
Jan 10 00:29:03 : evm-server   | [INFO] [01-10|00:29:03.229] input file at                            loc=/tmp/4125641747
Jan 10 00:29:03 : evm-server   | [INFO] [01-10|00:29:03.238] output file at:                          loc=/tmp/2526281305
Jan 10 00:29:03 : evm-server   | [INFO] [01-10|00:29:03.599] Wrote file                               file=/tmp/2526281305
Jan 10 00:29:03 : evm-server   |
Jan 10 00:29:03 : rudder       | [info] writing block result into "/tmp/briefly-1704/briefly-576460234861684398-58eIxtZS8R4t0vBSwSp"
Jan 10 00:29:03 : rudder       | [info] Counter for bsp_metrics - [execute: 2]
Jan 10 00:29:03 : rudder       | [info] LastValue for bsp_metrics - [execute_last_exec_time: 4.39e-4]
Jan 10 00:29:03 : ipfs-pinner  | 2024/01/10 00:29:03 generated dag has root cid: bafybeig5qkjjqiki4rksiqjizyunc32t2kgeabh34jowv7dkol4en4qpky
Jan 10 00:29:03 : ipfs-pinner  | 2024/01/10 00:29:03 car file location: /tmp/1157022261.car
Jan 10 00:29:05 : ipfs-pinner  | 2024/01/10 00:29:05 Received /health request: source= 127.0.0.1:59566 status= OK
Jan 10 00:29:05 : evm-server   | [INFO] [01-10|00:29:05.304] Received /health request:                source==127.0.0.1:38802 status==OK
Jan 10 00:29:07 : ipfs-pinner  | 2024/01/10 00:29:07 w3 up output: {"root":{"/":"bafybeig5qkjjqiki4rksiqjizyunc32t2kgeabh34jowv7dkol4en4qpky"}}
Jan 10 00:29:07 : ipfs-pinner  | 2024/01/10 00:29:07 uploaded file has root cid: bafybeig5qkjjqiki4rksiqjizyunc32t2kgeabh34jowv7dkol4en4qpky
Jan 10 00:29:07 : rudder       | [info] Counter for ipfs_metrics - [pin: 2]
Jan 10 00:29:07 : rudder       | [info] LastValue for ipfs_metrics - [pin_last_exec_time: 0.004251]
Jan 10 00:29:07 : rudder       | [info] 18972940:22c18b3e1463389e817de05b338c3741273c69334f2181a19a6d84b208a9e881 has been successfully uploaded at ipfs://bafybeig5qkjjqiki4rksiqjizyunc32t2kgeabh34jowv7dkol4en4qpky
Jan 10 00:29:07 : rudder       | [info] 18972940:22c18b3e1463389e817de05b338c3741273c69334f2181a19a6d84b208a9e881 proof submitting
Jan 10 00:29:08 : rudder       | [info] Counter for brp_metrics - [proof: 1]
Jan 10 00:29:08 : rudder       | [info] LastValue for brp_metrics - [proof_last_exec_time: 4.1999999999999996e-4]
Jan 10 00:29:08 : rudder       | [info] 18972940 txid is 0x8c745900cac8b8d5588613f6ee5bccfffb62d05b85b5a15fafa5d48ea2e5c0e0
Jan 10 00:29:08 : rudder       | [info] Counter for brp_metrics - [upload_success: 1]
Jan 10 00:29:08 : rudder       | [info] LastValue for brp_metrics - [upload_success_last_exec_time: 0.004691]
Jan 10 00:29:08 : rudder       | [info] Counter for rudder_metrics - [pipeline_success: 1]
Jan 10 00:29:08 : rudder       | [info] LastValue for rudder_metrics - [pipeline_success_last_exec_time: 0.009873]

The above changes have made rudder compatible to v1.5 cqt network however we will need to update this branch again soon since there are a few issues on the contracts. Currently on the bsp proofchain - "finalize result sessions" needs to be called instead of "finalize specimen session" and topic hash signatures blockSpecimenHash and resultHash should be called blockHash and specimenHash respectively. Once we update this the topic hashes would need to be updated and so would the finalizer/bridge-agent

Signed-off-by: Pranay Valson <pranay.valson@gmail.com>
Signed-off-by: Pranay Valson <pranay.valson@gmail.com>
Signed-off-by: Pranay Valson <pranay.valson@gmail.com>
Signed-off-by: Pranay Valson <pranay.valson@gmail.com>
Fix ci for w3p upload

Signed-off-by: Pranay Valson <pranay.valson@gmail.com>
@noslav noslav force-pushed the feature/migration-v1.5-mbase branch from 961bb55 to fbedbde Compare January 11, 2024 21:56
@noslav
Copy link
Member Author

noslav commented Jan 11, 2024

With the above commits CI testing and uploading using ipfs-pinner w3up method is fixed. The resolution needed converting the needed proof.out file into a hex txt representation using the xxd tool such that xxd -p proof.out > proof_out_hex.txt copying this .txt file over to an env var for GitHub actions with PROOF_OUT_HEX, importing this env var within ipfs-pinner during the docker run and converting it back to its binary format using xxd -r -p proof_out_hex.txt > proof_from_hex.out so that pinner can now use the binary file as the w3 delegation file. The W3_DELEGATION_FILE and W3_AGENT_KEY also needed to be set for GitHub actions. At no point are the secrets exposed in the CI run itself.

rudder        | Finished in 16.7 seconds (16.0s async, 0.6s sync)
[1590](https://github.com/covalenthq/rudder/actions/runs/7494994524/job/20404149892?pr=178#step:7:1591)
rudder        | 35 tests, 0 failures

Signed-off-by: Pranay Valson <pranay.valson@gmail.com>
@noslav noslav force-pushed the feature/migration-v1.5-mbase branch from 4b32de0 to 65a1718 Compare January 11, 2024 22:16
@noslav noslav mentioned this pull request Jan 11, 2024
@noslav noslav marked this pull request as draft January 16, 2024 20:18
@noslav noslav marked this pull request as ready for review January 16, 2024 21:45
@noslav noslav merged commit 42a919b into develop Jan 16, 2024
5 checks passed
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

1 participant