-
Notifications
You must be signed in to change notification settings - Fork 2
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
Update docker images names #79
Conversation
@@ -31,14 +31,14 @@ build_be: | |||
docker build \ | |||
--build-arg DEVICE=$(DEVICE) \ | |||
--build-arg STAGE=$(STAGE) \ | |||
-t $(REGISTRY)/$(IMAGE)_$(DEVICE):$(TAG)$(TAG_EXT) \ | |||
-t $(REGISTRY)/$(IMAGE):$(TAG)_$(DEVICE)$(TAG_EXT) \ |
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.
So we don't tag any image with latest
, right? Should we?
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.
Mmm tensorflow assigns latest to their CPU version, should we?
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.
So like latest
and latest-gpu
. That way if you forget to specify a tag you at least get something that works. On the other hand, making it explicit like you did prevents users thinking they are using GPU when they are not. I like both. You choose!
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.
Tiny comment, but I think it will be good to go.
Co-authored-by: Joseph Marinier <joseph.marinier@servicenow.com>
Description:
Update docker images names
Checklist:
You should check all boxes before the PR is ready. If a box does not apply, check it to acknowledge
it.
ran
pre-commit run --all-files
at the end.Run
cd webapp && yarn types
while the back-end is running.our users.
README
files and our wiki for any big design decisions, if relevant.