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

[FEA] Use app configurations to load constants and settings #563

Open
amahussein opened this issue Sep 13, 2023 · 0 comments
Open

[FEA] Use app configurations to load constants and settings #563

amahussein opened this issue Sep 13, 2023 · 0 comments
Labels
feature request New feature or request new-cli scope out future new-cli work for the next few months user_tools Scope the wrapper module running CSP, QualX, and reports (python)

Comments

@amahussein
Copy link
Collaborator

Is your feature request related to a problem? Please describe.

It will be nice to refactor the user_tools to use a configuration management.
The configuration file can be loaded from the resources folder but a user can supply modified config file that can be loaded from local disk. For example ~/.pyrapids/conf.{json,yaml..etc}

There are many advantage of use configuration management:

  • Easier to alter the behavior with less arguments and less hardcoded values in the code
  • We can use environments (production/test/dev) which makes it easier to have enviormnet variables and secrets enabling Integration tests against the CSPs

Describe the solution you'd like

We already have a configuration manager to load some metadata for each tool (i.e., propContainer).
However, for the global app configuration, it will be a plus to use an existing package such as DynaConf because of the following:

  • it has a strong community support
  • it supports multiple formats
  • it supports secrets and environments (dev/test/..etc)
  • it supports loading environment variables
@amahussein amahussein added feature request New feature or request user_tools Scope the wrapper module running CSP, QualX, and reports (python) ? - Needs Triage new-cli scope out future new-cli work for the next few months labels Sep 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request New feature or request new-cli scope out future new-cli work for the next few months user_tools Scope the wrapper module running CSP, QualX, and reports (python)
Projects
None yet
Development

No branches or pull requests

2 participants