agent-substrate / agent-substrate/substrate

TestActorSnapshotLifecycle reads only the first page of ListActorSnapshots

Aberta
#1,126 0 comentários 0 reações 0 responsáveis Ver no GitHub
area/storage area/tests kind/bug
Linguagem predominante
Go
Estrelas
1.8k
Forks
316
Merge médio
2d 43min
PRs com merge (30d)
287

Descrição

`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).

Guia de contribuição

Abrir o guia de contribuição

Avaliação

Esta issue ainda não foi avaliada.

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.