Skip to content

Conversation

@googs1025
Copy link
Member

What this PR does / why we need it

  • add in-tree BackendRuntime for preStop Hook

Which issue(s) this PR fixes

Fixes #288 (comment)

Special notes for your reviewer

Does this PR introduce a user-facing change?

add in-tree vllm BackendRuntime for preStop Hook

@InftyAI-Agent InftyAI-Agent added needs-triage Indicates an issue or PR lacks a label and requires one. needs-priority Indicates a PR lacks a label and requires one. do-not-merge/needs-kind Indicates a PR lacks a label and requires one. labels Mar 18, 2025
@InftyAI-Agent InftyAI-Agent requested a review from kerthcet March 18, 2025 01:43
@kerthcet
Copy link
Member

Thanks.
/lgtm
/approve
/kind feature

@InftyAI-Agent InftyAI-Agent added lgtm Looks good to me, indicates that a PR is ready to be merged. approved Indicates a PR has been approved by an approver from all required OWNERS files. feature Categorizes issue or PR as related to a new feature. and removed do-not-merge/needs-kind Indicates a PR lacks a label and requires one. labels Mar 18, 2025
@InftyAI-Agent InftyAI-Agent merged commit 211a0ef into InftyAI:main Mar 18, 2025
42 of 44 checks passed
@kerthcet
Copy link
Member

I'll create another issue tracking other backends.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. feature Categorizes issue or PR as related to a new feature. lgtm Looks good to me, indicates that a PR is ready to be merged. needs-priority Indicates a PR lacks a label and requires one. needs-triage Indicates an issue or PR lacks a label and requires one.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

PreStop command injection for inference engines

3 participants