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

deploy: add tests for vllm worker chart #93

Closed
wants to merge 2 commits into from

Conversation

whoisj
Copy link
Collaborator

@whoisj whoisj commented Feb 4, 2025

This change adds a very basic set of tests for the vLLM Worker chart.

This change includes improvements to the basic chart testing scripts.

DLIS-7497

@whoisj whoisj force-pushed the jwyman/helm-test-2 branch from 918c283 to 5695bef Compare February 4, 2025 18:19
@whoisj whoisj force-pushed the jwyman/helm-test-3 branch from b7876ab to 287cc06 Compare February 4, 2025 18:19
@whoisj whoisj force-pushed the jwyman/helm-test-2 branch from 5695bef to 6729a87 Compare February 4, 2025 18:24
@whoisj whoisj force-pushed the jwyman/helm-test-3 branch from 0ae6a01 to c311ff1 Compare February 4, 2025 18:24
@whoisj whoisj force-pushed the jwyman/helm-test-2 branch from 6729a87 to b4cce29 Compare February 4, 2025 18:29
@whoisj whoisj force-pushed the jwyman/helm-test-3 branch from c311ff1 to b3a0b91 Compare February 4, 2025 18:29
@whoisj whoisj force-pushed the jwyman/helm-test-2 branch from b4cce29 to f2e1ab1 Compare February 4, 2025 22:03
@whoisj whoisj force-pushed the jwyman/helm-test-3 branch from b3a0b91 to 0b77054 Compare February 4, 2025 22:04
@whoisj whoisj force-pushed the jwyman/helm-test-2 branch from f2e1ab1 to c86de4a Compare February 7, 2025 21:38
@whoisj whoisj force-pushed the jwyman/helm-test-3 branch from 0b77054 to a7281b7 Compare February 7, 2025 21:38
@whoisj whoisj force-pushed the jwyman/helm-test-2 branch from c86de4a to b8eda8b Compare February 7, 2025 21:54
@whoisj whoisj force-pushed the jwyman/helm-test-3 branch from a7281b7 to c2cae4d Compare February 7, 2025 21:55
@whoisj whoisj requested a review from nicolasnoble February 12, 2025 16:43
This change adds a very basic set of tests for the vLLM Worker chart.

This change includes improvements to the basic chart testing scripts.
This change adds the vLLM chart validation test suite to pre-merge CI automation.
@whoisj
Copy link
Collaborator Author

whoisj commented Feb 18, 2025

Closed at @nnshah1's request.

@whoisj whoisj closed this Feb 18, 2025
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.

1 participant