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

Add AddAddresses method to DHT #879

Merged
merged 9 commits into from
Sep 7, 2023
Merged

Conversation

iand
Copy link

@iand iand commented Sep 7, 2023

Added this to better understand how to test the DHT+coordinator together. Some things need to be refined as we haven't really thought about how things like routing update notifications should be exposed to users of the DHT (or even if they should be).

Added a Close method to the coordinator and the DHT calls it from its own Close method.

Added some test helper methods but there is overlap between some of the existing ones. We should work out a nice set of test helpers that can set up configurations of DHTs. I'd like to do something similar to the Topology test helper that the coordinator package uses but the current Topology implementation doesn't rely on libp2p which avoids some timing issues (like waiting for peerstore etc)

@dennis-tra dennis-tra merged commit 46ff621 into zikade-import Sep 7, 2023
6 of 16 checks passed
@dennis-tra dennis-tra deleted the zikade-import-addaddress branch September 7, 2023 16:10
dennis-tra added a commit that referenced this pull request Sep 7, 2023
* WIP

* revise protobuf

* remove gogo protobuf dependency

* WIP

* add kadt package

* Add routing test

* add custom zikade dependency

* Import zikade code

* Remove generics from zikade imported code

* Update to latest go-kademlia

* Cleanup naming of events

* Minor naming cleanup

* Change maintainers for v2 while being developed

* remove Zikade dependency

* Consolidate type parameters

* Change config test structure

* use opentelemetry

* use convenience attribute methods

* let coord package use tele

* fix golint warnings

* use clock.Clock

* add telemetry context tests

* Improve telemetry documentation

* fix test race

* fix garbage collection race

* Add AddAddresses method to DHT (#879)

* Add AddAddresses method to DHT

* Add AddAddresses method to DHT

* go mod tidy

* Rename Query Skip errors

* go fmt coordinator.go

* Fix test flakes

* Fix lint errors

---------

Co-authored-by: Ian Davis <18375+iand@users.noreply.github.com>
@dennis-tra dennis-tra added the v2 All issues related to the v2 rewrite label Sep 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
v2 All issues related to the v2 rewrite
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants