Skip to content

vshard: bump to 0.1.42 and cover backup of a sharded cluster - #1338

Merged
bigbes merged 2 commits into
bigbes/tntp-8670-backup-verifyfrom
bigbes/gh-no-vshard-update
Jul 31, 2026
Merged

vshard: bump to 0.1.42 and cover backup of a sharded cluster#1338
bigbes merged 2 commits into
bigbes/tntp-8670-backup-verifyfrom
bigbes/gh-no-vshard-update

Conversation

@bigbes

@bigbes bigbes commented Jul 30, 2026

Copy link
Copy Markdown
Collaborator

Every vshard pin in the tree still pointed at 0.1.25: the built-in vshard_cluster template and the three test rockspecs. They move to 0.1.42, which is the first release shipping roles/recovery-point-manager/backends/vshard-router.lua - the backend that lets the core roles.recovery-point-manager role take a cluster-wide recovery point through the router.

Part of TNTP-8032

@bigbes
bigbes force-pushed the bigbes/gh-no-vshard-update branch from 7e86de2 to f43a207 Compare July 30, 2026 06:14
@bigbes
bigbes changed the base branch from g.rybakov/tntp-8608-start-rework-tests to bigbes/tntp-8670-backup-verify July 30, 2026 06:14
@bigbes
bigbes force-pushed the bigbes/gh-no-vshard-update branch from f43a207 to 2135df6 Compare July 30, 2026 06:29
@bigbes
bigbes force-pushed the bigbes/gh-no-vshard-update branch from 2135df6 to a4cc7f7 Compare July 30, 2026 12:42
@bigbes
bigbes force-pushed the bigbes/gh-no-vshard-update branch 2 times, most recently from 8ebe6de to d37c671 Compare July 30, 2026 12:52
@bigbes
bigbes force-pushed the bigbes/gh-no-vshard-update branch from d37c671 to ef76794 Compare July 30, 2026 19:15
@bigbes
bigbes force-pushed the bigbes/gh-no-vshard-update branch from ef76794 to cacc0a6 Compare July 30, 2026 19:23
@bigbes
bigbes force-pushed the bigbes/gh-no-vshard-update branch from cacc0a6 to 4662da2 Compare July 30, 2026 19:56
@bigbes
bigbes force-pushed the bigbes/gh-no-vshard-update branch from 4662da2 to c69d30d Compare July 31, 2026 06:19
@bigbes bigbes mentioned this pull request Jul 31, 2026
@bigbes
bigbes force-pushed the bigbes/gh-no-vshard-update branch from c69d30d to a389bd9 Compare July 31, 2026 07:48
@bigbes
bigbes force-pushed the bigbes/gh-no-vshard-update branch from a389bd9 to 80e5d53 Compare July 31, 2026 08:52
bigbes added 2 commits July 31, 2026 13:24
Every vshard pin in the tree still pointed at 0.1.25: the built-in
vshard_cluster template and the three test rockspecs.

0.1.42 is the first release that ships
roles/recovery-point-manager/backends/vshard-router.lua, the backend
that lets the Tarantool recovery point manager role take a cluster-wide
recovery point via the router. Backup needs that to record a consistent
point across shards, so the pin has to move to at least 0.1.42.

Part of TNTP-8032
The backup tests so far ran against a single plain instance, so nothing
exercised the multi-shard artifact layout that the manifest aggregation
consumes, and recovery points were only ever created per instance.

Add a sharded_app fixture: two storage replicasets plus a router that
runs roles.recovery-point-manager with the vshard-router backend from
vshard 0.1.42. The tests assert the role picked that backend up, that a
cluster recovery point created on the router reaches both storage
masters, and that 'tt backup start' carries it into the manifest
fragment of every shard.

The app advertises over TCP because vshard needs a real advertise URI,
which is also why the storages are addressed by <URI> rather than
<APP:INSTANCE>. Unlike test_backup_start.py these tests do not compare
against golden fragments: on a sharded cluster the vclocks move on
their own, so the assertions are structural.

Part of TNTP-8032
@bigbes
bigbes force-pushed the bigbes/gh-no-vshard-update branch from 80e5d53 to bfccd2b Compare July 31, 2026 10:25
@bigbes
bigbes merged commit adcfe08 into master Jul 31, 2026
25 checks passed
@bigbes
bigbes deleted the bigbes/gh-no-vshard-update branch July 31, 2026 11:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants