setup.cfg & config.py: make sure we load_dotenv() - Githubissues

ceph / teuthology-api

A REST API to execute teuthology commands.

MIT License

2 stars 10 forks source link

setup.cfg & config.py: make sure we load_dotenv() #46

Closed kamoltat closed 9 months ago

kamoltat commented 9 months ago

Currently, setup.cfg doesn't have python-dotenv as a required dependency so we add that.

Moreover, in config.py we didn't really load_dotenv() so we didn't actually load .env into pydantic_settings

Contribution Guidelines

To sign and test your commits, please refer to Contibution guidelines.

Checklist

[x] Changes tested in container setup
[ ] Changes tested in non-containerized setup
[ ] Includes tests
[ ] Documentation updated

kamoltat commented 9 months ago

@VallariAg Just needs a thumbs up one more time