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

[Content Request] Document IPNI / Indexers #1296

Closed
honghaoq opened this issue Sep 29, 2022 · 17 comments · Fixed by #1552
Closed

[Content Request] Document IPNI / Indexers #1296

honghaoq opened this issue Sep 29, 2022 · 17 comments · Fixed by #1552
Assignees
Labels
dif/hard Having worked on the specific codebase is important effort/days Estimated to take multiple days, but less than a week need/maintainers-input Needs input from the current maintainer(s) need/triage Needs initial labeling and prioritization P1 High: Likely tackled by core team if no one steps up

Comments

@honghaoq
Copy link

We are working on Network indexer that index IPFS and Filecoin contents. We should add indexing doc / pointer on the ipfs page here under peer-to-peer-sharing https://docs.ipfs.tech/concepts/#peer-to-peer-sharing.

We can leverage the indexer gitbook to build the content page: docs.cid.contact

@honghaoq honghaoq added the need/triage Needs initial labeling and prioritization label Sep 29, 2022
@welcome

This comment was marked as resolved.

@honghaoq
Copy link
Author

@TMoMoreau fyi

@guseggert guseggert added dif/easy Someone with a little familiarity can pick up good first issue Good issue for new contributors P1 High: Likely tackled by core team if no one steps up and removed need/triage Needs initial labeling and prioritization labels Oct 6, 2022
@jacobheun
Copy link

Perhaps this is a separate issue but documenting reframe would be super useful. There's various resources you can find but querying reframe on the site doesn't yield any results.

https://github.com/ipfs/kubo/releases/tag/v0.16.0 has configuration guidance.
https://blog.ipfs.tech/2022-09-02-introducing-reframe/ has useful information for understanding reframe as well as some curl commands to verify reframe servers.
https://github.com/ipfs-shipyard/someguy is also a useful tool for it, from @aschmahmann -

Generally I'd just use someguy ask --endpoint=https://cid.contact/reframe findprovs <cid> (or put a --pretty before the findprovs keyword) (https://github.com/ipfs-shipyard/someguy).

@lidel
Copy link
Member

lidel commented Nov 17, 2022

"IPNI" (InterPlanetary Network Indexer) should be added to glossary: https://docs.ipfs.tech/concepts/glossary/
Ref. https://github.com/ipni

@TMoMoreau
Copy link
Contributor

I'll get a draft PR made for this. Going to make a list here as well, to verify and keep track of what needs to be done.

@jacobheun I think it would be best to make a separate issue for the reframe documentation topic.

@lidel @honghaoq

@honghaoq
Copy link
Author

Thanks. FYI Torfinn Olsen is gonna be the new indexer PM on this going forward.

@lidel lidel added the status/blocked Unable to be worked further until needs are met label Nov 22, 2022
@lidel
Copy link
Member

lidel commented Nov 22, 2022

I think this is blocked until specs solidify and the implementation of the new API ships with Kubo 0.18.

@lidel lidel removed good first issue Good issue for new contributors dif/easy Someone with a little familiarity can pick up labels Nov 22, 2022
@lidel lidel removed the status/blocked Unable to be worked further until needs are met label Feb 6, 2023
@lidel
Copy link
Member

lidel commented Feb 6, 2023

I guess this is unblocked because we've shipped Kubo 0.18 with default IPNI enabled (https://cid.contact/) by default.

cc @willscott – mind dropping good specs/design docs for docs team to catch up with latest state of things?

@lidel lidel changed the title [Content Request] Add indexing pointer/documentation to IPFS website under peer-to-peer-sharing [Content Request] Document IPNI / Indexers Feb 6, 2023
@willscott
Copy link
Contributor

ipfs/specs#337 is probably the most useful definition for the API that's been shipped

https://github.com/ipni/storetheindex-docs can be useful from the IPNI side

@johnnymatthews johnnymatthews moved this to Needs triage in Protocol Docs Feb 6, 2023
@johnnymatthews johnnymatthews added dif/hard Having worked on the specific codebase is important effort/days Estimated to take multiple days, but less than a week labels Feb 23, 2023
@johnnymatthews johnnymatthews moved this from 👣 Needs triage to 📋 Backlog in Protocol Docs Feb 23, 2023
@ElPaisano
Copy link
Contributor

@lidel can you help the docs team to determine if this is still relevant? https://docs.ipfs.tech/concepts/#peer-to-peer-sharing no longer exists, as it was removed when 2color and I redid the concepts section

@ElPaisano ElPaisano added the need/triage Needs initial labeling and prioritization label Mar 29, 2023
@willscott
Copy link
Contributor

@TorfinnOlsen is probably a good point of contact here.

The IPNI team would like to have docs here, and this remains a valid and effective way to do content routing. Since the use of delegated routing is on by default in current releases of kubo we probably should have documentation about it.

@ElPaisano
Copy link
Contributor

Thanks @willscott

@ElPaisano
Copy link
Contributor

I'm thinking we can add a new page in Concepts -> Subsystems and Components called IPNI that describes this subsystem

@ElPaisano ElPaisano self-assigned this Mar 30, 2023
@TorfinnOlsen
Copy link

@ElPaisano one thing to be aware of is a lot of the language around this specific topic is in a bit of flux right now.

The document @willscott mentioned is the most important starting place however upon reviewing it I submitted a PR to include an important update that might be confusing to readers regarding Reframe which has since v0.19 Kubo release been deprecated.

You should also be aware of the Delegated IPNS HTTP API which intends to delegate naming system functionality over HTTP APIs.

I think from a broader descriptive perspective of the IPNI beyond what you'll find in storetheindex-docs you can use our teams overview blogposts as a valuable reference point on some of the more important points.

How does the network indexer work
How to be an index provider

There are a few more of these that dive deeper into topics involving more how to operate the network indexer and less how it routes content efficiently. If that type of documentation is beneficial for context sake let me know and I'll happily load your plate up with additional links.

@ElPaisano ElPaisano moved this from 📋 Backlog to 👾 Up next in Protocol Docs Apr 3, 2023
@ElPaisano ElPaisano moved this from 👾 Up next to 🏗 In Progress in Protocol Docs Apr 6, 2023
@ElPaisano ElPaisano linked a pull request Apr 6, 2023 that will close this issue
@ElPaisano
Copy link
Contributor

Cheers thanks @TorfinnOlsen

Initial draft is here #1552. This would be good to quickly discuss in a sync call

@ElPaisano ElPaisano moved this from 🏗 In Progress to 👀 In review in Protocol Docs Apr 6, 2023
@ElPaisano
Copy link
Contributor

@TorfinnOlsen can you let me now if you will have a chance to review this week? I figure it might not happen with IPFS thing

@ElPaisano ElPaisano added the need/maintainers-input Needs input from the current maintainer(s) label Apr 18, 2023
@ElPaisano ElPaisano moved this from 👀 In review to 🏗 In Progress in Protocol Docs May 16, 2023
@ElPaisano
Copy link
Contributor

Redraft from @TorfinnOlsen is here https://github.com/TorfinnOlsen/IPNIdocumentation/blob/main/IPFSdocsIPNI.md. Will edit this and add to the PR

@github-project-automation github-project-automation bot moved this from 🏗 In Progress to ✅ Done in Protocol Docs May 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dif/hard Having worked on the specific codebase is important effort/days Estimated to take multiple days, but less than a week need/maintainers-input Needs input from the current maintainer(s) need/triage Needs initial labeling and prioritization P1 High: Likely tackled by core team if no one steps up
Projects
None yet
Development

Successfully merging a pull request may close this issue.

9 participants