| 9 Nov 2023 |
 @linus:schreibt.jetzt | mais on sait pas exactement ce qui s'est passé | 08:23:52 |
 Ambroisie | Est-ce qu'il y en a ici qui s'y connaisse en admin de DB (postgres), j'aimerais avoir un retour sur ce commentaire: https://github.com/NixOS/nixpkgs/pull/266469#discussion_r1388343321 | 17:32:02 |
 raitobezarius | ah oui c'est un peu vnr mdr | 17:46:04 |
| raitobezarius | pourquoi django-allauth veut tout ça? | 17:46:16 |
| raitobezarius | et pas juste un sous ensemble? | 17:46:18 |
| Ambroisie | Nan c'est tous les systèmes de migration, c'est lié à l'update de Postgres | 17:47:05 |
| raitobezarius | Non mais dans ce cas là, je pense pas que ça soit le bon fix | 17:47:30 |
| Ambroisie | Je suis en train de travailler sur un test pour Woodpecker, y'a le même problème | 17:47:32 |
| raitobezarius | L'update de PG16 on est en train de la gérer dans… | 17:47:38 |
| raitobezarius | https://github.com/NixOS/nixpkgs/pull/266270#event-10900878449 | 17:47:49 |
| raitobezarius | Si tu veux filer un coup de main, faut aller sur ce que Max est en train de faire | 17:48:04 |
| Ambroisie | Ah nice | 17:49:00 |
| Ambroisie | J'avais pas vu ce draft, ça a l'air bien | 17:49:11 |
| raitobezarius | je suis techniquement demi-mainteneur PG dans nixpkgs pour information | 17:49:34 |
| raitobezarius | si jamais tu as besoin | 17:49:43 |
| Ambroisie | Ça serait quoi la meilleure solution pour avoir le fix de mon package qui soit merge, même si le test est cassé à cause de Postgres
Je peux le faire utiliser SQLite en attendant, ça paraît être le mieux d'un point de vue "ne pas montrer d'option insecure à un utilisateur" | 17:50:40 |
 Minijackson | In reply to @ambroisie:belanyi.fr
Ça serait quoi la meilleure solution pour avoir le fix de mon package qui soit merge, même si le test est cassé à cause de Postgres
Je peux le faire utiliser SQLite en attendant, ça paraît être le mieux d'un point de vue "ne pas montrer d'option insecure à un utilisateur" Est-ce que les données seront automatiquement migrées de Postgresql vers SQLite ? Ça pourrait impacter des utilisateurs sinon | 18:00:49 |
| Ambroisie | C'est uniquement pour le test, le module ne touche pas aux options DB de l'utilisateur | 18:01:20 |
| raitobezarius | utiliser SQLite me paraît mieux pour le moment mais bon c'est plus vraiment le même test quoi | 18:01:56 |
| Ambroisie | Ouais c'est pas la DB recommandée, mais pour une installation perso'/le test isok | 18:03:15 |
| Ambroisie | raitobezarius: je vois d'autres MRs qui commencent à pop-up à propos de l'issue, faudrait peut-être faire quelque chose pour éviter de merge des chose pas secure
Exemple: https://github.com/NixOS/nixpkgs/pull/266511 | 21:44:50 |
| raitobezarius | je remonte ça | 21:45:09 |
| Ambroisie | Et sinon je suis plutôt d'accord avec une autre remarque que j'ai vu sur une MR: le choix de bump la version de Postgres pour la prochaine release a peut-être été fait un peu tard
Est-ce qu'il serait pas mieux dans le futur de le bump le plus tôt possible après le branch-off, comme ça on a un peu de temps de soak sur unstable ? | 22:11:12 |
