[Juhyun-Kim-Memphis]

INF-461 에서 넣은 master 의 mass-expiry circuit breaker 를 제거한다 (fork #13 의 breaker 커밋 두 개를 revert). INF-522 (upstream 7faef38b 동기화) 의 첫 단계다.

왜 지우는가

• upstream [Store] Add recoverable client liveness and asynchronous offboarding kvcache-ai/Mooncake#2991 이 breaker 의 입력을 없앤다. breaker 는 ping queue 와 client 별 TTL map, 그리고 client_mutex_ 의 exclusive hold 기록을 근거로 만료를 붙잡았다. [Store] Add recoverable client liveness and asynchronous offboarding kvcache-ai/Mooncake#2991 이후에는 ping queue 와 TTL map 이 사라지고, 되돌릴 수 없는 전이도 client 마다 각자의 liveness record 안에서 일어난다. breaker 가 한곳에서 붙잡을 결정 지점이 없다.
• 근본 원인은 upstream 쪽에서 다룬다. lock 의존은 [Store] Centralize client session lifecycle and resource ownership kvcache-ai/Mooncake#4180 이, 스레드 공유는 [Store] Serve Ping on a separate RPC server that clients discover on Connect kvcache-ai/Mooncake#4183 이 없앤다. [Store] Centralize client session lifecycle and resource ownership kvcache-ai/Mooncake#4180 이후에는 client_mutex_ 자체가 없어서 breaker 의 주 신호가 구조적으로 사라진다.
• [Store] Add recoverable client liveness and asynchronous offboarding kvcache-ai/Mooncake#2991 의 2단계 liveness 가 대체한다. client_ttl=60 기준으로 되돌릴 수 없는 단계가 모든 원인에 대해 약 120초 뒤로 밀린다. breaker 대신 TTL 값을 명시하고, 새 liveness metric (master_client_liveness_{suspected,offline}_clients) 에 알람을 건다.
• breaker 의 upstream 제출분 ([Store] Defer a client mass expiry the master cannot rule out as its own stall kvcache-ai/Mooncake#3936) 은 채택되지 않고 닫혔다.
• sync 전에 따로 지우는 이유: breaker 는 sync 에서 master_service.cpp, master_config.h, master.cpp, types.h, conf/*, metric manager 에 걸쳐 충돌하고, master_service_config_test.cpp 는 자동 병합되지만 breaker 설정을 참조해 빌드가 깨진다. 제거를 따로 떼어 두면 sync PR 의 충돌 해소가 "upstream 쪽을 택한다" 로 단순해진다.

무엇을 지우고 무엇을 남기는가

커밋	처리
8f7f8b5b [Store] Narrow the mass-expiry hold window…	revert
7f5e4ca2 [Store] Hold back a client mass expiry…	revert
68983cbe kvcache-ai#3533 백포트 (ReMountSegment shard walk gate)	유지 — upstream 에 병합됐고 sync 에서 흡수된다
7b40f370, e13f3601, 5c14eb10 [Bench] master_bench	유지

revert 는 충돌 없이 적용됐다. 결과 트리는 7f5e4ca2^ 와 비교해 #14 (parallel populate) 와 #15 (kvcache-ai#3006 백포트) 의 변경만 남는다. --client_mass_expiry_guard, --client_mass_expiry_grace_sec, master_client_expiry_deferred_*, EvaluateClientMassExpiry 등 breaker 참조는 src·tests·conf·docs 어디에도 남지 않는다 (git grep 0건).

운영 영향: 이 PR 은 our_main 만 바꾼다. 운영 master (21527a1f) 는 그대로이고, 이미지는 이후 단계에서 sync 결과로 새로 만든다. 롤아웃은 INF-522 범위 밖이다.

Unit gate

ba914f73 에서 magma oss/build/mooncake/Dockerfile.unittest (INF-522 에서 확장한 버전, master liveness 스위트 포함) 로 돌렸다. PASS — 7/7 스위트, 23분 5초.

스위트	테스트 수	결과
master_service_test	151	Passed (268.7 s)
master_service_config_test	5	Passed
master_service_ssd_test	25	Passed
promotion_on_hit_test	52	Passed
rpc_timeout_test	3	Passed
segment_test	12	Passed
file_storage_test	32	Passed

실행 환경 주의: 원래 gate 는 ganymede 의 BuildKit Job 으로 돌리는데, 지정 노드 w-0529 를 다른 캠페인이 쓰고 있어서 같은 Dockerfile 을 로컬 Docker (linux/amd64, Rosetta) 에서 돌렸다. 로컬 VM 메모리 때문에 MAX_JOBS 만 4 → 3 으로 낮췄고, 나머지는 동일하다. 이 트리에는 upstream liveness 쪽 선택 스위트 (client_liveness_test 등) 가 없어서 건너뛰었다.

Carries: nothing (removes the fork-only mass-expiry breaker carried by #13; adds no delta of ours)