Skip to content

Commit 62252a9

Browse files
committed
fix(logs): suppress noisy logs during setup and upgrade
1 parent 2fe741a commit 62252a9

File tree

4 files changed

+8
-2
lines changed

4 files changed

+8
-2
lines changed

testgen/commands/run_launch_db_config.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -86,6 +86,7 @@ def run_launch_db_config(delete_db: bool, drop_users_and_roles: bool = True) ->
8686
user_override=params_mapping["TESTGEN_ADMIN_USER"],
8787
password_override=params_mapping["TESTGEN_ADMIN_PASSWORD"],
8888
user_type="schema_admin",
89+
suppress_logs=True,
8990
)
9091
import_metadata_records_from_yaml(params_mapping)
9192

testgen/commands/run_upgrade_db_config.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -96,6 +96,7 @@ def _refresh_static_metadata(params_mapping):
9696
user_override=params_mapping["TESTGEN_ADMIN_USER"],
9797
password_override=params_mapping["TESTGEN_ADMIN_PASSWORD"],
9898
user_type="schema_admin",
99+
suppress_logs=True,
99100
)
100101
import_metadata_records_from_yaml(params_mapping)
101102

@@ -106,6 +107,7 @@ def _refresh_static_metadata(params_mapping):
106107
user_override=params_mapping["TESTGEN_ADMIN_USER"],
107108
password_override=params_mapping["TESTGEN_ADMIN_PASSWORD"],
108109
user_type="schema_admin",
110+
suppress_logs=True,
109111
)
110112

111113

testgen/common/database/database_service.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -134,8 +134,9 @@ def execute_db_queries(
134134
user_override: str | None = None,
135135
password_override: str | None = None,
136136
user_type: UserType = "normal",
137+
suppress_logs: bool = False,
137138
) -> tuple[list[Any], list[int]]:
138-
LOG.info(f"DB operation: execute_db_queries on {'Target' if use_target_db else 'App'} database (User type = {user_type})")
139+
LOG.info(f"DB operation: execute_db_queries ({len(queries)}) on {'Target' if use_target_db else 'App'} database (User type = {user_type})")
139140

140141
with _init_db_connection(use_target_db, user_override, password_override, user_type) as connection:
141142
return_values: list[Any] = []
@@ -144,7 +145,8 @@ def execute_db_queries(
144145
LOG.info("No queries to process")
145146
for index, (query, params) in enumerate(queries):
146147
LOG.debug(f"Query: {query}")
147-
LOG.info(f"Processing {index + 1} of {len(queries)} queries")
148+
if not suppress_logs:
149+
LOG.info(f"Processing {index + 1} of {len(queries)} queries")
148150
transaction = connection.begin()
149151
result = connection.execute(text(query), params)
150152
row_counts.append(result.rowcount)

testgen/common/read_yaml_metadata_records.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -164,6 +164,7 @@ def _process_yaml_for_import(params_mapping: dict, data:dict, parent_table:str,
164164
user_override=params_mapping["TESTGEN_ADMIN_USER"],
165165
password_override=params_mapping["TESTGEN_ADMIN_PASSWORD"],
166166
user_type="schema_admin",
167+
suppress_logs=True,
167168
)
168169
return
169170

0 commit comments

Comments
 (0)