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

The file size transferred during the deal is much larger than the actual file size. #5133

Closed
chaopengliu opened this issue Dec 7, 2020 · 4 comments · Fixed by #5853
Closed
Assignees
Labels
area/data-transfer area/ux Area: UX team/ignite Issues and PRs being tracked by Team Ignite at Protocol Labs

Comments

@chaopengliu
Copy link

chaopengliu commented Dec 7, 2020

Describe the bug
The file size transferred during the deal is much larger than the actual file size.

image

louts  client list-deals -v 
root@10-8-78-110:/data/deal_scripts# lc list-deals -v
Created          DealCid                                                      DealId   Provider  State                             On Chain?  Slashed?  PieceCID                                                          Size       Price              Duration  Verified  Message
Dec  8 22:16:12  bafyreifwbnnu7tzqccfrmevrwcxnx6ipoctwnzjtefymtiujimvjx7gtym  1436539  f085710   StorageDealAwaitingPreCommit      N          N         baga6ea4seaqicfrhqddzqb33kly4otpymvzfrviiiybe75aibpk5ud2xsoqpmey  1.984 GiB  0.000520669 FIL    520669    false
Dec  8 23:25:15  bafyreieo3v6k3qh7p3iig64kbsc5nfupoblfktnthslarhmdq45jhzwjaq  0        f023868   StorageDealCheckForAcceptance     N          N         baga6ea4seaqicfrhqddzqb33kly4otpymvzfrviiiybe75aibpk5ud2xsoqpmey  1.984 GiB  0.00001041416 FIL  520708    false     Provider state: StorageDealUnknown
Dec  9 10:46:25  bafyreifwgfmm2aq544cgmjjdcwnfqhjvcnmedcdjpogsjvjwzkikblroky  0        f034545   StorageDealClientTransferRestart  N          N         baga6ea4seaqicfrhqddzqb33kly4otpymvzfrviiiybe75aibpk5ud2xsoqpmey  1.984 GiB  0 FIL              522043    false
Dec  9 11:58:56  bafyreibhfr2pah3ha26yddngdgv22o4aktafq2ucghizfol6nqqe66a6pu  1439889  f044160   StorageDealAwaitingPreCommit      N          N         baga6ea4seaqicfrhqddzqb33kly4otpymvzfrviiiybe75aibpk5ud2xsoqpmey  1.984 GiB  0.000522358 FIL    522358    false
Dec  9 14:04:19  bafyreiekxxbplgm26xd6onihe3wk7wgrljrzoqjk3foike2na5w2pejto4  1438964  f063869   StorageDealAwaitingPreCommit      N          N         baga6ea4seaqdvheocjffe6nppls2i7yxfvxr52em43xg3drr4i4iknxbxxn2ugq  1.984 GiB  0 FIL              521264    false
Dec  9 17:56:25  bafyreibvi7hjqmtwlb7h35mkkbmvpe2zuntv5n3q72c2nyoddiysq4ffjy  0        f010493   StorageDealCheckForAcceptance     N          N         baga6ea4seaqdvheocjffe6nppls2i7yxfvxr52em43xg3drr4i4iknxbxxn2ugq  1.984 GiB  0.000521296 FIL    521296    false     Provider state: StorageDealReserveProviderFunds
@chaopengliu
Copy link
Author

OS:root@10-8-78-110:/data/deal_scripts# cat /proc/version
Linux version 5.4.0-53-generic (buildd@lcy01-amd64-007) (gcc version 9.3.0 (Ubuntu 9.3.0-17ubuntu1~20.04)) #59-Ubuntu SMP Wed Oct 21 09:38:44 UTC 2020

Please let me know if you need to provide any other information.

@dirkmc
Copy link
Contributor

dirkmc commented Dec 10, 2020

Yesterday a PR was merged into master so that lotus client list-deals -v will include the transfer channel ID, which you can cross-reference with lotus client list-transfers. To use it you will need to build lotus from source or wait till the next lotus release. This should make it easier to see which transfers apply to which deals.

Please let us know if you still see a discrepancy when comparing list-deals to list-transfers for a particular deal/transfer.

Note that list-transfers shows the padded size of the data transferred, whereas list-data shows the original (unpadded) size of the data.
https://docs.filecoin.io/store/lotus/store-data/#importing-data

@dineshshenoy dineshshenoy added this to the 💹Storage Deal Success milestone Dec 17, 2020
@dirkmc
Copy link
Contributor

dirkmc commented Jan 5, 2021

May have the same underlying cause as #5225

@dineshshenoy dineshshenoy added the P2 P2: Should be resolved label Jan 5, 2021
@dineshshenoy dineshshenoy removed this from the 💹Storage Deal Success milestone Jan 5, 2021
@dineshshenoy dineshshenoy added area/ux Area: UX and removed P2 P2: Should be resolved labels Jan 5, 2021
@dineshshenoy dineshshenoy added this to the 💹Storage Deal Success milestone Jan 18, 2021
@raulk raulk added team/ignite Issues and PRs being tracked by Team Ignite at Protocol Labs and removed team/ignite Issues and PRs being tracked by Team Ignite at Protocol Labs labels Feb 23, 2021
@dirkmc
Copy link
Contributor

dirkmc commented Feb 24, 2021

Resolved by filecoin-project/go-data-transfer#140 (once it's been bubbled up to lotus)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/data-transfer area/ux Area: UX team/ignite Issues and PRs being tracked by Team Ignite at Protocol Labs
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants