Skip to content

Commit 5fecbcc

Browse files
Guillem96Guillem Orellana Trullols
andauthored
fix: RepackStep must use the same KMS key as the Model (#3692)
Co-authored-by: Guillem Orellana Trullols <[email protected]>
1 parent ca1e535 commit 5fecbcc

File tree

2 files changed

+2
-0
lines changed

2 files changed

+2
-0
lines changed

src/sagemaker/workflow/model_step.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -268,6 +268,7 @@ def _append_repack_model_step(self):
268268
depends_on=self.depends_on,
269269
retry_policies=self._repack_model_retry_policies,
270270
output_path=self._runtime_repack_output_prefix,
271+
output_kms_key=model.model_kms_key,
271272
)
272273
self.steps.append(repack_model_step)
273274

src/sagemaker/workflow/step_collections.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -400,6 +400,7 @@ def __init__(
400400
security_group_ids=estimator.security_group_ids,
401401
description=description,
402402
display_name=display_name,
403+
output_kms_key=estimator.output_kms_key,
403404
)
404405
steps.append(repack_model_step)
405406
model_data = repack_model_step.properties.ModelArtifacts.S3ModelArtifacts

0 commit comments

Comments
 (0)