Skip to content

Conversation

@amathewc
Copy link

@amathewc amathewc commented Dec 4, 2025

No description provided.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the CI test configuration to use a different libfabric-based NIXL branch and PR. The changes switch from the stable v0.6.0_OFI branch to the libfabric branch with PR#3, and updates the backend environment variable accordingly.

  • Updates NIXL repository branch from v0.6.0_OFI to libfabric with PR#3
  • Changes NIXL backend configuration from OFI to LIBFABRIC
  • Adds additional git commands to fetch and checkout a specific pull request

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.


# pd_disaggregate_nixl_libfabric
run_pd_disaggregate_nixl_libfabric_test() {
echo "[CI Test]: clone libfabric based NIXL along with PR3and run PD disaggregate test."
Copy link

Copilot AI Dec 4, 2025

Choose a reason for hiding this comment

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

Missing space between 'PR3' and 'and'. Should be 'PR3 and'.

Suggested change
echo "[CI Test]: clone libfabric based NIXL along with PR3and run PD disaggregate test."
echo "[CI Test]: clone libfabric based NIXL along with PR3 and run PD disaggregate test."

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant