| 15 Nov 2024 |
 John Ericson | Mindavi: oh you can be sure I won't cause a CA regression haha | 20:03:56 |
| John Ericson | In reply to @janne.hess:helsinki-systems.de
So it can only go up from here are you saying it is worth merging? because no one wants to maintain hydra-eval-jobs? | 20:04:55 |
 Rick (Mindavi) | I would think it's a good thing to use nix-eval-jobs in hydra instead of trying to maintain in the hydra repo | 20:07:26 |
| John Ericson | Mindavi: I think we all agree, I just don't wanna merge my own PR | 20:08:58 |
 das_j | In reply to @Ericson2314:matrix.org
@janne.hess:helsinki-systems.de: maybe I saw those words twice but didn't understand them I'm sorry, not sure what was wrong with my grammar. The hydra component lacks maintainership which makes a switch to a maintained component a good idea. Is this grammar more understandable? | 21:39:30 |
| John Ericson | das_j: gotcha, so you are saying this is definitely a net improvement, right? | 21:40:06 |
| John Ericson | if you like it, feel free to hit the merge button | 21:40:42 |
| John Ericson | I can inform the rest of the SC but I would not block a decision on this | 21:40:58 |
| das_j | In reply to @Ericson2314:matrix.org
das_j: gotcha, so you are saying this is definitely a net improvement, right? Pretty much, apart from the fact that it's nix-community. Which is not something I have a personal issue with but I can imagine other people do | 21:41:31 |
| John Ericson | I say better to ask for forgiveness than permission :) | 21:41:54 |
| das_j | In reply to @Ericson2314:matrix.org
if you like it, feel free to hit the merge button I will take a look this weekend I hope. If I don't, feel free to ping and annoy me again because I likely forgot ;) I'll do my best reviewing the change | 21:42:10 |
| John Ericson | OK thanks! | 21:42:19 |
| John Ericson | all the code is due to the lix people, who are already using this | 21:42:29 |
| das_j | In reply to @Ericson2314:matrix.org
OK thanks! No, thank you for upstreaming all that work | 21:42:41 |
| John Ericson | I just changed nix-eval-jobs (already merged) and stuff in the flake | 21:42:43 |
| John Ericson | so feel free to give it a quick and dirty review | 21:42:56 |
| John Ericson | since the code is already in production....somewhere | 21:43:02 |
| John Ericson | frankly, I don't mind if you don't look at the code at all | 21:43:48 |
| John Ericson | I just want to not merge it myself since you brought up the nix-comunity issue, want to make sure no one reads that as me merging over your objection | 21:44:14 |
| das_j | In reply to @Ericson2314:matrix.org
frankly, I don't mind if you don't look at the code at all I'd feel bad if I didn't tbh. That's something I would expect someone in my role to do | 21:44:23 |
| John Ericson | (if one just glances, they might not see you are raising a concern just to dismiss it) | 21:44:28 |
| John Ericson | OK fair enough :) | 21:44:34 |
| John Ericson | I won't tell you to not hold yourself to your own high standard :) | 21:44:46 |
| das_j | In reply to @Ericson2314:matrix.org
I won't tell you to not hold yourself to your own high standard :) Just trying to give something back :) | 21:45:18 |
| John Ericson | :) | 21:45:26 |
 hexa | The machine status page shows all active machines twice 🙃 | 21:45:49 |
| hexa |
|
