bitcoin-dev-project / bitcoin-dev-project/warnet

configs, configs, configs

Open
#511 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
149
Forks
83
Avg merge
4d 8h
Merged PRs (30d)
2

Description

Per https://github.com/bitcoin-dev-project/warnet/pull/507#discussion_r1741850547, we are starting to have a few places where configs are defined that rely on shared defaults (rpc ports, rpc usernames and passwords, etc).

Generally speaking, I don't think config values should need a PR to change: they should have a sane default (specified in the values.yaml for each chart, perhaps?) and then a way for the user to supply an override at run time.

We might need to rely on just making sure each values.yaml uses the same default for things like rpc ports. Even better, have a CI check that ensures these don't get out of sync over time.

Another idea would be for anything that requires coordination across multiple helm charts, e.g. rpc port, these must be defined outside the values.yaml file and injected into each chart via some sort of generic config.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.