Skip to content

Commit b5d8a4a

Browse files
authored
Fix passing of some sagemaker settings (#3221)
1 parent 9bfa980 commit b5d8a4a

File tree

1 file changed

+14
-6
lines changed

1 file changed

+14
-6
lines changed

src/zenml/integrations/aws/orchestrators/sagemaker_orchestrator.py

Lines changed: 14 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -305,21 +305,29 @@ def prepare_or_run_pipeline(
305305
# Retrieve Executor arguments provided in the Step settings.
306306
if use_training_step:
307307
args_for_step_executor = step_settings.estimator_args or {}
308+
args_for_step_executor.setdefault(
309+
"volume_size", step_settings.volume_size_in_gb
310+
)
311+
args_for_step_executor.setdefault(
312+
"max_run", step_settings.max_runtime_in_seconds
313+
)
308314
else:
309315
args_for_step_executor = step_settings.processor_args or {}
316+
args_for_step_executor.setdefault(
317+
"volume_size_in_gb", step_settings.volume_size_in_gb
318+
)
319+
args_for_step_executor.setdefault(
320+
"max_runtime_in_seconds",
321+
step_settings.max_runtime_in_seconds,
322+
)
310323

311324
# Set default values from configured orchestrator Component to
312325
# arguments to be used when they are not present in processor_args.
313326
args_for_step_executor.setdefault(
314327
"role",
315328
step_settings.execution_role or self.config.execution_role,
316329
)
317-
args_for_step_executor.setdefault(
318-
"volume_size_in_gb", step_settings.volume_size_in_gb
319-
)
320-
args_for_step_executor.setdefault(
321-
"max_runtime_in_seconds", step_settings.max_runtime_in_seconds
322-
)
330+
323331
tags = step_settings.tags
324332
args_for_step_executor.setdefault(
325333
"tags",

0 commit comments

Comments
 (0)