Skip to content

Supercache dev extern mock clean - #10

Open
yyyuanhao426-hash wants to merge 8 commits into
LinQuickDev:supercachefrom
qinyufei63:supercache_dev_extern_mock_clean
Open

Supercache dev extern mock clean#10
yyyuanhao426-hash wants to merge 8 commits into
LinQuickDev:supercachefrom
qinyufei63:supercache_dev_extern_mock_clean

Conversation

@yyyuanhao426-hash

@yyyuanhao426-hash yyyuanhao426-hash commented Jul 16, 2026

Copy link
Copy Markdown

Description

将 UbDiag 从 Mooncake 编译期硬依赖(find_package(REQUIRED))改为三层 CMake 自动回退机制,确保用户无论是否安装 UbDiag 均可编译 Mooncake。

新增 extern/ubdiag git submodule,指向 v0.5.1
新增 mooncake-common/FindUbDiag.cmake,实现 Layer 1 submodule → Layer 2 system → Layer 3 mock 三层分发
新增 mooncake-common/ubdiag-mock/ubdiag/auto_perf.h header-only mock,全签名对齐真实 UbDiag v0.5.1(含模板构造函数、Abandon、析构、PerfLevel:uint8_t)
mooncake-transfer-engine/src/CMakeLists.txt、mooncake-store/src/CMakeLists.txt、mooncake-integration/CMakeLists.txt 三处将 find_package(UbDiag REQUIRED) 替换为 include(FindUbDiag.cmake)
新增 mooncake-common/ubdiag-mock/README.md 集成说明文档

Module

  • Transfer Engine (mooncake-transfer-engine)
  • Mooncake Store (mooncake-store)
  • Mooncake EP (mooncake-ep)
  • Mooncake PG (mooncake-pg)
  • Integration (mooncake-integration)
  • P2P Store (mooncake-p2p-store)
  • Python Wheel (mooncake-wheel)
  • Common (mooncake-common)
  • Mooncake RL (mooncake-rl)
  • CI/CD
  • Docs
  • Other

Type of Change

  • Bug fix
  • New feature
  • Refactor
  • Breaking change
  • Documentation update
  • Performance improvement
  • Other

How Has This Been Tested?

Test commands:

# Example: bash scripts/run_ci_test.sh

Test results:

  • Unit tests pass
  • Integration tests pass (if applicable)
  • Manual testing done (describe below)

Checklist

  • I have performed a self-review of my own code
  • I have formatted my code using ./scripts/code_format.sh
  • I have run pre-commit run --all-files and all hooks pass
  • I have updated the documentation (if applicable)
  • I have added tests to prove my changes are effective
  • For changes >500 LOC: I have filed an RFC issue

AI Assistance Disclosure

  • No AI tools were used
  • AI tools were used (specify below)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants