aws / sagemaker-experiments

Experiment tracking and metric logging for Amazon SageMaker notebooks and model training.
Apache License 2.0
125 stars 36 forks source link

Remove unused service principals in IAM role policy #183

Closed leoroy closed 1 year ago

leoroy commented 1 year ago

Describe the bug Unused service principals in the IAM role policy for integ tests: https://github.com/aws/sagemaker-experiments/blob/main/tests/conftest.py#L322-L324

To Reproduce N/A

Expected behavior We should only keep sagemaker.amazonaws.com as the sole allowed service principal to assume the training role.

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

Environment: Framework (e.g. TensorFlow) / Algorithm (e.g. KMeans): Framework Version: Python Version: CPU or GPU: Python SDK Version: Are you using a custom image:

Additional context Add any other context about the problem here.