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

[P2P] Move Peerstore interface to a module-shared pkg #811

Open
8 tasks
bryanchriswhite opened this issue Jun 8, 2023 · 0 comments
Open
8 tasks

[P2P] Move Peerstore interface to a module-shared pkg #811

bryanchriswhite opened this issue Jun 8, 2023 · 0 comments
Assignees
Labels
core Core infrastructure - protocol related p2p P2P specific changes

Comments

@bryanchriswhite
Copy link
Contributor

bryanchriswhite commented Jun 8, 2023

Objective

Support usage of the Peerstore interface type in (sub)module method signatures while preventing import cycles.

Origin Document

Observation made while working on #732 & #855. In order to complete the peerstore provider refactor as a submodule the Peerstore type needs to be moved to avoid creating an import cycle between the p2p/types package and the shared/modules package.

Goals

  • Avoid import cycle on Peerstore.

Deliverable

  • Move the Peerstore interface from p2p/types/peerstore.go to a shared/core/types (or similar / more appropriate)

Non-goals / Non-deliverables

  • Unrelated refactoring

General issue deliverables

  • Update the appropriate CHANGELOG(s)
  • Update any relevant local/global README(s)
  • Update relevant source code tree explanations
  • Add or update any relevant or supporting mermaid diagrams

Testing Methodology

  • All tests: make test_all
  • LocalNet: verify a LocalNet is still functioning correctly by following the instructions at docs/development/README.md
  • k8s LocalNet: verify a k8s LocalNet is still functioning correctly by following the instructions here

Creator: @bryanchriswhite

@bryanchriswhite bryanchriswhite added core Core infrastructure - protocol related p2p P2P specific changes triage It requires some decision-making at team level (it can't be worked on as it stands) labels Jun 8, 2023
@bryanchriswhite bryanchriswhite self-assigned this Jun 9, 2023
@bryanchriswhite bryanchriswhite removed the triage It requires some decision-making at team level (it can't be worked on as it stands) label Jun 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core Core infrastructure - protocol related p2p P2P specific changes
Projects
Status: Up Next
Development

No branches or pull requests

1 participant