Skip to content
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

Add Gaudi2 Support to TGI Chart #414

Merged
merged 2 commits into from
Sep 26, 2024
Merged

Conversation

tylertitsworth
Copy link
Contributor

Description

Update Chart so that more accelerator options are available

Related Issue

MLOPS-2133

Changes Made

  • Improve Configmap handling
  • Add a PVC Spec
  • Add configuration for Security Context
  • Remove ARGs in favor of ENVs
  • Replace probes
  • Fix Secret Bug with apiVersion
  • The code follows the project's coding standards.
  • No Intel Internal IP is present within the changes.
  • The documentation has been updated to reflect any changes in functionality.

Validation

Validate on ITDC w/ Gaudi2

  • I have tested any changes in container groups locally with test_runner.py with all existing tests passing, and I have added new tests where applicable.

Copy link

github-actions bot commented Sep 25, 2024

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

OpenSSF Scorecard

PackageVersionScoreDetails

Scanned Manifest Files

Signed-off-by: tylertitsworth <tyler.titsworth@intel.com>
@tylertitsworth tylertitsworth force-pushed the tylertitsworth/tgi-gaudi branch from 5624cdf to f26ee13 Compare September 26, 2024 22:25
@tylertitsworth tylertitsworth merged commit 600e196 into main Sep 26, 2024
7 checks passed
@tylertitsworth tylertitsworth deleted the tylertitsworth/tgi-gaudi branch September 26, 2024 22:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants