-
Notifications
You must be signed in to change notification settings - Fork 70
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
'trainingJobName' failed to satisfy constraint #41
Comments
I think this may be because an additional tag seems to be added to image_name from two locations after the changes in 0.11.0. The first time docker_tag is added is in sagify/api/cloud.py: image_name = config.image_name+':'+docker_tag def _construct_image_location(self, image_name):
account = self.boto_session.client('sts').get_caller_identity()['Account']
region = self.boto_session.region_name
return '{account}.dkr.ecr.{region}.amazonaws.com/{image}:latest'.format(
account=account,
region=region,
image=image_name
) Note that in the return statement ':latest' is always appended - so I think what potentially happens is you get an image_name with ':something:something' appended and sagemaker doesn't like that. I found these changes have solved this error for me for now: https://github.com/andreas-grivas/sagify/commit/d23cccd19106c50f1f7a50e17cbf9e0aac636755 Until this is investigated further, a quick fix is to specify the name parameter in the call to sage.Model() - eg. sage.Model(name='MyModel', restofparams) in the deploy function under sagify/sagemaker/sagemaker.py: |
I think I found the issue. SageMaker doesn't like underscores ( |
Does sagify add underscores anywhere though? |
@pm3310 It could be underscores too, but what @andreas-grivas suggests is definitely happening and needs fixing. A colon is appended to the image name every time we call
In other words, if not tag is provided, we end up with an image name like this:
and when a tag e.g. 'my-tag' is indeed provided the image name looks like this:
The fix is straightforward I think. We need to check for the existence of a tag in the above linked lines and append the tag or The docker tag feature was implemented rather hastily. In addition to the above, no additional method doc entries were added where the |
With #44 merged, @pm3310 can you try to see if the errors you were getting persist? @andreas-grivas You should not need to use your workaround anymore, let me know if that's not the case and I'll get to it 😄 |
New version deployed https://pypi.org/project/sagify/ :-D Version 0.12.1 is available via pypi |
@andreas-grivas Would you like to become a Sagify active contributor? If you wish so, I can give you contributor access permissions :-) That means you'll be able to deploy new versions and review PRs :-) |
Hey @andreas-grivas no worries :-) We're planning to launch an enterprise solution early next year for training/deploying ML models on SageMaker much easier. Of course, it's going to be based on this open-source project :-) The benefits of the solution will be:
Please, let me know if you'd like to be of the early adopters |
I get the following error with the latest version but not for sagify version 0.10
The text was updated successfully, but these errors were encountered: