aws / aws/sagemaker-python-sdk

`kms_key` in `sagemaker.processing.Processor` should default to `output_kms_key`

Aperta
#4,874 1 commento 0 reazioni 1 assegnatario Rivendicata da @pintaoz-aws Vedi su GitHub
component: processing type: bug
Lingua principale
Python
Stelle
2.3k
Fork
1.3k
Merge medio
1g 22h
PR unite (30g)
35

Descrizione

**Describe the bug**
The `kms_key` used to [encrypt either the user code file or local inputs when uploading to S3](https://github.com/aws/sagemaker-python-sdk/blob/master/src/sagemaker/processing.py#L367-L434) should default to `output_kms_key`.

This would align the behaviour of with `sagemaker.estimator.Estimator` where `output_kms_key` is used to [encrypt the tar'd user training code when uploading to S3](https://github.com/aws/sagemaker-python-sdk/blob/master/src/sagemaker/estimator.py#L990-L1054).

Also, since `output_kms_key` is resolved from the config it means that `kms_key` can inherit this default from the config.

**To reproduce**
A clear, step-by-step set of instructions to reproduce the bug.
The provided code need to be **complete** and **runnable**, if additional data is needed, please include them in the issue.

**Expected behavior**
The `kms_key` should default to `output_kms_key`. This can be implemented in either:

- [`sagemaker.processing.Processor._normalize_args`](https://github.com/aws/sagemaker-python-sdk/blob/master/src/sagemaker/processing.py#L284-L325)
- [`sagemaker.processing.Processor._normalize_inputs`](https://github.com/aws/sagemaker-python-sdk/blob/master/src/sagemaker/processing.py#L367-L434)

**Screenshots or logs**
If applicable, add screenshots or logs to help explain your problem.

**System information**
A description of your system. Please provide:
- **SageMaker Python SDK version**: v2.232.0
- **Framework name (eg. PyTorch) or algorithm (eg. KMeans)**: N/A
- **Framework version**: N/A
- **Python version**: 3.10
- **CPU or GPU**: CPU
- **Custom Docker image (Y/N)**: N/A

**Additional context**
Add any other context about the problem here.

Guida per i contributori

Apri la guida per i contributori

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.