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

ci: Fix L0_lifecycle server shutdown (#7933) #7936

Closed
wants to merge 3,492 commits into from

Conversation

kthui
Copy link
Contributor

@kthui kthui commented Jan 14, 2025

Pick #7933 into 25.01 release

krishung5 and others added 30 commits February 14, 2024 10:18
* Add unit test reports to L0_dlpack_multi_gpu

* Add unit test reports to L0_warmup
* Add response statistics

* Add L0_response_statistics

* Enable http vs grpc statistics comparison

* Add docs for response statistics protocol

* Add more comments for response statistics test

* Remove model name from config

* Improve docs wordings

* [Continue] Improve docs wordings

* [Continue] Add more comments for response statistics test

* [Continue 2] Improve docs wordings

* Fix typo

* Remove mentioning decoupled from docs

* [Continue 3] Improve docs wordings

* [Continue 4] Improve docs wordings

Co-authored-by: Ryan McCormick <[email protected]>

---------

Co-authored-by: Ryan McCormick <[email protected]>
* Switch to Python model for busyop test

* Clean up

* Address comment

* Remove unused import
* Add cancellation into response statistics

* Add test for response statistics cancel

* Remove debugging print

* Use is None comparison

* Fix docs

* Use default args None

* Refactor RegisterModelStatistics()
* Modify "header_forward_pattern" to match headers case-insensitively. Add unit tests.

* fix indentation

* fix pre-comiit errors

* Update doc

* Update copyright

* Add test case for "(?-i)", which disables regex case-insensitive mode.

* fix pre-commit

* Name each test. Remove support of disabling --http-header-forward-pattern case-insensitive mode on http python client.

* Update .md file.

* fix typo

* Reformat args.

* Fix pre-commit

* Fix test name issue.

* Fix pre-commit.

* Update md file and copyright.
* Update README and versions for 2.43.0 / 24.02

* Update Dockefile to reduce image size.

* Update path in patch file for model generation

Update README.md post-24.02
* patching git repository parameterization from production branch 1

* Fix go package directory name

* pre-commit fixes

* pre-commit fixes

---------

Co-authored-by: kyle <[email protected]>
* Enhance bound check for shm offset

* Add test for enhance bound check for shm offset

* Fix off by 1 on max offset

* Improve comments

* Improve comment and offset

* Separate logic between computation and validation
…6017)

* Allow non-decoupled model to send response and FINAL flag separately

* Update copyright

* Defer sending error until FINAL flag is seen to avoid invalid reference

* Move timestamp capture location

* Delay time-point of response complete timestamp in GPRC and SageMaker endpoint

* Move location of RESPONSE_COMPLETE timestamp capture to better align with the meaning.
Added a test case to check for optional/required input params in a request and appropriate response from server.
Includes addition of 3 simple models with a combination of required/optional input params
Add flag to enable compile of OpenAI support in PA
* Test Correlation Id string support for BLS
* Add AsyncIO HTTP compression test

* Improve command line option handling
* Update Docerkfile to install genai

* Change the installation script

* install both build and hatch

* Update name

---------

Co-authored-by: Elias Bermudez <[email protected]>
* Added TRITONSERVER_InferenceTraceSetContext logic
…odes (#6992)

* Add documentation for mapping between Triton Errors and HTTP status codes

* formatting

* Update README.md
@kthui kthui changed the base branch from main to r25.01 January 14, 2025 22:36
@kthui kthui marked this pull request as ready for review January 14, 2025 22:37
@kthui kthui requested review from statiraju and mc-nv January 14, 2025 22:38
@pvijayakrish pvijayakrish force-pushed the jacky-lifecycle-shutdown-r25.01 branch from ec108a2 to 6a03df8 Compare January 15, 2025 17:13
Copy link
Contributor

@rmccorm4 rmccorm4 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think Pavithra's force push broke the history - can you make a fresh branch/PR?

Copy link
Contributor

@mc-nv mc-nv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

update the branch, use r25.01

@mc-nv
Copy link
Contributor

mc-nv commented Jan 17, 2025

Closing this PR as it contains history rewrite.
Please provide new PR or reopen current after branch update.

@kthui
Copy link
Contributor Author

kthui commented Jan 17, 2025

GitHub refused to reopen this PR stating the "branch was force-pushed or recreated".

Opened a new PR for the pick: #7951

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

Successfully merging this pull request may close these issues.