| Lists: | pgsql-www |
|---|
| From: | Akshat Jaimini <destrex271(at)gmail(dot)com> |
|---|---|
| To: | pgsql-www(at)lists(dot)postgresql(dot)org |
| Subject: | [INFO] Regarding bugs in Testing Harness |
| Date: | 2024-01-02 08:07:25 |
| Message-ID: | CAMaW3VgbJmp8BQw_QUshk4JfFhOco--ymhEwjwGRTvftzqpiZg@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Lists: | pgsql-www |
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
| From: | Akshat Jaimini <destrex271(at)gmail(dot)com> |
|---|---|
| To: | pgsql-www(at)lists(dot)postgresql(dot)org |
| Cc: | Magnus Hagander <magnus(at)hagander(dot)net> |
| Subject: | Re: [INFO] Regarding bugs in Testing Harness |
| Date: | 2024-01-02 17:06:47 |
| Message-ID: | CAMaW3ViEK3=hkR9bEeaHoNFfos5s_xnNMqdXZUGVqL_k39xL8Q@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Lists: | pgsql-www |
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 allows 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 PM Akshat Jaimini <destrex271(at)gmail(dot)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
>
| From: | Magnus Hagander <magnus(at)hagander(dot)net> |
|---|---|
| To: | Akshat Jaimini <destrex271(at)gmail(dot)com> |
| Cc: | pgsql-www(at)lists(dot)postgresql(dot)org |
| Subject: | Re: [INFO] Regarding bugs in Testing Harness |
| Date: | 2024-01-02 17:39:14 |
| Message-ID: | CABUevEzpjjCPjWzcs+L5CtcLPeCyVTECKP-LScu7tWfgUmnwPQ@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Lists: | pgsql-www |
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 PM Akshat Jaimini <destrex271(at)gmail(dot)com> 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 allows 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 PM Akshat Jaimini <destrex271(at)gmail(dot)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
| From: | Akshat Jaimini <destrex271(at)gmail(dot)com> |
|---|---|
| To: | Magnus Hagander <magnus(at)hagander(dot)net> |
| Cc: | pgsql-www(at)lists(dot)postgresql(dot)org |
| Subject: | Re: [INFO] Regarding bugs in Testing Harness |
| Date: | 2024-01-03 07:02:58 |
| Message-ID: | CAMaW3VgM5J4ifSvCL+XZ2Z1hSfLV9368gZKYXH1NUQX5CV53Tw@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Lists: | pgsql-www |
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 PM Magnus Hagander <magnus(at)hagander(dot)net> 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 PM Akshat Jaimini <destrex271(at)gmail(dot)com>
> 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 allows
> 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 PM Akshat Jaimini <destrex271(at)gmail(dot)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
>
| From: | Akshat Jaimini <destrex271(at)gmail(dot)com> |
|---|---|
| To: | Magnus Hagander <magnus(at)hagander(dot)net> |
| Cc: | pgsql-www(at)lists(dot)postgresql(dot)org |
| Subject: | Re: [INFO] Regarding bugs in Testing Harness |
| Date: | 2024-01-05 02:52:40 |
| Message-ID: | CAMaW3Vj-J5GgTmYN4Q_J48c92jEY0=5Wvnf1Rn2gWGRF6wA2sQ@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Lists: | pgsql-www |
Migration Tests added to the harness.
Now it should report any such errors properly without failing silently.
Regards,
Akshat Jaimini
On Wed, 3 Jan, 2024, 12:32 Akshat Jaimini, <destrex271(at)gmail(dot)com> wrote:
> 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 PM Magnus Hagander <magnus(at)hagander(dot)net>
> 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 PM Akshat Jaimini <destrex271(at)gmail(dot)com>
>> 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 allows
>> 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 PM Akshat Jaimini <destrex271(at)gmail(dot)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
>>
>