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

WIP: overhaul the config management #113

Closed
wants to merge 2 commits into from
Closed

WIP: overhaul the config management #113

wants to merge 2 commits into from

Conversation

ffromani
Copy link
Contributor

@ffromani ffromani commented Jun 15, 2022

Leveraging the the koanf package, we can achieve a longstanding goal for RTE: really useful configuration file handling, AND 1:1 mapping between the configuration file and the command line flags.

This mean we can have:

  1. hardcoded defaults.
  2. a configuration file, which can override any hardcoded default
  3. the commandline flags, which can override defaults and take
    precedence over the configuration file settings.

This allow to greatly reduce the overabundance of commandline flags, and allow shipping configuration file to simplify the deployment.

TBD: need to improve the unit and e2e test coverage

Fixes: #111

Separate bump (go mod tidy + go mod vendor) to make
the followup change leaner.

Signed-off-by: Francesco Romani <[email protected]>
Leveraging the 3rd party koanf package, we can achieve a longstanding goal for
RTE: really useful configuration file handling, AND 1:1 mapping between
the configuration file and the command line flags.

This mean we can have:
1. hardcoded defaults.
2. a configuration file, which can override any hardcoded default
3. the commandline flags, which can override defaults and take
   precedence over the configuration file settings.

This allow to greatly reduce the overabundance of commandline flags,
and allow shipping configuration file to simplify the deployment.

Signed-off-by: Francesco Romani <[email protected]>
@ffromani
Copy link
Contributor Author

we will resume once we have capacity

@ffromani ffromani closed this Sep 30, 2022
@ffromani ffromani deleted the redo-config branch March 16, 2023 11:50
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.

add configuration file
1 participant