~bigbes/tarantool-etcd · parade

main · last commit 1 month ago · 2npkec0r

← Back to the parade

tarantool-etcd-hzu Watch on a future revision gets progress notifies below start_revision Lined Up

status: open P2 bug etcd-changelog-sweep
bd update tarantool-etcd-hzu --claim bd close tarantool-etcd-hzu
Created byEugene Blikh
Ownerbigbes@gmail.com
Created2026-07-17T13:20:14Z
Updated2026-07-17T13:20:14Z
Description
etcd suppresses progress notifications for a watcher whose start_revision is ahead of the current revision — watchable_store.go:524, 'if rev < w.startRev { return false }' (PR #20286, 3.6).

We emit the CURRENT header unconditionally on both the periodic path and ProgressRequest (app/etcd/grpc.lua:394-402, grpc.lua:522-533, app/etcd/json.lua:289-298). The wire layers do not even retain start_revision on the watcher handle, so the check cannot be made where the notify is sent.

Impact: a client watching from a future revision sees header.revision values below the revision it asked to start at, which breaks the 'progress notify means I am caught up to here' contract.
Acceptance criteria
A watcher created with start_revision > current gets no progress notify until the store reaches that revision; both wires; test pins it.

Depends on

No outgoing dependencies.

Depended on by

Nothing depends on this issue.

No comments.

  • Eugene Blikh created the issue · 2026-07-17T16:20:14Z
  • Eugene Blikh added label etcd-changelog-sweep · 2026-07-17T16:20:14Z
Stored rows — what this pane was built from, as read
issues 1 row
id tarantool-etcd-hzu
content_hash 14fdd6958ceed40cb4c8fba54490d15898bbb96373552fcfcf59575bef43a86f
title Watch on a future revision gets progress notifies below start_revision
description etcd suppresses progress notifications for a watcher whose start_revision is ahead of the current revision — watchable_store.go:524, 'if rev < w.startRev { return false }' (PR #20286, 3.6). We emit the CURRENT header unconditionally on both the periodic path and ProgressRequest (app/etcd/grpc.lua:394-402, grpc.lua:522-533, app/etcd/json.lua:289-298). The wire layers do not even retain start_revision on the watcher handle, so the check cannot be made where the notify is sent. Impact: a client watching from a future revision sees header.revision values below the revision it asked to start at, which breaks the 'progress notify means I am caught up to here' contract.
design
acceptance_criteria A watcher created with start_revision > current gets no progress notify until the store reaches that revision; both wires; test pins it.
notes
status open
priority 2
issue_type bug
assignee NULL
estimated_minutes NULL
created_at 2026-07-17T13:20:14Z
created_by Eugene Blikh
owner bigbes@gmail.com
updated_at 2026-07-17T13:20:14Z
closed_at NULL
closed_by_session
external_ref NULL
spec_id
compaction_level 0
compacted_at NULL
compacted_at_commit NULL
original_size NULL
sender
ephemeral 0
wisp_type
pinned 0
is_template 0
mol_type
work_type
source_system
metadata <binary>
source_repo
close_reason
event_kind
actor
target
payload
await_type
await_id
timeout_ns 0
waiters
hook_bead
role_bead
agent_state
last_activity NULL
role_type
rig
due_at NULL
defer_until NULL
no_history 0
started_at NULL
is_blocked 0
labels 1 row
issue_id tarantool-etcd-hzu
label etcd-changelog-sweep
events 2 rows
id 019f703c-189d-7445-a7d9-48e66184cde8
issue_id tarantool-etcd-hzu
event_type created
actor Eugene Blikh
old_value
new_value
comment NULL
created_at 2026-07-17T16:20:14Z
id 019f703c-189d-7a9a-9135-8d91cbf26dc1
issue_id tarantool-etcd-hzu
event_type label_added
actor Eugene Blikh
old_value NULL
new_value NULL
comment Added label: etcd-changelog-sweep
created_at 2026-07-17T16:20:14Z