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

ConfigurationManager Audit and Improvements #155

Open
ibrodkin opened this issue Apr 22, 2024 · 1 comment
Open

ConfigurationManager Audit and Improvements #155

ibrodkin opened this issue Apr 22, 2024 · 1 comment
Assignees
Milestone

Comments

@ibrodkin
Copy link
Collaborator

ibrodkin commented Apr 22, 2024

This task is about Configuration Management improvements:

  1. Client configuration management should be separate and minimalistic from the ConfigurationManager used for Chronolog Processes. This would allos to remove current ClientLib dependency on chronolog::ConfigurationManager implementation
  2. Audit ChronoVisor, ChronoGrapher , and ChronoKeeper code and make sure all configurable parameters that are currently hard coded are made configurable
  3. Separate RPC configuration parameters from the parameters that control data processing behavior internal to the respective modules and move the parsing and validation of these parameters' values from the ConfigurationManager and into the respective modules themselves. These methods just take json_objects that generic ConfigurationManager makes available.
  4. Consider making Base RPCConfigurationManager class and module specific configuration classes that derive from the base one.
@ibrodkin
Copy link
Collaborator Author

ibrodkin commented Feb 5, 2025

I've started refactoring of the CodeManager code a few months back , then shelved the changes.

this needs to be picked up in connection with #256

@ibrodkin ibrodkin modified the milestones: 2024-10-18, 2025-02-28 Feb 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant