Instead of configuring a global "cleanup_age" for every process in config:
https://github.com/citysciencelab/urban-model-platform/blob/67665f7477370ea41e59ef24db12e7c7dc54bb83/src/ump/config.py#L29
the user should be able to configure a per-process clearing of jobs and results in providers.yaml.
Also the job cleaning has currently a bug because of inconsistent type handling, when set through env vars: https://github.com/citysciencelab/urban-model-platform/blob/67665f7477370ea41e59ef24db12e7c7dc54bb83/src/ump/main.py#L68
Someone forgot to make sure cleanup_age is of proper int type. This bug will not be detected unless cleanup_age is hit AND is set via env vars.
This bug will be handles in this PR, where the configuration and validation is handled by a pydantic class: #86
Instead of configuring a global "cleanup_age" for every process in config:
https://github.com/citysciencelab/urban-model-platform/blob/67665f7477370ea41e59ef24db12e7c7dc54bb83/src/ump/config.py#L29
the user should be able to configure a per-process clearing of jobs and results in providers.yaml.
Also the job cleaning has currently a bug because of inconsistent type handling, when set through env vars: https://github.com/citysciencelab/urban-model-platform/blob/67665f7477370ea41e59ef24db12e7c7dc54bb83/src/ump/main.py#L68
Someone forgot to make sure
cleanup_ageis of proper int type. This bug will not be detected unless cleanup_age is hit AND is set via env vars.This bug will be handles in this PR, where the configuration and validation is handled by a pydantic class: #86