| From: | Alexander Lakhin <exclusion(at)gmail(dot)com> |
|---|---|
| To: | Daniel Gustafsson <daniel(at)yesql(dot)se> |
| Cc: | Andres Freund <andres(at)anarazel(dot)de>, Heikki Linnakangas <hlinnaka(at)iki(dot)fi>, Tomas Vondra <tomas(at)vondra(dot)me>, Bernd Helmle <mailings(at)oopsware(dot)de>, Michael Paquier <michael(at)paquier(dot)xyz>, Michael Banck <mbanck(at)gmx(dot)net>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
| Subject: | Re: Changing the state of data checksums in a running cluster |
| Date: | 2026-08-29 05:00:00 |
| Message-ID: | a14e16d3-a9bb-4bd1-9501-a66b906048b4@gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hello Daniel,
28.08.2026 22:07, Daniel Gustafsson пишет:
> I'll have a go at reproducing the error when I am back in the office, but off
> the cuff I would expect the below to fix it. The state transition to off
> didn't wait for completion which can lead to that timing error in reading back
> the state.
>
> -disable_data_checksums($node);
> +disable_data_checksums($node, wait => 1);
Yes, this change makes the test pass (50 iterations) reliably for me.
Thank you for paying attention to it!
> Interesting, I'll have a closer look when back but test 15 in that file doesn't
> test for off at all so the reference to 15 in the above must be a red herring?
> Which test was it according to the not ok line in the log?
It's the same case 3:
[07:31:26.623](0.102s) not ok 3 - ensure checksums are set to off on injection_node
[07:31:26.630](0.008s)
[07:31:26.630](0.000s) # Failed test 'ensure checksums are set to off on injection_node'
# at /pgtest/postgresql.git/src/test/modules/test_checksums/t/DataChecksums/Utils.pm line 69.
[07:31:26.631](0.000s) # got: 'inprogress-on'
# expected: 'off'
....
[07:31:37.327](0.002s) # Looks like you failed 1 test of 17.
Best regards,
Alexander
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Bharath Rupireddy | 2026-08-29 05:09:00 | Re: Online enable/disable data checksums functions return success even when the launcher fails to start |
| Previous Message | shihao zhong | 2026-08-29 04:57:18 | Re: Allow tuple visibility checks without hint-bit, maintenance |