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

feat: Add vLLM counter metrics access through Triton #7493

Merged
merged 4 commits into from
Aug 16, 2024

Conversation

yinggeh
Copy link
Contributor

@yinggeh yinggeh commented Aug 3, 2024

What does the PR do?

Add vLLM counter metrics access through python_backend custom metrics.

Checklist

  • PR title reflects the change and is of format <commit_type>: <Title>
  • Changes are described in the pull request.
  • Related issues are referenced.
  • Populated github labels field
  • Added test plan and verified test passes.
  • Verified that the PR passes existing CI.
  • Verified copyright is correct on all changed files.
  • Added succinct git squash message before merging ref.
  • All template sections are filled out.
  • Optional: Additional screenshots for behavior/output changes with before/after.

Commit Type:

Check the conventional commit type
box here and add the label to the github PR.

  • feat

Related PRs:

triton-inference-server/vllm_backend#53

Where should the reviewer start?

n/a

Test plan:

L0_backend_vllm/metrics_test

  • CI Pipeline ID:
    17372863

Caveats:

n/a

Background

Customers requested.

Related Issues: (use one of the action keywords Closes / Fixes / Resolves / Relates to)

n/a

@yinggeh yinggeh added the enhancement New feature or request label Aug 3, 2024
@yinggeh yinggeh self-assigned this Aug 3, 2024
@yinggeh yinggeh changed the title feat: Add vLLM metrics access through Triton feat: Add vLLM metrics (counter and gauge) access through Triton Aug 7, 2024
@oandreeva-nv
Copy link
Contributor

remove gauge in the name of the PR

@yinggeh yinggeh changed the title feat: Add vLLM metrics (counter and gauge) access through Triton feat: Add vLLM counter metrics access through Triton Aug 9, 2024
@yinggeh yinggeh requested review from kthui and GuanLuo August 9, 2024 21:25
oandreeva-nv
oandreeva-nv previously approved these changes Aug 9, 2024
krishung5
krishung5 previously approved these changes Aug 9, 2024
kthui
kthui previously approved these changes Aug 9, 2024
GuanLuo
GuanLuo previously approved these changes Aug 13, 2024
@yinggeh yinggeh dismissed stale reviews from GuanLuo, kthui, oandreeva-nv, and krishung5 via 35cd4a3 August 15, 2024 08:39
@yinggeh yinggeh force-pushed the yinggeh-DLIS-7061-add-vllm-metrics branch from 35cd4a3 to 6f601f4 Compare August 15, 2024 08:42
@yinggeh yinggeh requested a review from kthui August 15, 2024 19:14
@yinggeh yinggeh merged commit e7c8e7b into main Aug 16, 2024
3 checks passed
@yinggeh yinggeh deleted the yinggeh-DLIS-7061-add-vllm-metrics branch August 16, 2024 03:35
@yinggeh yinggeh restored the yinggeh-DLIS-7061-add-vllm-metrics branch August 16, 2024 10:18
mc-nv pushed a commit that referenced this pull request Aug 19, 2024
Report vLLM counter metrics through Triton server
mc-nv added a commit that referenced this pull request Aug 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Development

Successfully merging this pull request may close these issues.

5 participants