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

feat: add CLI flag support to builder config #315

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

7suyash7
Copy link
Contributor

Implement #303 to add CLI flag support to override config.toml values. This allows operators to quickly modify configuration without editing the config file.

Changes

  • Add CLI Flag support
  • All config values (except el_node_ipc_path, waiting for Make IPC Path optional #314 to land), can be overridden via flags.
  • Secret keys remain config/env only.
  • Added a few tests and help text for all flags.

Example usage

Basic override:

cargo run --bin rbuilder run config.toml --log-json false --log-level debug

Multiple overrides:

cargo run --bin rbuilder run config.toml \
  --log-json false \
  --log-level debug \
  --full-telemetry-server-port 7500 \
  --dry-run true \
  --max-concurrent-seals 2

✅ I have completed the following steps:

  • [✅] Run make lint
  • [✅] Run make test
  • [✅] Added tests (if applicable)

Signed-off-by: 7suyash7 <suyashnyn1@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant