Skip to content

[Build] Fix for DeepEP manylinux pidfd sycall usage#49814

Open
tlrmchlsmth wants to merge 2 commits into
mainfrom
fix/deepep-manylinux-pidfd
Open

[Build] Fix for DeepEP manylinux pidfd sycall usage#49814
tlrmchlsmth wants to merge 2 commits into
mainfrom
fix/deepep-manylinux-pidfd

Conversation

@tlrmchlsmth

Copy link
Copy Markdown
Member

Fix the release pipeline.

I think this comment explains the issue pretty well

    # DeepEPv2 requires Linux 5.6+ at runtime for pidfd_getfd (pidfd_open was
    # added in Linux 5.3), but manylinux headers predate both definitions.
    # DeepEP is built as a separate wheel for the vLLM container image and is
    # not included in the vLLM wheel, so this does not change its manylinux ABI.

tlrmchlsmth and others added 2 commits July 25, 2026 12:19
Co-authored-by: OpenAI Codex <codex@openai.com>
Signed-off-by: Tyler Michael Smith <tlrmchlsmth@gmail.com>
Co-authored-by: OpenAI Codex <codex@openai.com>
Signed-off-by: Tyler Michael Smith <tlrmchlsmth@gmail.com>

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Claude Code Review

This repository is configured for manual code reviews. Comment @claude review for a one-time review, or @claude review always to subscribe this PR to a review on every future push.

Tip: disable this comment in your organization's Code Review settings.

@tlrmchlsmth tlrmchlsmth added the ready ONLY add when PR is ready to merge/full CI is needed label Jul 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready ONLY add when PR is ready to merge/full CI is needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant