Patches waiting for review/integration (pglister, pgarchives, pgweb)

Поиск
Список
Период
Сортировка
От Célestin Matte
Тема Patches waiting for review/integration (pglister, pgarchives, pgweb)
Дата
Msg-id 6fc41ae5-f547-4cbd-a2d5-54ad75e33fe5@cmatte.me
обсуждение исходный текст
Список pgsql-www
Hello,

I've been sending patches for pglister and pgarchives since 2021, and some of them seem to have been overlooked.
Would it be possible to have a feedback on these patches, or pushing the ones already reviewed to master?
Hopefully this list will help sorting through the opened discussions.

- pglister: make organization name generic
https://www.postgresql.org/message-id/cbd18681-4bb0-d4d2-50f8-14ed20e1dddf@cmatte.me
Final patches here: https://www.postgresql.org/message-id/9bea3703-57a6-db28-907a-465fb891a44a%40cmatte.me
Status: General idea was approved, final series of patches did not get a response

- pgarchives: Bugfix: missing ids in pglister_sync
https://www.postgresql.org/message-id/23a04b00-ab59-6460-6ce2-d845f1be666c@cmatte.me
Patch here:
https://www.postgresql.org/message-id/CABUevEyo3T2QY9sEFvthmNd2vYRzUnESWZ%3Di9GaotQsyhuEQww%40mail.gmail.com
Status: Magnus wrote a patch, I tested and approved it, but it was never pushed to master
Possible commit message:
Bugfix: missing ids in pglister_sync
groupid and listid are not explicitly set in pglister_sync's queries, which
causes the script to fail as these fields are NOT NULL and postgresql default
configuration does not auto-fill id fields.

- pglister: Add README.md and INSTALL.md
https://www.postgresql.org/message-id/6f092810-5625-1f47-34f8-3522193a5cc1@cmatte.me
Status: never got a review

- pgarchives: Always load auth, even when using PUBLIC_ARCHIVES=True
https://www.postgresql.org/message-id/bb55b32a-c744-d770-fb7e-4df090175a17@cmatte.me
Final patch here: https://www.postgresql.org/message-id/846c9d77-bfff-19e3-9622-eaad5f9bf7b0%40cmatte.me
Status: New patch sent following review, but never reviewed/integrated

- pgarchives: Add search front-end from pgweb
https://www.postgresql.org/message-id/1d65043e-ef91-77d0-0c2f-2c51af5af1e5@cmatte.me
Status: It was acknowledged that there could be an interface in the archives code, but this patch was never reviewed
(it'sa big patch)
 

- pgarchives: merge schema.sql into Django's model
https://www.postgresql.org/message-id/12eb75f0-3fc2-14f3-0931-4f29e145f182@cmatte.me
Final patches here: https://www.postgresql.org/message-id/a31a7071-7338-4423-a8e5-16d70f19c2c8%40cmatte.me
Status: I've been told that this big patch should be written in place of smaller ones that fixed existing issues. I've
hadthis in production for two years and fixed issues that have arisen. It's working great now. Still waiting for a
review.

- auth.py: make it possible to customize email address in error message
pgweb: https://www.postgresql.org/message-id/79ee1e4d-ef04-414e-a5e0-1732590ff763@cmatte.me
pglister & pgarchives: https://www.postgresql.org/message-id/ae313e50-5ae1-4285-af2b-9d4b48b2f76a%40cmatte.me
Status: Got a response on the first patch, waiting for review/integration for all of them

- pglister: gitlab-ci: bump tool for syntax checking
+ pgarchives: Convert tools/ scripts to python3
https://www.postgresql.org/message-id/152870d5-4443-4a72-aabd-ee3506447926%40cmatte.me
Status: I sent these corrective patches to address comments, but they were not integrated
I also still think that the gitlab CI in the original message would help to quickly detect issues

- pgarchives & pglister: pglister_sync: import lists with subscriber_access set to True
https://www.postgresql.org/message-id/1d060463-e562-7783-decd-b5a7f3c4c06c@cmatte.me
Final patches here: https://www.postgresql.org/message-id/a99938be-c9ad-8d94-0a16-8305e6391f1e%40cmatte.me
Status: I just relaunched the discussion as the patches I sent did not satisfy requirements

Regards,
-- 
Célestin Matte



В списке pgsql-www по дате отправления:

Предыдущее
От: Célestin Matte
Дата:
Сообщение: Re: [PATCHES] pglister: Add README.md and INSTALL.md
Следующее
От: Daniel Gustafsson
Дата:
Сообщение: Re: Add PostgreSQL books