Skip to content

test: use flydsl==0.1.6.dev20260508 from devreleases for CI validation#3083

Open
xudoyuan wants to merge 10 commits into
mainfrom
test/flydsl-devrelease-0.1.6
Open

test: use flydsl==0.1.6.dev20260508 from devreleases for CI validation#3083
xudoyuan wants to merge 10 commits into
mainfrom
test/flydsl-devrelease-0.1.6

Conversation

@xudoyuan
Copy link
Copy Markdown
Contributor

@xudoyuan xudoyuan commented May 8, 2026

Motivation

Just for test

Technical Details

Test Plan

Test Result

Submission Checklist

Co-Authored-By: Claude Opus 4 <noreply@anthropic.com>
@xudoyuan xudoyuan requested a review from a team May 8, 2026 06:37
@xudoyuan xudoyuan added the ci:all label May 8, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 8, 2026

🏷️ CI Guide

Runs automatically on every PR:

  • ✅ Pre-checks (submodule verification, code formatting)
  • ✅ Aiter op tests (gfx942 + gfx950)
  • ✅ Triton tests on MI35X (only when aiter/ops/triton/** or related paths are changed)

Extended tests (opt-in via labels):

Label Tests
ci:triton-300x Run an additional Triton test job on MI300X in PRs; main branch always runs both MI35X and MI300X
ci:sglang SGLang integration tests
ci:atom ATOM benchmark (DeepSeek-R1 + GPT-OSS)
ci:vllm vLLM benchmark
ci:all All of the above

Add labels via the sidebar or gh pr edit 3083 --add-label <label>

xudoyuan and others added 9 commits May 8, 2026 06:42
…lure

pyproject.toml build-system.requires cannot specify --find-links,
so pip build isolation only searches PyPI where 0.1.6.dev20260508
does not exist. Use a relaxed constraint here; setup.py develop
mode will install the exact devrelease version via --find-links.

Co-Authored-By: Claude Opus 4 <noreply@anthropic.com>
install_requires is also resolved from PyPI only. The exact
devrelease version is installed by the develop-mode handler
at the top of setup.py via --find-links.

Co-Authored-By: Claude Opus 4 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4 <noreply@anthropic.com>
Revert the find_spec("pip") workaround. Instead, all CI workflows
now run `pip install -r requirements.txt` first (which has
--find-links pointing to the devreleases index) to ensure
flydsl==0.1.6.dev20260508 is installed, then use
`pip install --no-build-isolation -e .` to skip build isolation.

Co-Authored-By: Claude Opus 4 <noreply@anthropic.com>
The devrelease wheel may have broken metadata (version shows as None
after install, no RECORD file). Use --force-reinstall --no-deps to
bypass uninstall issues, and catch failures since requirements.txt
already handles the flydsl installation in CI.

Co-Authored-By: Claude Opus 4 <noreply@anthropic.com>
Use direct wheel URL instead of --find-links + version specifier to
ensure the exact +05f188b commit is installed, not the +aac7a5e variant.

Co-Authored-By: Claude Opus 4 <noreply@anthropic.com>
The direct wheel URL was hardcoded to cp312 and returned 403 because
wheels live at /whl/gfx942-gfx950/ not /whl/gfx942-gfx950/flydsl/.
Switch back to --find-links (which resolves relative hrefs correctly)
with ==0.1.6.dev20260508+05f188b to pin the exact commit while letting
pip auto-select the correct cpXXX wheel for the runtime Python version.

Co-Authored-By: Claude Opus 4 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant