Skip to content

Provide roots only strategy (rough)#6472

Closed
michaelavila wants to merge 1 commit intomasterfrom
features/provide-roots-only-strategy-rough
Closed

Provide roots only strategy (rough)#6472
michaelavila wants to merge 1 commit intomasterfrom
features/provide-roots-only-strategy-rough

Conversation

@michaelavila
Copy link
Contributor

@michaelavila michaelavila commented Jun 27, 2019

Requires: ipfs/go-ipfs-config#38

This is a quick and straightforward change to (1) disable bitswap providing, and (2) enable providing of roots in the new providing system. This does not make the codebase better and will be factored differently in time.

The following configs are needed:

ipfs config Experimental.StrategicProviding true
ipfs config Provider.Strategy "roots"
# restart ipfs

Here "roots" is defined by the root hashes created during ipfs add and ipfs pin add commands.

@Stebalien Stebalien force-pushed the features/provide-roots-only-strategy-rough branch from 89b752e to face434 Compare July 9, 2019 22:41
Copy link
Member

@Stebalien Stebalien left a comment

Choose a reason for hiding this comment

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

I've gone ahead and merged the config changes anyways as that likely won't change.

However, let's make sure to update the config documentation while we're at it.

@Stebalien Stebalien changed the base branch from feat/stabilize-dht to master July 9, 2019 22:42
@michaelavila
Copy link
Contributor Author

I'm closing this as it was just an option for the recent dht push. Ultimately, provide roots will not use the old reprovider system as is the case in this PR. Thanks for taking a look.

@michaelavila michaelavila deleted the features/provide-roots-only-strategy-rough branch July 11, 2019 18:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants