IntersectMBO / IntersectMBO/ouroboros-consensus

Clean up node arguments

Open
#630 1 comment 0 reactions 0 assignees View on GitHub
technical debt
Dominant language
Haskell
Stars
67
Forks
43
Avg merge
5d 13h
Merged PRs (30d)
43

Description

The argument handling in [`Ouroboros.Consensus.Node`](https://github.com/input-output-hk/ouroboros-network/blob/master/ouroboros-consensus/src/Ouroboros/Consensus/Node.hs) should be cleaned up. There are: `RunNodeArgs`, `NodeArgs`, `DiffusionArguments`, `ChainDbArgs`, ... :exploding_head:.

We should strike a balance between:
* Simple: keeping things as simple as possible for its main user, i.e., `cardano-node`. No footguns, good defaults, minimal number of arguments to provide.
* Re-usability: the ThreadNet tests should be able to use the same `run` function (#2472)
* Power: allow overriding them in `cardano-node` when necessary, e.g., changing the cache parameters without having to recompile the consensus layer.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.