Skip to content

fix: Fix race condition in model loading/unloading in Sagemaker server - #8571

Merged
yinggeh merged 1 commit into
mainfrom
yinggeh/tri-444-psirt-triton-inference-server-null-pointer
Dec 11, 2025
Merged

fix: Fix race condition in model loading/unloading in Sagemaker server#8571
yinggeh merged 1 commit into
mainfrom
yinggeh/tri-444-psirt-triton-inference-server-null-pointer

Conversation

@yinggeh

@yinggeh yinggeh commented Dec 11, 2025

Copy link
Copy Markdown
Contributor

What does the PR do?

Fix race condition causing segfault in model loading/unloading in Sagemaker server.

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.

  • fix

Related PRs:

Where should the reviewer start?

Test plan:

L0_sagemaker--base

  • CI Pipeline ID:
    40037063

Caveats:

Background

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

  • closes GitHub issue: #xxx

@yinggeh
yinggeh requested review from pskiran1 and whoisj December 11, 2025 16:44
@yinggeh yinggeh self-assigned this Dec 11, 2025
@yinggeh yinggeh added the bug Something isn't working label Dec 11, 2025
@yinggeh
yinggeh merged commit f3bdcc7 into main Dec 11, 2025
3 checks passed
@yinggeh
yinggeh deleted the yinggeh/tri-444-psirt-triton-inference-server-null-pointer branch December 11, 2025 19:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Development

Successfully merging this pull request may close these issues.

2 participants