| 8 May 2023 |
 raitobezarius | mais ptet que c'est galère à backport | 17:19:01 |
| raitobezarius | hldmna: si tu peux me faire une PR, je suis chaud pour backport | 17:19:35 |
 hldmna | il y a deux vuln sur 1.36.1 qui est fix avec 1.37 : https://github.com/netdata/netdata/releases/tag/v1.37.0 | 17:20:52 |
| hldmna | In reply to @raitobezarius:matrix.org
hldmna: si tu peux me faire une PR, je suis chaud pour backport Ce sera ma première PR, je vais voir comment faire ^^ | 17:21:38 |
| raitobezarius | si tu peux chopper les patchs | 17:21:51 |
| raitobezarius | plutôt que d'upgrade 1.36.1 -> 1.37.0 | 17:21:57 |
| raitobezarius | ensuite on les applique à netdata 1.36.1 | 17:22:02 |
| raitobezarius | tu ouvres la PR pour la branche release-22.11 | 17:22:20 |
| raitobezarius | écris [Backport] devant ton titre de PR | 17:22:28 |
| raitobezarius | met un tag security | 17:22:35 |
| raitobezarius | et met moi en reviewer | 17:22:38 |
| raitobezarius | ou ping moi | 17:22:43 |
| hldmna | Ok | 17:23:01 |
| hldmna | J'aurai dû regarder avant.., il y a déjà eu une PR pour ça mais elle a été rejeté à cause de breaking change. https://github.com/NixOS/nixpkgs/pull/207270 | 18:35:13 |
| raitobezarius | Winter a été un peu brutal mais oui il faut isoler les patchs | 18:35:38 |
| hldmna | * J'aurai dû regarder avant.., il y a déjà eu une PR pour ça mais elle a été rejeté à cause de breaking change (entre autre). https://github.com/NixOS/nixpkgs/pull/207270 | 18:35:43 |
| raitobezarius | In reply to @raitobezarius:matrix.org
si tu peux chopper les patchs ^ c'est le sens de ce propos | 18:35:52 |
| raitobezarius | Trouve les commits, prend les patchs et applique par dessus et si tu ouvres une telle PR, c'est tout bon ;) | 18:36:22 |
| hldmna | Yep, mais pour ça aussi les gens étaient contre dans la PR | 18:36:25 |
| hldmna | La suggestion était d'ajouter un 'nouveau' paquet, genre netdata_1_37 | 18:37:03 |
| raitobezarius | Non? | 18:37:05 |
| raitobezarius | Non ça c'est pas bien | 18:37:13 |
| raitobezarius | Il faut prendre les PRs qui ont fixé la security vuln | 18:37:21 |
| raitobezarius | Isoler les fix | 18:37:24 |
| raitobezarius | Les appliquer sur la version stable | 18:37:29 |
| raitobezarius | C'est ça qui est la proposition acceptée dans cette issue | 18:37:39 |
| raitobezarius | Y a un warning mais ça on s'en fout on le patch aussi | 18:38:00 |
| raitobezarius | Pour appliquer un patch dans nixpkgs hldmna: | 18:39:14 |
| raitobezarius | Tu peux utiliser `patches` qui prend une liste de chemins | 18:39:25 |
| raitobezarius | (grep nixpkgs pour des exemples c'est assez documenté) | 18:39:41 |
