neon icon indicating copy to clipboard operation
neon copied to clipboard

Long-running replication tests

Open save-buffer opened this issue 1 year ago • 2 comments

Problem

We need long-running stress tests for replication.

Summary of changes

Add them

Related to neondatabase/cloud#13390, neondatabase/cloud#14490, neondatabase/cloud#14724

save-buffer avatar May 28 '24 15:05 save-buffer

3092 tests run: 2977 passed, 0 failed, 115 skipped (full report)


Flaky tests (3)

Postgres 16

  • test_lsn_lease_size[False]: debug

Postgres 14

  • test_subscriber_restart: release
  • test_lsn_lease_size[False]: debug

Code coverage* (full report)

  • functions: 32.6% (6935 of 21279 functions)
  • lines: 50.0% (54525 of 109034 lines)

* collected from Rust tests only


The comment gets automatically updated with the latest test results
5e109066e79df739d02f60d7b483c2477911d574 at 2024-07-07T09:34:33.528Z :recycle:

github-actions[bot] avatar May 28 '24 17:05 github-actions[bot]

Let's merge this one, then change it to use the same project every time.

Use long-running queries, "start a transaction, query some tables, then use pg_sleep()". Change transaction isolation level in tests.

andreasscherbaum avatar Jun 04 '24 15:06 andreasscherbaum

@save-buffer is OOO for few days, so feel free to merge this

kelvich avatar Jul 01 '24 22:07 kelvich

I'll take this over for Sasha while he is out.

tristan957 avatar Jul 02 '24 20:07 tristan957

I cleaned up the commit history, and I think got to all of the review comments. I also rebased on main. How can we test this to make sure I didn't screw it up touching everything up?

tristan957 avatar Jul 03 '24 20:07 tristan957

I cleaned up the commit history, and I think got to all of the review comments. I also rebased on main. How can we test this to make sure I didn't screw it up touching everything up?

Triggered it: https://github.com/neondatabase/neon/actions/runs/9784944626/job/27016926175

bayandin avatar Jul 03 '24 21:07 bayandin

Fixed bad merge, a couple of missed issues and retriggered it here: https://github.com/neondatabase/neon/actions/runs/9785504530/job/27018662789

UPD: the test run looks good!

bayandin avatar Jul 03 '24 22:07 bayandin

Good work on this everybody!

tristan957 avatar Jul 05 '24 18:07 tristan957

Ugh. @bayandin I just overwrote your commits rebasing on main to get the conclusion job. Can you add them back when you get a chance. I am such an idiot.

tristan957 avatar Jul 05 '24 18:07 tristan957

Can you add them back when you get a chance

No problem! Will do tomorrow (I'm afk for the rest of the day)

bayandin avatar Jul 05 '24 18:07 bayandin

Please enjoy your weekend :)

tristan957 avatar Jul 05 '24 18:07 tristan957