| 7 Nov 2023 |
raitobezarius | Julien: je peux te laisser driver ^ | 13:15:27 |
raitobezarius | * Julien: je peux te laisser driver ^ ? | 13:15:28 |
| 9 Nov 2023 |
Gaétan Lepage | On dirait qu'hydra est down... Vous savez ce qu'il se passe ? | 08:20:57 |
@linus:schreibt.jetzt | oui, la db marche plus | 08:21:16 |
@linus:schreibt.jetzt | https://matrix.to/#/!RROtHmAaQIkiJzJZZE:nixos.org/$ANsS_0Tz69ySzxgQ8bmszbndvd1r48W5D25ldrV1Qu8?via=nixos.org&via=matrix.org&via=nixos.dev | 08:21:42 |
Gaétan Lepage | Oh ok | 08:23:27 |
@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 |
| 10 Nov 2023 |
Julien | Je crois surtout que la PR a été merged sans le due diligence. Quand on mets à jour ce type de chose il fautse rendre compte que ça casse tout NixOS avant de merger. | 09:58:42 |