Magickbase / neuron-public-issues

Neuron Issues
6 stars 3 forks source link

Neuron does not show `Looking for assumed valid target` tip #398

Open yanguoyu opened 3 months ago

yanguoyu commented 3 months ago

The ckb's find the assumed valid target log has changed. So Neuron can not match it. Before 0.112 https://github.com/nervosnetwork/ckb/blob/v0.112.1/sync/src/synchronizer/mod.rs#L132-L136

best known header number: {}, hash: {:#?}, \
                                 can't find assume valid target temporarily, hash: {:#?} \
                                 please wait

0.113~0.115 https://github.com/nervosnetwork/ckb/blob/v0.114.0/sync/src/synchronizer/mod.rs#L126-L129

best known header number: {}, hash: {:#?}, \
                                 temporarily can't find assume valid target, hash: {:#?} \
                                 Please wait

And 0.115 https://github.com/nervosnetwork/ckb/blob/v0.117.0-rc1/sync/src/synchronizer/mod.rs#L129-L132

"best known header {}-{}, \
                                 CKB is syncing to latest Header to find the assume valid target: {}. \
                                 Please wait. {}",

The log may change for other unknown reasons in the future, and we can not monitor the log's change. So we'd better research another way to match that ckb is looking for assumed valid target.

Danie0918 commented 3 months ago
  1. Fixing neuron issues with hotfix.
  2. Create issue feedback in the ckbcore project.
yanguoyu commented 3 months ago

https://github.com/nervosnetwork/ckb/issues/4485

yanguoyu commented 3 months ago

https://github.com/nervosnetwork/neuron/pull/3195

yanguoyu commented 1 month ago

https://github.com/nervosnetwork/ckb/pull/4486 has merged, we can reopen this issue by using RPC to check the status. @Danie0918

yanguoyu commented 1 month ago

I guess it's better to use RPC result after https://github.com/ckb-js/lumos/issues/733 lumos adapt 0.117 RPC method.