fix(alembic): Making Alembic logger config optional (#17774)

* Making Alembic logger config optional

* Switching to app config

superset/config.py

@@ -103,6 +103,11 @@ def _try_json_readsha(filepath: str, length: int) -> Optional[str]:
         return None
 
 
+#
+# If True, we will skip the call to load the logger config found in alembic.init
+#
+ALEMBIC_SKIP_LOG_CONFIG = False
+
 # Depending on the context in which this config is loaded, the
 # version_info.json file may or may not be available, as it is
 # generated on install via setup.py. In the event that we're

superset/migrations/env.py

@@ -31,7 +31,9 @@ config = context.config
 
 # Interpret the config file for Python logging.
 # This line sets up loggers basically.
-fileConfig(config.config_file_name)
+if not current_app.config["ALEMBIC_SKIP_LOG_CONFIG"]:
+    # Skip loading logger config if the user has this config set
+    fileConfig(config.config_file_name)
 logger = logging.getLogger("alembic.env")
 
 DATABASE_URI = current_app.config["SQLALCHEMY_DATABASE_URI"]