Skip to content

Commit e6875e1

Browse files
committed
fix log levels for docker run
1 parent 0eb6d97 commit e6875e1

File tree

2 files changed

+10
-11
lines changed

2 files changed

+10
-11
lines changed

automation/script/docker.py

+6-10
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,6 @@ def dockerfile(self_module, input_params):
2020
run_command_arc = prune_result['new_input']
2121
current_directory = os.getcwd()
2222
is_quiet_mode = input_params.get('quiet', False)
23-
verbose = input_params.get('v', False)
2423
is_console_output = input_params.get('out') == 'con'
2524

2625
# Step 2: Search for scripts
@@ -88,8 +87,7 @@ def dockerfile(self_module, input_params):
8887
new_state_keys_from_meta=[], # Add state keys from meta if needed
8988
add_deps_recursive=add_deps_recursive,
9089
run_state=run_state,
91-
recursion_spaces='',
92-
verbose=verbose # Set to True or False as needed
90+
recursion_spaces=''
9391
)
9492
if update_variations_result['return'] > 0:
9593
return update_variations_result
@@ -109,7 +107,7 @@ def dockerfile(self_module, input_params):
109107
deps = docker_settings.get('build_deps', [])
110108
if deps:
111109
r = self_module._run_deps(
112-
deps, [], env, {}, {}, {}, {}, '', [], '', False, '', verbose,
110+
deps, [], env, {}, {}, {}, {}, '', [], '', False, '',
113111
show_time, ' ', run_state)
114112
if r['return'] > 0:
115113
return r
@@ -180,7 +178,7 @@ def dockerfile(self_module, input_params):
180178
'comments': comments, 'run_cmd': f"{run_command_string} --quiet",
181179
'script_tags': input_params.get('tags'), 'env': env,
182180
'dockerfile_env': dockerfile_env,
183-
'quiet': True, 'v': input_params.get('v', False), 'real_run': True
181+
'quiet': True, 'real_run': True, '-v': is_true(input_params.get('v', input_params.get('verbose', False))), '-s': is_true(input_params.get('s', input_params.get('silent', False)))
184182
}
185183
mlc_docker_input.update(docker_inputs)
186184

@@ -209,7 +207,6 @@ def docker_run(self_module, i):
209207

210208
# Extract and handle basic inputs
211209
quiet = i.get('quiet', False)
212-
verbose = i.get('v', False)
213210
show_time = i.get('show_time', False)
214211
logger = self_module.logger
215212
env = i.get('env', {})
@@ -296,8 +293,7 @@ def docker_run(self_module, i):
296293
posthook_deps=[],
297294
new_env_keys_from_meta=[],
298295
new_state_keys_from_meta=[],
299-
add_deps_recursive=add_deps_recursive, run_state=run_state, recursion_spaces='',
300-
verbose=False)
296+
add_deps_recursive=add_deps_recursive, run_state=run_state, recursion_spaces='')
301297
if r['return'] > 0:
302298
return r
303299

@@ -306,7 +302,7 @@ def docker_run(self_module, i):
306302
deps = docker_settings.get('deps', [])
307303
if deps:
308304
r = self_module._run_deps(
309-
deps, [], env, {}, {}, {}, {}, '', [], '', False, '', verbose,
305+
deps, [], env, {}, {}, {}, {}, '', [], '', False, '',
310306
show_time, ' ', run_state)
311307
if r['return'] > 0:
312308
return r
@@ -374,7 +370,7 @@ def docker_run(self_module, i):
374370
'action': 'run', 'target': 'script', 'tags': 'run,docker,container',
375371
'rebuild': rebuild_docker_image,
376372
'env': env, 'mounts': mounts,
377-
'script_tags': i.get('tags'), 'run_cmd': final_run_cmd, 'v': verbose,
373+
'script_tags': i.get('tags'), 'run_cmd': final_run_cmd,
378374
'quiet': True, 'real_run': True, 'add_deps_recursive': {'build-docker-image': {'dockerfile': dockerfile_path}},
379375
**docker_inputs
380376
}

automation/script/docker_utils.py

+4-1
Original file line numberDiff line numberDiff line change
@@ -344,7 +344,10 @@ def rebuild_flags(
344344
command_line += f" --{full_key},={list_values}"
345345
# Process scalar values.
346346
else:
347-
command_line += f" --{full_key}={quote}{str(value)}{quote}"
347+
if full_key in ['s', 'v']:
348+
command_line += f" -{full_key}"
349+
else:
350+
command_line += f" --{full_key}={quote}{str(value)}{quote}"
348351

349352
return command_line
350353

0 commit comments

Comments
 (0)