Enforce the Dynaconf validaton

SatelliteQE · Jun 17, 2024 · 8c67806 · 8c67806
1 parent 90ffb9b
commit 8c67806
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 4 deletions.
diff --git a/.github/workflows/pull_request.yml b/.github/workflows/pull_request.yml
@@ -9,6 +9,7 @@ env:
     PYCURL_SSL_LIBRARY: openssl
     ROBOTTELO_BUGZILLA__API_KEY: ${{ secrets.BUGZILLA_KEY }}
     ROBOTTELO_JIRA__API_KEY: ${{ secrets.JIRA_KEY }}
+    ROBOTTELO_ROBOTTELO__SETTINGS__SKIP_VALIDATION: true
 
 jobs:
   codechecks:

diff --git a/conf/robottelo.yaml.template b/conf/robottelo.yaml.template
@@ -21,3 +21,4 @@ ROBOTTELO:
   # Dynaconf and Dynaconf hooks related options
   SETTINGS:
     GET_FRESH: true
+    SKIP_VALIDATION: false
diff --git a/robottelo/config/__init__.py b/robottelo/config/__init__.py
@@ -40,10 +40,12 @@ def get_settings():
         try:
             settings.validators.validate()
         except ValidationError as err:
-            logger.warning(
-                f'Dynaconf validation failed, continuing for the sake of unit tests\n{err}'
-            )
-
+            if settings.robottelo.settings.get('skip_validation'):
+                logger.warning(
+                    f'Dynaconf validation failed, continuing for the sake of unit tests\n{err}'
+                )
+            else:
+                raise err
         return settings
 
 

diff --git a/robottelo/config/validators.py b/robottelo/config/validators.py
@@ -335,6 +335,9 @@
         Validator('remotedb.ssl', default=True),
         Validator('remotedb.port', default=5432),
     ],
+    robottelo=[
+        Validator('robottelo.settings.skip_validation', is_type_of=bool, default=False),
+    ],
     shared_function=[
         Validator('shared_function.storage', is_in=('file', 'redis'), default='file'),
         Validator('shared_function.share_timeout', lte=86400, default=86400),