Skip to content

Upgrade tgi and tei version #931

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

Merged
merged 2 commits into from
Apr 3, 2025
Merged

Upgrade tgi and tei version #931

merged 2 commits into from
Apr 3, 2025

Conversation

yongfengdu
Copy link
Collaborator

Description

Upgrade the TGI and TEI to be consistent with compose deployment

Issues

#921 #906

Type of change

List the type of change like below. Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds new functionality)
  • Breaking change (fix or feature that would break existing design and interface)

Dependencies

List the newly introduced 3rd party dependency if exists.

Tests

Describe the tests that you ran to verify your changes.

@yongfengdu yongfengdu requested review from poussa and mkbhanda April 2, 2025 06:47
@yongfengdu yongfengdu requested a review from lianhao as a code owner April 2, 2025 06:47
@yongfengdu
Copy link
Collaborator Author

The CI failure is due to an GenAIExample change, which will be fixed at #910

Use 2.4.1 for GPU.
Others already synced with Gaudi 2.3.1 and cpu 2.4.0-intel-cpu

Signed-off-by: Dolpher Du <[email protected]>
Copy link
Collaborator

@lianhao lianhao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@yongfengdu maybe we should remove liveliness check for both tei/teirerank/tgi ?

@yongfengdu yongfengdu force-pushed the tgitei branch 2 times, most recently from 2f467bf to e5e0d7d Compare April 2, 2025 14:37
@yongfengdu
Copy link
Collaborator Author

@yongfengdu maybe we should remove liveliness check for both tei/teirerank/tgi ?

I tried to remove the tgi/tei/teirerank livenessProbe, but we also need to remove the tgi/tei/teirerank livenessProbe settings in all Examples, which will trigger a bigger test matrix. (Otherwise helm will report error)
Let's discuss offline and handle this in another PR if necessary.

@lianhao lianhao merged commit bde78dd into opea-project:main Apr 3, 2025
82 of 86 checks passed
@yongfengdu yongfengdu deleted the tgitei branch April 7, 2025 02:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants