-
Notifications
You must be signed in to change notification settings - Fork 816
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
docker nightly builds #1562
docker nightly builds #1562
Conversation
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
|
||
# Build Nightly images and append the date in the name | ||
os.system(f"./build_image.sh -bt dev -t pytorch/{cpu_version}") | ||
os.system(f"./build_image.sh -bt dev -g -cv cu102 -t pytorch/{gpu_version}") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a reason we're building this with cu102 and not cu113?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Our experience is generally it's been more stable but if that's no longer the case if we just update this line to cu113
and the nightly builds will work just fine
@@ -0,0 +1,26 @@ | |||
name: Push Docker Nightly |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: Filename is missing an 'l'
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah whoops thank you for catching this
Oh I see, I was just curious
…On Wed, Apr 13, 2022 at 20:48 Mark Saroufim ***@***.***> wrote:
***@***.**** commented on this pull request.
------------------------------
In docker/docker_nightly.py
<#1562 (comment)>:
> @@ -0,0 +1,26 @@
+from datetime import date
+import os
+
+def get_nightly_version():
+ today = date.today()
+ return today.strftime("%Y.%m.%d")
+
+if __name__ == "__main__":
+ project = "torchserve-nightly"
+ cpu_version = f"{project}:cpu-{get_nightly_version()}"
+ gpu_version = f"{project}:gpu-{get_nightly_version()}"
+
+ # Build Nightly images and append the date in the name
+ os.system(f"./build_image.sh -bt dev -t pytorch/{cpu_version}")
+ os.system(f"./build_image.sh -bt dev -g -cv cu102 -t pytorch/{gpu_version}")
Our experience is generally it's been more stable but if that's no longer
the case if we just update this line to cu113 and the nightly builds will
work just fine
—
Reply to this email directly, view it on GitHub
<#1562 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADFYED7GZGLIPGCVHPYPZG3VE6IRJANCNFSM5TLOSPQA>
.
You are receiving this because your review was requested.Message ID:
***@***.***>
|
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
Images will be here: https://hub.docker.com/repository/docker/pytorch/torchserve-nightly
Tested here: https://github.com/msaroufim/serve/actions
Fixes #1558