Skip to content

Commit 5a69838

Browse files
committed
bypass SystemLogger, which requires a database connection
Signed-off-by: Adrian Edwards <adredwar@redhat.com>
1 parent 666605f commit 5a69838

1 file changed

Lines changed: 15 additions & 2 deletions

File tree

docker/backend/preflight.py

Lines changed: 15 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,23 @@
11
from collectoss.util.startup import collect_env_variables, check_init_schema, check_update_schema, setup_facade_directory, merge_config, warn_import_repos, print_platform_information
2-
from collectoss.application.logs import SystemLogger
2+
from collectoss.application.logs import getFormatter
33
from collectoss.application.cli import DatabaseContext
44
import sys
5+
import logging
56

67
if __name__ == "__main__":
7-
logger = SystemLogger("backend", reset_logfiles=False).get_logger()
8+
# We cannot use systemLogger here because it depends on the database
9+
# At this point in execution, the database may not yet be initialized or
10+
# usable for configuration. So for now we DIY it as a temporary measure
11+
# until we can more comprehensively improve the high level configuration system
12+
logger = logging.getLogger("collectoss.preflight")
13+
log_level = logging.INFO
14+
if not logger.handlers:
15+
handler = logging.StreamHandler()
16+
handler.setLevel(log_level)
17+
handler.setFormatter(getFormatter(log_level))
18+
logger.addHandler(handler)
19+
logger.setLevel(log_level)
20+
logger.propagate = False
821

922
collect_env_variables(logger)
1023

0 commit comments

Comments
 (0)