pulp-oci-images icon indicating copy to clipboard operation
pulp-oci-images copied to clipboard

Allow boolean settings to accept 'true', 't', 'yes', and 'y'

Open dkliban opened this issue 1 year ago • 2 comments

The boolean settings that control the behavior of the multi process container should accept 'true', 't', 'yes', and 'y' for True.

PULP_HTTPS PULP_GUNICORN_RELOAD PULP_OTEL_ENABLED

dkliban avatar Apr 03 '24 21:04 dkliban

I'm unsure what the naively expected condition should be. We could also add "1" or check for "set and not empty".

mdellweg avatar Apr 04 '24 10:04 mdellweg

I'm unsure what the naively expected condition should be. We could also add "1" or check for "set and not empty".

we will check for "1" also.

We will not check for "set and not empty" because we do not want "false" to evaluate to "true".

mikedep333 avatar Apr 09 '24 15:04 mikedep333