Skip to content
This repository has been archived by the owner on Aug 1, 2023. It is now read-only.

test: adds tests for adding and modifying mfs hamt shards #54

Merged
merged 4 commits into from
May 24, 2019

Conversation

achingbrain
Copy link
Member

Requires ipfs-inactive/js-ipfs-mfs#46 to be released.

I couldn't find a way to enable sharding for go and js without putting some conditionals in the daemon config - have I missed something?

@achingbrain achingbrain requested a review from alanshaw March 12, 2019 12:21
@ghost ghost assigned achingbrain Mar 12, 2019
@ghost ghost added the status/in-progress In progress label Mar 12, 2019
Copy link
Member

@alanshaw alanshaw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@alanshaw
Copy link
Member

Any chance you can rebase this and get some 💚 CI @achingbrain?

@achingbrain achingbrain force-pushed the add-tests-for-modifying-shards branch 2 times, most recently from c9fcac0 to 0f1c506 Compare April 15, 2019 12:50
@achingbrain
Copy link
Member Author

I'm seeing some seemingly unrelated errors in CI:

No available transports to dial peer QmNWQHexj877kvgjoH1udyNmxEDcVyuxHBW5PhkzpF6c3Y!

@achingbrain achingbrain force-pushed the add-tests-for-modifying-shards branch from 0f1c506 to f863b58 Compare May 23, 2019 22:50
@achingbrain
Copy link
Member Author

Will probably need a rebase after #68 goes in

@achingbrain
Copy link
Member Author

It's mostly green?

@alanshaw
Copy link
Member

Merging as test failures are unrelated.

@alanshaw alanshaw merged commit 184e103 into master May 24, 2019
@alanshaw alanshaw deleted the add-tests-for-modifying-shards branch May 24, 2019 16:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants