awslabs/sagemaker-deep-demand-forecast

[Bug]

juliensimon opened this issue · 2 comments

juliensimon commented

In src/preprocess/container/build_and_push.sh:

The get-login command is deprecated and must be replaced by the get-login-password command.

Here's the (tested) fix:

# Get the login command from ECR and execute it directly
aws ecr get-login-password --region ${region}| docker login \
     --username AWS \
     --password-stdin ${account}.dkr.ecr.${region}.amazonaws.com
ehsanmok commented

Thanks Julien! Fixed.

  • 👍1
juliensimon commented

That was fast!