Skip to content

CI: Add FlagGems-vllm checkout for test purpose - #388

Open
HermiaHuan wants to merge 7 commits into
flagos-ai:mainfrom
BrianPei:PR-0818-flaggems-vllm-checkout
Open

CI: Add FlagGems-vllm checkout for test purpose#388
HermiaHuan wants to merge 7 commits into
flagos-ai:mainfrom
BrianPei:PR-0818-flaggems-vllm-checkout

Conversation

@HermiaHuan

Copy link
Copy Markdown
Collaborator

Summary

  • Clone flagos-ai/FlagGems-vllm in each platform setup script
  • Use an absolute path under the parent directory of the current workspace
  • Checkout the main branch before platform-specific setup continues

Test

  • bash -n .github/scripts/ascend/setup.sh .github/scripts/cuda/setup.sh .github/scripts/enflame/setup.sh .github/scripts/hygon/setup.sh .github/scripts/metax/setup.sh .github/scripts/musa/setup.sh
  • git diff --check

@github-actions github-actions Bot added the ci label Aug 18, 2026

print(f"vLLM import ok: {vllm.__version__}")
print(f"vLLM-FL import ok: {vllm_fl.__file__}")
print(f"FlagGems import ok: {getattr(flag_gems, '__version__', 'unknown')}")

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

import flag_gems
import flaggems_vllm

need to check flaggems_vllm

@@ -3,6 +3,10 @@
# Setup script for Ascend NPU CI environment.
set -euo pipefail

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Install FlagGems for test purpose

FLAGGEMS_VLLM_DIR="$(cd .. && pwd -P)/FlagGems-vllm"
git clone https://github.com/flagos-ai/FlagGems-vllm.git "${FLAGGEMS_VLLM_DIR}"
git -C "${FLAGGEMS_VLLM_DIR}" checkout main
pip install --no-build-isolation -e ${FLAGGEMS_VLLM_DIR}

Install vLLM-Plugin-FL

@HermiaHuan HermiaHuan changed the title Add FlagGems-vllm checkout to platform setup scripts [WIP] Add FlagGems-vllm checkout for test purpose Aug 18, 2026
xmhubj
xmhubj previously approved these changes Aug 18, 2026
@xmhubj xmhubj changed the title [WIP] Add FlagGems-vllm checkout for test purpose Add FlagGems-vllm checkout for test purpose Aug 18, 2026
Comment thread .github/scripts/enflame/setup.sh Outdated
FLAGGEMS_VLLM_DIR="$(cd .. && pwd -P)/FlagGems-vllm"
git clone https://github.com/flagos-ai/FlagGems-vllm.git "${FLAGGEMS_VLLM_DIR}"
git -C "${FLAGGEMS_VLLM_DIR}" checkout main
pip install --no-build-isolation -e "${FLAGGEMS_VLLM_DIR}"

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

python -m pip install --no-build-isolation -e "${FLAGGEMS_VLLM_DIR}"

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

it's already changed

Comment thread .github/scripts/hygon/setup.sh Outdated
FLAGGEMS_VLLM_DIR="$(cd .. && pwd -P)/FlagGems-vllm"
git clone https://github.com/flagos-ai/FlagGems-vllm.git "${FLAGGEMS_VLLM_DIR}"
git -C "${FLAGGEMS_VLLM_DIR}" checkout main
pip install --no-build-isolation -e "${FLAGGEMS_VLLM_DIR}"

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

python -m pip install --no-build-isolation -e "${FLAGGEMS_VLLM_DIR}"

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

it's already changed

Comment thread .github/scripts/metax/setup.sh Outdated
FLAGGEMS_VLLM_DIR="$(cd .. && pwd -P)/FlagGems-vllm"
git clone https://github.com/flagos-ai/FlagGems-vllm.git "${FLAGGEMS_VLLM_DIR}"
git -C "${FLAGGEMS_VLLM_DIR}" checkout main
pip install --no-build-isolation -e "${FLAGGEMS_VLLM_DIR}"

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

python -m ?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

it's already changed

Comment thread .github/scripts/musa/setup.sh Outdated
FLAGGEMS_VLLM_DIR="$(cd .. && pwd -P)/FlagGems-vllm"
git clone https://github.com/flagos-ai/FlagGems-vllm.git "${FLAGGEMS_VLLM_DIR}"
git -C "${FLAGGEMS_VLLM_DIR}" checkout main
pip install --no-build-isolation -e "${FLAGGEMS_VLLM_DIR}"

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

python -m ?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

it's already changed

@xmhubj xmhubj changed the title Add FlagGems-vllm checkout for test purpose CI: Add FlagGems-vllm checkout for test purpose Aug 18, 2026
Comment thread .github/scripts/ascend/setup.sh Outdated
FLAGGEMS_VLLM_DIR="$(cd .. && pwd -P)/FlagGems-vllm"
rm -rf "${FLAGGEMS_VLLM_DIR}"
git clone https://github.com/flagos-ai/FlagGems-vllm.git "${FLAGGEMS_VLLM_DIR}"
git -C "${FLAGGEMS_VLLM_DIR}" checkout main

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

git clone --branch ${FLAGGEMS_VERSION} --depth 1 https://github.com/flagos-ai/FlagGems /workspace/FlagGems

Comment thread .github/scripts/ascend/setup.sh Outdated
FLAGGEMS_VLLM_DIR="$(cd .. && pwd -P)/FlagGems-vllm"
rm -rf "${FLAGGEMS_VLLM_DIR}"
git clone https://github.com/flagos-ai/FlagGems-vllm.git "${FLAGGEMS_VLLM_DIR}"
git -C "${FLAGGEMS_VLLM_DIR}" checkout main

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

git clone --branch ${FLAGGEMS_VERSION} --depth 1 https://github.com/flagos-ai/FlagGems /workspace/FlagGems

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.

2 participants