Re: CFbot does not recognize patch contents

From: Tatsuo Ishii <ishii(at)sraoss(dot)co(dot)jp>
To: thomas(dot)munro(at)gmail(dot)com
Cc: byavuz81(at)gmail(dot)com, pgsql-hackers(at)postgresql(dot)org
Subject: Re: CFbot does not recognize patch contents
Date: 2024-05-15 04:52:42
Message-ID: 20240515.135242.564198775252389130.t-ishii@sranhm.sra.co.jp
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi Thomas,

> This obviously fixed itself automatically soon after this message, but
> I figured out what happened: I had not actually fixed that referenced
> bug in cfbot :-(. It was checking for HTTP error codes correctly in
> the place that reads emails from the archives, but not the place that
> downloads patches, so in this case I think when it tried to follow the
> link[1] to download the patch, I guess it must have pulled down a
> transient Varnish error message (I don't know what, I don't store it
> anywhere), and tried to apply that as a patch. Oops. Fixed[2].
>
> [1] https://www.postgresql.org/message-id/attachment/160138/v18-0001-Row-pattern-recognition-patch-for-raw-parser.patch
> [2] https://github.com/macdice/cfbot/commit/ec33a65a877a88befc29ea220e87b98c89b27dcc

Thank you for looking into this. I understand the situation. BTW I
have just posted a v19 patch [1] and cfbot took care of it nicely.

[1] https://www.postgresql.org/message-id/20240515.090203.2255390780622503596.t-ishii%40sranhm.sra.co.jp
--
Tatsuo Ishii
SRA OSS LLC
English: http://www.sraoss.co.jp/index_en/
Japanese:http://www.sraoss.co.jp

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Peter Smith 2024-05-15 05:02:36 Re: Slow catchup of 2PC (twophase) transactions on replica in LR
Previous Message David Rowley 2024-05-15 04:44:32 Re: explain format json, unit for serialize and memory are different.