CFBot has failures on 027_stream_regress for a number of patches

Started by Bruce Momjianabout 4 years ago2 messageshackers
Jump to latest
#1Bruce Momjian
bruce@momjian.us

I notice a number of patches have a failure in the cfbot on the
027_stream_regress test. I think these are related to a bug in that
test being discussed in a thread somewhere though I don't have it
handy. Is that right?

I think it doesn't indicate anything wrong with the individual patches right?

For an example the "Add checkpoint and redo LSN to LogCheckpointEnd
log message" patch which is a fairly simple patch adding a few details
to some log messages is failing with this which doesn't make much
sense since it shouldn't be affecting the actual recovery at all:

diff -U3 /tmp/cirrus-ci-build/src/test/regress/expected/cluster.out
/tmp/cirrus-ci-build/src/test/recovery/tmp_check/results/cluster.out
--- /tmp/cirrus-ci-build/src/test/regress/expected/cluster.out
2022-03-28 01:18:36.126774178 +0000
+++ /tmp/cirrus-ci-build/src/test/recovery/tmp_check/results/cluster.out
2022-03-28 01:23:24.489517050 +0000
@@ -467,7 +467,8 @@
 where row(hundred, thousand, tenthous) <= row(lhundred, lthousand, ltenthous);
  hundred | lhundred | thousand | lthousand | tenthous | ltenthous
 ---------+----------+----------+-----------+----------+-----------
-(0 rows)
+       0 |       99 |        0 |       999 |        0 |      9999
+(1 row)

reset enable_indexscan;
reset maintenance_work_mem;

--
greg

#2Andres Freund
andres@anarazel.de
In reply to: Bruce Momjian (#1)
Re: CFBot has failures on 027_stream_regress for a number of patches

Hi,

On 2022-03-28 15:28:20 -0400, Greg Stark wrote:

I notice a number of patches have a failure in the cfbot on the
027_stream_regress test. I think these are related to a bug in that
test being discussed in a thread somewhere though I don't have it
handy. Is that right?

It looks more like a bug in the general regression tests that are more likely
to be triggered by 027_stream_regress.

I think it doesn't indicate anything wrong with the individual patches right?

Depends on the patch, I think. It certainly also has triggered for bugs in
patches...

For an example the "Add checkpoint and redo LSN to LogCheckpointEnd
log message" patch which is a fairly simple patch adding a few details
to some log messages is failing with this which doesn't make much
sense since it shouldn't be affecting the actual recovery at all:

027_stream_regress.pl runs the normal regression tests via streaming rep - the
failure here is on the primary (i.e. doesn't involve recovery itself). Due to
the smaller shared_buffers setting the test uses, some edge cases are
triggered more often.

diff -U3 /tmp/cirrus-ci-build/src/test/regress/expected/cluster.out
/tmp/cirrus-ci-build/src/test/recovery/tmp_check/results/cluster.out ---
/tmp/cirrus-ci-build/src/test/regress/expected/cluster.out 2022-03-28
01:18:36.126774178 +0000 +++
/tmp/cirrus-ci-build/src/test/recovery/tmp_check/results/cluster.out
2022-03-28 01:23:24.489517050 +0000 @@ -467,7 +467,8 @@ where row(hundred,
thousand, tenthous) <= row(lhundred, lthousand, ltenthous); hundred |
lhundred | thousand | lthousand | tenthous | ltenthous
---------+----------+----------+-----------+----------+----------- -(0 rows)
+ 0 | 99 | 0 | 999 | 0 | 9999 +(1 row)

This one is: /messages/by-id/CA+hUKGLV3wzmYFbNs+TZ1+w0e=hc61fcvrF3OmutuaPBuZMd0w@mail.gmail.com

Greetings,

Andres Freund