Conversation
Added the eu-west-1 ECR registry so it's possible to run sm-docker with eu-west-1 as AWS_DEFAULT_REGION
jaipreet-s
left a comment
There was a problem hiding this comment.
Thanks for the contribution!
I checked at [1] and the repository ID for eu-west-1 is 763104351884. Where is this registry ID coming from?
[1] https://github.com/aws/deep-learning-containers/blob/master/available_images.md
|
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 |
|
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. |
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.