agent-substrate / agent-substrate/substrate

TestActorSnapshotLifecycle reads only the first page of ListActorSnapshots

未關閉
#1,126 0 則留言 0 個 reaction 已指派 0 人 在 GitHub 檢視
area/storage area/tests kind/bug
主要語言
Go
星號
1.8k
分支
316
平均合併
2 天 43 分鐘
30 天內合併 PR
287

描述

`TestActorSnapshotLifecycle` fails with `snapshot %q missing from ListActorSnapshots` on a cluster that has been up a while.

The snapshot is there. `GetActorSnapshot` succeeds on it one line earlier. The test then calls `ListActorSnapshots` once and treats the first page as the whole list, but the list is name-ordered and the API says there is more in `NextPageToken`. On the cluster where this reproduced, the newly created snapshot sorted 1408th of 1518 against a page size of 1000 — page two.

`createActor` reads `ListActors` the same way.

CI never sees it: every run gets a fresh cluster, and the `demo` atespace stays far below one page. It bites long-lived and shared clusters, and it cost time during the #1050 investigation by looking like a real snapshot loss.

Measured on the cluster where it reproduced: 15/35 runs failed before, 0/35 after following the page tokens (Fisher one-sided p = 4.5e-6).

貢獻指南

開啟貢獻指南

評估

這個 Issue 還沒有評估資料。

把新 issue 寄到你的電子郵件信箱

精選適合新手參與的 GitHub issue 摘要。