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

venus-market订单状态异常 #4943

Closed
8 of 20 tasks
Tracked by #4972
caijian76 opened this issue Jun 17, 2022 · 8 comments
Closed
8 of 20 tasks
Tracked by #4972

venus-market订单状态异常 #4943

caijian76 opened this issue Jun 17, 2022 · 8 comments
Assignees
Labels
C-Blocked Category: temporarily stuck without a good solution C-bug Category: This is a bug CU-community Category: from community P3 Low - not important right now V-venus-market venus-market component

Comments

@caijian76
Copy link

Checklist

  • This is not a security-related bug/issue. If it is, please follow please follow the security policy.
  • This is not a question or a support request. If you have any lotus related questions, please ask in the venus forum.
  • This is not a new feature request. If it is, please file a feature request instead.
  • This is not an enhancement request. If it is, please file a improvement suggestion instead.
  • I have searched on the issue tracker and the venus forum, and there is no existing related issue or discussion.
  • I am running the latest release, or the most recent RC(release canadiate) for the upcoming release or the dev branch(master), or have an issue updating to any of these.
  • I did not make any code changes to venus.

Venus component

  • venus daemon - chain sync
  • venus auth - authentication
  • venus gateway - chain service gateway
  • venus miner - mining and block production
  • venus sealer/worker - sealing
  • venus sealer - proving(WindowPoSt)
  • venus market - storage deal
  • venus market - retrieval deal
  • venus market - data transfer
  • venus light-weight client
  • venus JSON-RPC API
  • venus messager - message management (mpool)
  • Other

Venus Version

venus-market version venus-market v2.0.0

Describe the Bug

封装完订单过了1天后(snapup方法),storage-deals list查看订单状态,依旧是Proving ,而不是Assigned ,用market-client查看订单状态,是StorageDealAwaitingPreCommit,而不是StorageDealActive 。我确认订单已经上链了。

Logging Information

venus-market storage-deals list
...2stxykae  7242221  StorageDealActive     Proving     t3wzid6h2dv5osdmvvvcyxezgupytsiqgdwxcmucr7yclylogw36xrma23qc57ogi65udaylimt7bmbpdr2daa  t01212159  64GiB  0 FIL  866397
...i64g5rou  7253020  StorageDealActive     Proving     t3wzid6h2dv5osdmvvvcyxezgupytsiqgdwxcmucr7yclylogw36xrma23qc57ogi65udaylimt7bmbpdr2daa  t01212159  64GiB  0 FIL  1027197


root@work10:~# market-client  storage deals list
DealCid      DealId   Provider   State                         On Chain?          Slashed?  PieceCID     Size       Price  Duration  Verified  
...cliw5ohm  4732745  t01212159  StorageDealActive             Y (epoch 1654059)  N         ...jgqgn6fa  63.5 GiB   0 FIL  1527581   true      
...4kgh5yxy  5094019  t01212159  StorageDealActive             Y (epoch 1682093)  N         ...6brmwmhy  31.75 GiB  0 FIL  1442838   true      
...2stxykae  7242221  t01212159  StorageDealAwaitingPreCommit  Y (epoch 1902527)  N         ...5ri7c2da  63.5 GiB   0 FIL  866397    true      
...i64g5rou  7253020  t01212159  StorageDealAwaitingPreCommit  Y (epoch 1902774)  N         ...zxvxh6oy  63.5 GiB   0 FIL  1027197   true

Repo Steps

  1. Run '...'
  2. Do '...'
  3. See error '...'
    ...
@zl03jsj
Copy link
Contributor

zl03jsj commented Jun 17, 2022

先简单回复一下: @caijian76
关于PieceState部分:

封装完订单过了1天后(snapup方法),storage-deals list查看订单状态,依旧是Proving ,而不是Assigned

Proving 是订单的piecestate的一种状态, piecestate的顺序依次为:

  1. Undefine: 还未关联到任何Sector
  2. Assigned: 已经被选中到某个Sector
  3. Packing: Sector正在add package中.
  4. Proving: 已经封装完成.

所以, 看到的这个状态Proving状态是正确的.

至于后面market-client查看订单状态, 如果确认已经封装完成, 此时的状态不应该为StorageDealAwaitingPreCommit, 后续我们会对此进行确认.

@caijian76
Copy link
Author

caijian76 commented Jun 18, 2022

我是基于原来采用venus-sealer在线订单封装(非snapup模式),和现在使用snapup模式封装的扇区比较,以为完成的正常状态是Assigned。

root@work10:/mnt/data/tmpdir/64G# venus-market storage-deals list
/root/.venusmarket
ProposalCid DealId State PieceState Client Provider Size Price Duration
...mwodo72e 0 StorageDealRejecting t3wzid6h2dv5osdmvvvcyxezgupytsiqgdwxcmucr7yclylogw36xrma23qc57ogi65udaylimt7bmbpdr2daa t01212159 64GiB 0 FIL 1556394
...cliw5ohm 4732745 StorageDealActive Assigned t3wzid6h2dv5osdmvvvcyxezgupytsiqgdwxcmucr7yclylogw36xrma23qc57ogi65udaylimt7bmbpdr2daa t01212159 64GiB 0 FIL 1527581
...ektskskm 0 StorageDealRejecting t3wzid6h2dv5osdmvvvcyxezgupytsiqgdwxcmucr7yclylogw36xrma23qc57ogi65udaylimt7bmbpdr2daa t01212159 32GiB 0 FIL 1558042
...4kgh5yxy 5094019 StorageDealActive Assigned t3wzid6h2dv5osdmvvvcyxezgupytsiqgdwxcmucr7yclylogw36xrma23qc57ogi65udaylimt7bmbpdr2daa t01212159 32GiB 0 FIL 1442838
...2stxykae 7242221 StorageDealActive Proving t3wzid6h2dv5osdmvvvcyxezgupytsiqgdwxcmucr7yclylogw36xrma23qc57ogi65udaylimt7bmbpdr2daa t01212159 64GiB 0 FIL 866397
...i64g5rou 7253020 StorageDealActive Proving t3wzid6h2dv5osdmvvvcyxezgupytsiqgdwxcmucr7yclylogw36xrma23qc57ogi65udaylimt7bmbpdr2daa t01212159 64GiB 0 FIL 1027197

root@work10:/mnt/data/tmpdir/64G# market-client storage deals list
DealCid DealId Provider State On Chain? Slashed? PieceCID Size Price Duration Verified
...cliw5ohm 4732745 t01212159 StorageDealActive Y (epoch 1654059) N ...jgqgn6fa 63.5 GiB 0 FIL 1527581 true
...4kgh5yxy 5094019 t01212159 StorageDealActive Y (epoch 1682093) N ...6brmwmhy 31.75 GiB 0 FIL 1442838 true
...2stxykae 7242221 t01212159 StorageDealAwaitingPreCommit Y (epoch 1902527) N ...5ri7c2da 63.5 GiB 0 FIL 866397 true
...i64g5rou 7253020 t01212159 StorageDealAwaitingPreCommit Y (epoch 1902774) N ...zxvxh6oy 63.5 GiB 0 FIL 1027197 true

4732745和5094019 订单状态在client看起来是对的,在market上的状态为Assigned

@Fatman13
Copy link
Contributor

proving意思应该是这些扇区需要每天做windowPost去证明他的意思。所以是proving。

@Fatman13 Fatman13 added C-bug Category: This is a bug CU-community Category: from community labels Jun 20, 2022
@hunjixin hunjixin added the P2 Medium - we should get to this soon label Jun 22, 2022
@diwufeiwen diwufeiwen mentioned this issue Jun 27, 2022
40 tasks
@zl03jsj zl03jsj added the V-venus-market venus-market component label Jun 27, 2022
@LinZexiao LinZexiao self-assigned this Jul 4, 2022
@LinZexiao LinZexiao moved this to In Progress in Venus Project Jul 8, 2022
@LinZexiao
Copy link
Collaborator

market client 在订单 snapup 的过程中 确实会出现不能正确识别订单状态的情况,原因是在client状态机中,尚需增加对snapup 的支持(此前是通过识别 precommit 的信息来进行状态跳转。

@LinZexiao LinZexiao removed the status in Venus Project Jul 15, 2022
@LinZexiao LinZexiao removed their assignment Jul 21, 2022
@zl03jsj zl03jsj assigned zl03jsj and unassigned zl03jsj Jul 21, 2022
@zl03jsj zl03jsj moved this to In Progress in Venus Project Jul 21, 2022
@zl03jsj zl03jsj self-assigned this Jul 21, 2022
@zl03jsj zl03jsj mentioned this issue Jul 22, 2022
27 tasks
@hunjixin
Copy link
Contributor

@hunjixin hunjixin added P3 Low - not important right now and removed P2 Medium - we should get to this soon labels Jul 25, 2022
@zl03jsj zl03jsj added the C-Blocked Category: temporarily stuck without a good solution label Jul 26, 2022
@diwufeiwen diwufeiwen removed the status in Venus Project Aug 19, 2022
@Fatman13 Fatman13 added the C-triage Category: Need triage label Aug 22, 2022
@Fatman13 Fatman13 removed the C-triage Category: Need triage label Sep 2, 2022
@zl03jsj
Copy link
Contributor

zl03jsj commented Oct 11, 2022

close as not planned

@zl03jsj zl03jsj closed this as completed Oct 11, 2022
@zl03jsj zl03jsj moved this to Done in Venus Project Oct 11, 2022
@Fatman13
Copy link
Contributor

没有实现的话,用“close as not planned”。[doge]

@Fatman13 Fatman13 closed this as not planned Won't fix, can't repro, duplicate, stale Oct 11, 2022
@zl03jsj
Copy link
Contributor

zl03jsj commented Oct 11, 2022

没有实现的话,用“close as not planned”。[doge]

噢啦.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Blocked Category: temporarily stuck without a good solution C-bug Category: This is a bug CU-community Category: from community P3 Low - not important right now V-venus-market venus-market component
Projects
Archived in project
Development

No branches or pull requests

5 participants