sagemaker-studio-image-build-cli icon indicating copy to clipboard operation
sagemaker-studio-image-build-cli copied to clipboard

Added eu-west-1 registry

Open jverstraaten opened this issue 4 years ago • 2 comments

Added the eu-west-1 ECR registry so it's possible to run sm-docker with eu-west-1 as AWS_DEFAULT_REGION

Issue #, if available:

Description of changes: Added the eu-west-1 registry for sagemaker studio images so it's possible to use sm-docker in ireland region

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

jverstraaten avatar Apr 08 '21 09:04 jverstraaten

We were getting these from the AWS Sagemaker docs: https://docs.aws.amazon.com/sagemaker/latest/dg/sagemaker-algo-docker-registry-paths.html. I looks like they changed it to have a separate account per region and per algorithm. I guess a second PR should be made to handle these changes in a more complete fashion but if we can merge this at least I can continue for now

jverstraaten avatar Apr 09 '21 06:04 jverstraaten

Thanks for the details. I recommend adding all the regions' account IDs for at-least the algo in question. Otherwise, it's a confusing user experience where the tool only works in one specific region.

jaipreet-s avatar Apr 19 '21 04:04 jaipreet-s