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.

Sign up for GitHub

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

Jump to bottom

kad: Providers part 8: unit, e2e, and libp2p conformance tests #258

Merged
merged 60 commits into from
Sep 30, 2024
Merged

kad: Providers part 8: unit, e2e, and libp2p conformance tests #258

merged 60 commits into from
Sep 30, 2024

Conversation

dmitry-markin
Copy link
Collaborator

This PR concludes the series of content providers PRs with introduction of the tests:

  1. Unit tests in Kademlia
  2. E2E tests with litep2p connecting to litep2p.
  3. Conformance tests with itep2p connecting to libp2p and vice versa.

Builds upon #246.

Closes #196, #201, #202.

@dmitry-markin
Introduce ADD_PROVIDER query
1048320
@dmitry-markin
Merge remote-tracking branch 'origin/master' into dm-provider-queries
2ab84ae
@dmitry-markin
Execute ADD_PROVIDER query
778078b
@dmitry-markin
Introduce local providers in MemoryStore
323a26f
@dmitry-markin
Add provider refresh interval to Kademlia config
8946809
@dmitry-markin
Merge remote-tracking branch 'origin/master' into dm-republish-providers
474a58a
@dmitry-markin
Move FuturesStream to a separate file
d02acee
@dmitry-markin
Refresh providers: dry-run without network queries
5432c9b
@dmitry-markin
Remove try_get_record() and other try_...() non-async methods
feb493d
@dmitry-markin
Move query ID generation from KademliaHandle to Kademlia
982c73d
@dmitry-markin
Republish providers
68c7a87
@dmitry-markin
Merge remote-tracking branch 'origin/master' into dm-republish-providers
f9ed3e7
@dmitry-markin
Merge remote-tracking branch 'origin/master' into dm-add-providers
764a3f2
@dmitry-markin
Use getter TransportService::local_peer_id() instead of accessing d…
5e97484 
…irectly
@dmitry-markin
Use getter TransportService::local_peer_id() instead of accessing d…
7caf290 
…irectly
@dmitry-markin
Introduce GET_PROVIDERS query
60b8fb6
@dmitry-markin
Update get_providers() to receive query ID via oneshot channel
d340c35
@dmitry-markin
Make lines fit into 100 characters
10b96db
@dmitry-markin
Introduce GetProvidersContext & GetProvidersConfig
6146540
@dmitry-markin
Implement GET_PROVIDERS query
e167419
@dmitry-markin
Merge returned provider addresses and deduplicate records
eb6eee8
@dmitry-markin
Merge branch 'dm-add-providers' into dm-republish-providers
8314d95
@dmitry-markin
minor: fix log target
974eac5
@dmitry-markin
Revert "Remove try_get_record() and other try_...() non-async met…
e895a44 
…hods"

This reverts commit feb493d.
@dmitry-markin
Revert "Move query ID generation from KademliaHandle to Kademlia"
ce18594 
This reverts commit 982c73d.
@dmitry-markin
Use AtomicUsize to generate QueryId in both KademliaHandle and …
0fcd621 
…`Kademlia`
@dmitry-markin
Merge branch 'dm-republish-providers' into dm-get-providers
5f26161
@dmitry-markin
Use open_substream_or_dial() to add provider records to peers
57f17ce
@dmitry-markin
Merge remote-tracking branch 'origin/master' into dm-add-providers
8594103
@dmitry-markin
Test local provider refresh scheduled in MemoryStore
a65ff10
@dmitry-markin
Extend local providers tests in MemoryStore
dc11ff1
@dmitry-markin
Test GetProvidersContext
6ebb51b
@dmitry-markin
Introduce ContentProvider type
17fd943
@dmitry-markin
Merge branch 'dm-add-providers' into dm-republish-providers
e853d3b
@dmitry-markin
Merge branch 'dm-republish-providers' into dm-get-providers
aeb59cf
@dmitry-markin
Merge branch 'dm-get-providers' into dm-stop-providing
674c54e
@dmitry-markin
Merge branch 'dm-stop-providing' into dm-improve-providers-api
98523e7
@dmitry-markin
Use PublicAddresses API
d32bd56
@dmitry-markin
Use locally known providers when performing `KademliaHandle::get_prov…
54e8bf5 
…iders()` request
@dmitry-markin
Emit IncomingProvider event
6b9b24a
@dmitry-markin
Add e2e test: retrieving provider
7671f9d
@dmitry-markin
Merge branch 'dm-improve-providers-api' into dm-test-providers
7a5de31
@dmitry-markin
Update GetProvidersContext tests for ContentProvider API
b2e7e33
@dmitry-markin
Update MemoryStore tests for ContentProvider API
4861342
@dmitry-markin
Update Kademlia e2e tests for ContentProvider API
3bb7898
@dmitry-markin
Add e2e test for ADD_PROVIDER
06a2c4d
@dmitry-markin
Test ADD_PROVIDER litep2p -> libp2p
dda8be2
@dmitry-markin
Test ADD_PROVIDER libp2p -> litep2p
a2027b7
@dmitry-markin
Test GET_PROVIDERS libp2p -> litep2p
ecee1fd
@dmitry-markin
Test GET_PROVIDERS litep2p -> libp2p
e7f7b08
@dmitry-markin dmitry-markin added the enhancement New feature or request label Sep 27, 2024
@dmitry-markin dmitry-markin self-assigned this Sep 27, 2024
This was linked to issues Sep 27, 2024
kad: Add support for content providers #196
Closed
kad: Implement ADD_PROVIDER & GET_PROVIDERS RPCs #201
Closed
kad: Implement queries for publishing & retreiving provider records #202
Closed
lexnv
lexnv approved these changes Sep 30, 2024
View reviewed changes
Copy link
Collaborator

@lexnv lexnv left a comment

Choose a reason for hiding this comment

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

LGTM! Nice job here!

Base automatically changed from dm-improve-providers-api to master September 30, 2024 17:02
@dmitry-markin dmitry-markin merged commit d765111 into master Sep 30, 2024
8 checks passed
@dmitry-markin dmitry-markin deleted the dm-test-providers branch September 30, 2024 17:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lexnv lexnv lexnv approved these changes

Assignees

@dmitry-markin dmitry-markin

Labels
enhancement New feature or request
Projects
Networking
Status: Blocked ⛔️
Milestone
No milestone
2 participants
@dmitry-markin @lexnv