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

HPA enabling usability improvement #770

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

eero-t
Copy link
Contributor

@eero-t eero-t commented Jan 28, 2025

Description

  • HPA requires metrics, so enable monitoring automatically in hpa-values.yaml
  • Minor updates to docs

Issues

n/a.

Type of change

n/a.

Dependencies

n/a.

Tests

Verified that the Helm generated manifests with the updated hpa-values.yaml are identical to ones generated earlier with --set global.monitoring=true option.

Generalized inferencing engine references now that that there are more
alternatives, and default ones have changed.

Signed-off-by: Eero Tamminen <[email protected]>
@eero-t
Copy link
Contributor Author

eero-t commented Jan 28, 2025

@lianhao ChatQnA fails in CI, but failures are not related to changes in this PR. Any ideas?

@lianhao
Copy link
Collaborator

lianhao commented Feb 5, 2025

issue #773 created to track this. I can reproduced the failure locally both on cpu/gaudi environment, I don't know why the CI succeed in cpu environment.

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.

2 participants