search

SatelliteQE / satellite-populate

This new module adds tools and commands to populate and validate entities in the system based in YAML file.
Other
8 stars 10 forks source link

Added functionality for populating satellites with config file #63 #86

Closed chbrown13 closed 7 years ago

chbrown13 commented 7 years ago

@rochacbruno First attempt at using a config file to load settings for satellite-populate. Moving to new branch.

rochacbruno commented 7 years ago

As in unix the precedence should be:

1) read all values from config file and use as default (~/.satellite-populate or /etc/satellite-populate) 2) read all values from environment variables and override default values (click already does it for some vars, we must configure for others) 3) if argument explicit passed it override ENV and config file.

chbrown13 commented 7 years ago

@rochacbruno Do you mean there should there be an environment variable for each option?

rochacbruno commented 7 years ago

@chbrown13 there is already https://github.com/SatelliteQE/satellite-populate/blob/master/satellite_populate/commands.py#L123

chbrown13 commented 7 years ago

@rochacbruno Added some unittests for the config file functionality.

rochacbruno commented 7 years ago

Python 3.x is failing with tempfile.write https://travis-ci.org/SatelliteQE/satellite-populate/jobs/264820612