-
Notifications
You must be signed in to change notification settings - Fork 33
[P2P] Define outstanding parameters & reasonable defaults #629
Copy link
Copy link
Open
Labels
code healthNice to have code improvementNice to have code improvementcore starter taskGood for newcomers, but aimed at core team members though still open for everyoneGood for newcomers, but aimed at core team members though still open for everyonep2pP2P specific changesP2P specific changes
Milestone
Description
Objective
Support configuration of any new degrees of freedom introduced by or since libp2p integration.
Origin Document
Out of scope for #553.
Goals
- Address outstanding
// TECHDEBTin the P2P regarding values which should be parameterized - Determine what "reasonable" defaults are for each
Deliverable
- Add missing P2P config fields
- Ensure defaults are defined in
runtime/defaults/defaults.go
Non-goals / Non-deliverables
- Addressing additional
// TECHDEBTcomments or refactoring outside the scope of updating the P2P config
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
- Task specific tests or benchmarks: Tests should exercise new config fields
- All tests:
make test_all - LocalNet: verify a
LocalNetis still functioning correctly by following the instructions at docs/development/README.md
Creator: @bryanchriswhite
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
code healthNice to have code improvementNice to have code improvementcore starter taskGood for newcomers, but aimed at core team members though still open for everyoneGood for newcomers, but aimed at core team members though still open for everyonep2pP2P specific changesP2P specific changes
Type
Projects
Status
Backlog
