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

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 Signed-off-by: Kamoltat Sirivadhna <[email protected]>
ceph · Feb 1, 2024 · 1fb2f3c · 1fb2f3c
1 parent 9af7fb0
commit 1fb2f3c
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/setup.cfg b/setup.cfg
@@ -55,6 +55,7 @@ install_requires =
     httpx
     itsdangerous
     pydantic-settings
+    python-dotenv
     # Temporarily, using teuthology without monkey patching the thread
     teuthology @ git+https://github.com/ceph/teuthology@teuth-api#egg=teuthology[test]"
     # Original: git+https://github.com/ceph/teuthology#egg=teuthology[test]

diff --git a/src/teuthology_api/config.py b/src/teuthology_api/config.py
@@ -1,6 +1,7 @@
 from functools import lru_cache
 from pydantic_settings import BaseSettings, SettingsConfigDict
-
+from dotenv import load_dotenv
+load_dotenv()
 
 class APISettings(BaseSettings):
     """