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

chore: update ipni components #36

Merged
merged 8 commits into from
Sep 15, 2023
Merged

chore: update ipni components #36

merged 8 commits into from
Sep 15, 2023

Conversation

rvagg
Copy link
Member

@rvagg rvagg commented Sep 6, 2023

No description provided.

@rvagg
Copy link
Member Author

rvagg commented Sep 12, 2023

Adds integration tests for IPNI! Which is really exciting, it's been the piece I've felt most uncertain about and had the most breakage with. I can now exercise various scenarios and new features without too much fear of breakage.

Along with integration tests I've added --announce-url (helpful to test internally rather than against cid.contact!) and --ipni-path which now defaults to /ipni/, so requests via /ipni/v1/ad/ is the default.

Depends on ipni/storetheindex#2242.


// install the indexer to announce to
indexer := filepath.Join(tr.Dir, "storetheindex")
tr.Run("go", "install", "github.com/ipni/storetheindex@e56485343dd8e235581191b4668b5bfc0cea0781") // TODO: use @latest when we have a release
Copy link
Member Author

Choose a reason for hiding this comment

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

TODO before merge

@rvagg rvagg force-pushed the rvagg/ipni-upd branch 4 times, most recently from edb85d5 to b26059d Compare September 12, 2023 11:30
@rvagg
Copy link
Member Author

rvagg commented Sep 13, 2023

Added lassie to the flow so it's a proper e2e now

  1. Start local indexer
  2. Make multiple CARs with unixfs data in them
  3. Serve it with Frisbii and tell it to announce to indexer
  4. Do a lassie fetch <root> for each of the unixfs blobs we're serving, only telling it where the indexer endpoint is so it has to work out how to get to Frisbii from that

@rvagg rvagg merged commit fac8891 into master Sep 15, 2023
@rvagg rvagg deleted the rvagg/ipni-upd branch September 15, 2023 00:25
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.

1 participant