public inbox for [email protected]  
help / color / mirror / Atom feed
From: Akshat Jaimini <[email protected]>
To: Magnus Hagander <[email protected]>
To: [email protected]
Subject: Re: Permission to allow testing harness to send error reports for pgweb directly to mailing list.
Date: Wed, 4 Oct 2023 01:00:38 +0530
Message-ID: <CAMaW3VgFmQH6Qz_5rE3mmGrSqNXk-0T0z_czufZOnMai2Yo61w@mail.gmail.com> (raw)
In-Reply-To: <CABUevEyiDjSY3iR6V-3EWqRmpgX490uVoxKWzCFXJUD5NOUvKQ@mail.gmail.com>
References: <CAMaW3VhRaUvSi_mR+_th7b=LQ3NZ-=Kg_aqTmAQpRXhC9zoDJg@mail.gmail.com>
	<CABUevEyiDjSY3iR6V-3EWqRmpgX490uVoxKWzCFXJUD5NOUvKQ@mail.gmail.com>

Hello,

> That is, if it finds the same issue on a later run, it must not re-send
the same thing. How does it work in regards to that today?

As per the current flow whenever a new commit is pushed to the pgweb repo,
the tests are executed. If some tests fail, an error report is sent with
the information of all the failed tests. So if that particular issue has
been resolved, the same report won't be sent but if some other commit is
pushed without resolving that particular issue then that particular error
will be reported again.

As far as the previous test runs have been considered, most of the errors
that have been caught are related to broken urls or some other small
problems that are usually fixed with a single commit so I don't think that
a single report would be re-sent that often.

If this functionality is required then I'll be more than happy to add it.
We can add some sort of a log/cache type structure that checks if a given
report with similar content has been sent to the mailing list in a given
time interval, let's say 15 days for now.

Regards,
Akshat Jaimini

On Wed, Oct 4, 2023 at 12:08 AM Magnus Hagander <[email protected]> wrote:

> On Tue, Oct 3, 2023 at 8:24 AM Akshat Jaimini <[email protected]>
> wrote:
> >
> > Hello everyone,
> > The testing harness has been successfull at catching multiple issues in
> pgweb in the past few months. The reports are for now being sent to me via
> email in case of any failure.
> >
> > Is it ok if  I can setup the harness to send the error reports directly
> to the mailing list?
>
>
> Hi!
>
> I think that would be reasonable, *if* it is ensured that a report is
> only sent once. That is, if it finds the same issue on a later run, it
> must not re-send the same thing. How does it work in regards to that
> today?
>
> --
>  Magnus Hagander
>  Me: https://www.hagander.net/
>  Work: https://www.redpill-linpro.com/
>


reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected], [email protected]
  Subject: Re: Permission to allow testing harness to send error reports for pgweb directly to mailing list.
  In-Reply-To: <CAMaW3VgFmQH6Qz_5rE3mmGrSqNXk-0T0z_czufZOnMai2Yo61w@mail.gmail.com>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox