Skip to content

feat: Remove Upper-Bound on grpcio Package#892

Draft
whoisj wants to merge 2 commits into
mainfrom
jwyman/upper-bound
Draft

feat: Remove Upper-Bound on grpcio Package#892
whoisj wants to merge 2 commits into
mainfrom
jwyman/upper-bound

Conversation

@whoisj

@whoisj whoisj commented Apr 24, 2026

Copy link
Copy Markdown
Contributor

Removes the upper-bound on the grpcio package set in the gRPC requirements file.

CI Pipeline ID: 49436834

@whoisj whoisj requested review from mc-nv and yinggeh April 24, 2026 20:49
@yinggeh

yinggeh commented Apr 24, 2026

Copy link
Copy Markdown
Contributor

Pipeline ID?

@yinggeh

yinggeh commented Apr 24, 2026

Copy link
Copy Markdown
Contributor

@whoisj

whoisj commented Apr 24, 2026

Copy link
Copy Markdown
Contributor Author

Might also need to update https://github.com/triton-inference-server/server/blob/f7fe8dc84bae9711b0ddefec2b2facab9314392c/Dockerfile.QA#L355-L357 https://github.com/triton-inference-server/server/blob/f7fe8dc84bae9711b0ddefec2b2facab9314392c/Dockerfile.sdk#L87-L88 and more ...

neither of your links resolve, but I'll take a look at the dockerfiles and check. Thanks for pointing those out.

@yinggeh

yinggeh commented Apr 25, 2026

Copy link
Copy Markdown
Contributor

Might also need to update https://github.com/triton-inference-server/server/blob/f7fe8dc84bae9711b0ddefec2b2facab9314392c/Dockerfile.QA#L355-L357 https://github.com/triton-inference-server/server/blob/f7fe8dc84bae9711b0ddefec2b2facab9314392c/Dockerfile.sdk#L87-L88 and more ...

neither of your links resolve, but I'll take a look at the dockerfiles and check. Thanks for pointing those out.

Links fixed

@whoisj whoisj force-pushed the jwyman/upper-bound branch from b2442d8 to ea56a86 Compare June 15, 2026 20:50
@whoisj whoisj force-pushed the jwyman/upper-bound branch from ea56a86 to 33f9ef2 Compare June 26, 2026 17:47
The pkg_resources wheel has been deprecated.
@whoisj whoisj marked this pull request as draft June 29, 2026 17:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants