MIME-Version: 1.0
References: 
 <CAMaW3VgbJmp8BQw_QUshk4JfFhOco--ymhEwjwGRTvftzqpiZg@mail.gmail.com>
 <CAMaW3ViEK3=hkR9bEeaHoNFfos5s_xnNMqdXZUGVqL_k39xL8Q@mail.gmail.com>
 <CABUevEzpjjCPjWzcs+L5CtcLPeCyVTECKP-LScu7tWfgUmnwPQ@mail.gmail.com>
In-Reply-To: 
 <CABUevEzpjjCPjWzcs+L5CtcLPeCyVTECKP-LScu7tWfgUmnwPQ@mail.gmail.com>
From: Akshat Jaimini <destrex271@gmail.com>
Date: Wed, 3 Jan 2024 12:32:58 +0530
Message-ID: 
 <CAMaW3VgM5J4ifSvCL+XZ2Z1hSfLV9368gZKYXH1NUQX5CV53Tw@mail.gmail.com>
Subject: Re: [INFO] Regarding bugs in Testing Harness
To: Magnus Hagander <magnus@hagander.net>
Cc: pgsql-www@lists.postgresql.org
Content-Type: multipart/alternative; boundary="00000000000035fc59060e05353b"
Archived-At: 
 <https://www.postgresql.org/message-id/CAMaW3VgM5J4ifSvCL%2BXZ2Z1hSfLV9368gZKYXH1NUQX5CV53Tw%40mail.gmail.com>
Precedence: bulk

--00000000000035fc59060e05353b
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

Great! The harness is working properly now too.
Thanks!
I'll add another test to check the migration process too so that these
problems are also notified automatically.

Regards,
Akshat Jaimini

On Tue, Jan 2, 2024 at 11:09=E2=80=AFPM Magnus Hagander <magnus@hagander.ne=
t> wrote:

> The whole table account_communityauthlastlogin is managed outside of
> the django ORM, because django doesn't support proper primary keys
> (see commit 9e70a4e0c32f8db0178f05dac4c1fca7b317e7c5).
>
> We don't have anything beyond this on the prod server, but it does
> look like this is just a plain bug in the commit, which breaks it. And
> the testing harness found it!
>
> Fix pushed in fbfe3cfc.
>
> //Magnus
>
>
> On Tue, Jan 2, 2024 at 6:07=E2=80=AFPM Akshat Jaimini <destrex271@gmail.c=
om>
> wrote:
> >
> > Turns out the harness was not able to build the website locally in the
> dev environment. This is due to the recent patch for 'making the foreign
> key from last login table be ON DELETE CASCADE'. As far as I understood
> from the commit message this needs to be handled by postgres instead of
> Django. Have we implemented something on our production server which allo=
ws
> this? If yes then kindly share that so we can get the harness back up and
> running. Currently, it is unable to parse the file
> 'pgweb/account/migrations/0004_cauth_last_login.py' while running
> migrations.
> >
> > Regards,
> > Akshat Jaimini
> >
> > On Tue, Jan 2, 2024 at 1:37=E2=80=AFPM Akshat Jaimini <destrex271@gmail=
.com>
> wrote:
> >>
> >> Hi,
> >> This is to update that recently the accessibility tests of the testing
> harness are not working as intended. Currently looking into the possible
> causes. This might take a few days to get back up and running; The
> functionality tests are running for now.
> >>
> >> Regards,
> >> Akshat Jaimini
>

--00000000000035fc59060e05353b
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr"><div>Great! The harness is working properly now too. <br><=
/div><div>Thanks!</div><div>I&#39;ll add another test to check the migratio=
n process too so that these problems are also notified automatically.<br></=
div><div><br></div><div>Regards,</div><div>Akshat Jaimini<br></div></div><b=
r><div class=3D"gmail_quote"><div dir=3D"ltr" class=3D"gmail_attr">On Tue, =
Jan 2, 2024 at 11:09=E2=80=AFPM Magnus Hagander &lt;<a href=3D"mailto:magnu=
s@hagander.net">magnus@hagander.net</a>&gt; wrote:<br></div><blockquote cla=
ss=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;border-left:1px solid =
rgb(204,204,204);padding-left:1ex">The whole table account_communityauthlas=
tlogin is managed outside of<br>
the django ORM, because django doesn&#39;t support proper primary keys<br>
(see commit 9e70a4e0c32f8db0178f05dac4c1fca7b317e7c5).<br>
<br>
We don&#39;t have anything beyond this on the prod server, but it does<br>
look like this is just a plain bug in the commit, which breaks it. And<br>
the testing harness found it!<br>
<br>
Fix pushed in fbfe3cfc.<br>
<br>
//Magnus<br>
<br>
<br>
On Tue, Jan 2, 2024 at 6:07=E2=80=AFPM Akshat Jaimini &lt;<a href=3D"mailto=
:destrex271@gmail.com" target=3D"_blank">destrex271@gmail.com</a>&gt; wrote=
:<br>
&gt;<br>
&gt; Turns out the harness was not able to build the website locally in the=
 dev environment. This is due to the recent patch for &#39;making the forei=
gn key from last login table be ON DELETE CASCADE&#39;. As far as I underst=
ood from the commit message this needs to be handled by postgres instead of=
 Django. Have we implemented something on our production server which allow=
s this? If yes then kindly share that so we can get the harness back up and=
 running. Currently, it is unable to parse the file &#39;pgweb/account/migr=
ations/0004_cauth_last_login.py&#39; while running migrations.<br>
&gt;<br>
&gt; Regards,<br>
&gt; Akshat Jaimini<br>
&gt;<br>
&gt; On Tue, Jan 2, 2024 at 1:37=E2=80=AFPM Akshat Jaimini &lt;<a href=3D"m=
ailto:destrex271@gmail.com" target=3D"_blank">destrex271@gmail.com</a>&gt; =
wrote:<br>
&gt;&gt;<br>
&gt;&gt; Hi,<br>
&gt;&gt; This is to update that recently the accessibility tests of the tes=
ting harness are not working as intended. Currently looking into the possib=
le causes. This might take a few days to get back up and running; The funct=
ionality tests are running for now.<br>
&gt;&gt;<br>
&gt;&gt; Regards,<br>
&gt;&gt; Akshat Jaimini<br>
</blockquote></div>

--00000000000035fc59060e05353b--