rptaylor / kapel

APEL accounting for Kubernetes.
GNU General Public License v3.0
3 stars 3 forks source link

Follow up on strict check for config file in environs/dotenv #44

Open rptaylor opened 5 months ago

rptaylor commented 5 months ago

Currently the chart invokes KAPEL without a config file, instead using environment variables: https://github.com/rptaylor/kapel/blob/master/chart/templates/cronjob.yaml#L73

Nevertheless it is possible, and for CLI testing could be useful, to use a config file: https://github.com/rptaylor/kapel/blob/master/python/KAPEL.py#L266

Follow up on https://github.com/sloria/environs/issues/294 to fail explicitly if a config file is requested but not found.

However I'm not sure if this handles the undesirable situation that a stray config file could get used accidentally: https://github.com/rptaylor/kapel/blob/master/python/KAPELConfig.py#L6