-
Notifications
You must be signed in to change notification settings - Fork 24
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
feat(TGI): add release docker image build and push to registry workflow #62
Conversation
4797e31
to
132f72c
Compare
This will be triggered at each release.
132f72c
to
c75a314
Compare
c75a314
to
61c4d42
Compare
labels: ${{ steps.meta.outputs.labels }} | ||
build-args: | | ||
VERSION=${{ steps.version.outputs.version }} | ||
TGI_VERSION=5bc3d65dd32ba1f979540caeccbf3dd8798dd9df |
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 TGI_VERSION supposed to be frozen here ?
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.
short answer: yes, for now. Long answer: in the future, TPU TGI development should move into TGI repo, so image building and version will be different. But I guess this should be fine for now.
labels: ${{ steps.meta-ie.outputs.labels }} | ||
build-args: | | ||
VERSION=${{ steps.version.outputs.version }} | ||
TGI_VERSION=5bc3d65dd32ba1f979540caeccbf3dd8798dd9df |
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.
same question as above
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.
same answer
What does this PR do?
Prepares CI to upload TGI image on TPUs.