turbaszek commented on issue #52:
URL: https://github.com/apache/kibble/issues/52#issuecomment-715078800


   @skekre98 thanks for help!
   
   > Do you want refactor `../setup/setup.py` to use `configparser` instead or 
`yaml`?
   
   Yes, I think this would be more python-native solution. However, I would 
suggest to add this `setup/configuration.py` file where we do:
   ```
   class KibbleConfigParser(...):
       ...
   
   conf = KibbleConfigParser()
   ```
   
   Then we can import the `conf` anywhere and access the configuration 
information. But I'm open to any suggestion.
   
   One more thing: I think it would be simpler to merge dbname, host and mort 
of elastiscearch into single connection uri field. WDYT?
   
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to