application-research / autoretrieve

A server to make GraphSync data accessible on IPFS
22 stars 7 forks source link

feat: update lassie to sync Retriever (after provider rewrite revert) #173

Closed rvagg closed 1 year ago

rvagg commented 1 year ago

This is #167 but against current master, i.e. with the provider rewrite reverted.

It's using https://github.com/filecoin-project/lassie/pull/54 as the "latest" lassie, because that's the newest incarnation of the interface - taking a linkSystem per retrieval.

Unfortunately this won't build because there's a dependency problem involving the infamous btcd breaking change .. and I don't have the patience today to figure out which bit broke that, maybe the go-data-transfer upgrade? It was working with in #167 with a version of lassie that's not too far off this one.


codecov-commenter commented 1 year ago

Codecov Report

Base: 5.04% // Head: 5.44% // Increases project coverage by +0.40% :tada:

Coverage data is based on head (03956e7) compared to base (c0de64f). Patch coverage: 14.94% of modified lines in pull request are covered.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #173 +/- ## ========================================= + Coverage 5.04% 5.44% +0.40% ========================================= Files 15 14 -1 Lines 1666 1635 -31 ========================================= + Hits 84 89 +5 + Misses 1577 1541 -36 Partials 5 5 ``` | [Impacted Files](https://codecov.io/gh/application-research/autoretrieve/pull/173?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=application-research) | Coverage Δ | | |---|---|---| | [autoretrieve.go](https://codecov.io/gh/application-research/autoretrieve/pull/173?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=application-research#diff-YXV0b3JldHJpZXZlLmdv) | `0.00% <0.00%> (ø)` | | | [bitswap/provider.go](https://codecov.io/gh/application-research/autoretrieve/pull/173?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=application-research#diff-Yml0c3dhcC9wcm92aWRlci5nbw==) | `0.00% <0.00%> (ø)` | | | [endpoint/estuary.go](https://codecov.io/gh/application-research/autoretrieve/pull/173?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=application-research#diff-ZW5kcG9pbnQvZXN0dWFyeS5nbw==) | `0.00% <0.00%> (ø)` | | | [blocks/manager.go](https://codecov.io/gh/application-research/autoretrieve/pull/173?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=application-research#diff-YmxvY2tzL21hbmFnZXIuZ28=) | `78.50% <92.85%> (+1.05%)` | :arrow_up: | Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=application-research). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=application-research)

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.