Closed wdbaruni closed 3 weeks ago
[!IMPORTANT]
Review skipped
Auto reviews are disabled on this repository.
Please check the settings in the CodeRabbit UI or the
.coderabbit.yaml
file in this repository. To trigger a single review, invoke the@coderabbitai review
command.You can disable this status message by setting the
reviews.review_status
tofalse
in the CodeRabbit configuration file.
:man_facepalming: well there goes auto-merge. Might need to address several of these comments in a follow on.
This PR fully removes embedded IPFS from our code base, including tests where we are now using
local
publisher andlocalDirectory
storage source instead.Testing
IPFS related tests are skipped unless IPFS connect endpoint is provided, such as through
BACALHAU_NODE_IPFS_CONNECT
environment variable.These tests are running in circleci test workflow for liux-amd64-integration where we also run an IPFS daemon in that environment and set IPFS connect configuration.
Other Changes
Few other issues were surfaced when switching to local publisher that had to be fixed, including: