| 30 Jan 2022 |
 Winter (she/her) | Sandro: so is it fine if I remove it in this case? | 06:40:02 |
 Sandro 🐧 | In reply to @winterqt:nixos.dev
Sandro: so is it fine if I remove it in this case? just ignore the label. You don't need to do anything with it. | 06:40:27 |
| Winter (she/her) | In reply to @sandro:supersandro.de
just ignore the label. You don't need to do anything with it. will do, but why? if it's evaling fine now, why keep it? | 15:02:13 |
| Winter (she/her) | just seems useless to me idk | 15:03:02 |
| Sandro 🐧 | Because it is an indicator which PRs failed and only people with access to ofborg can figure out why it failed. | 15:07:31 |
| Sandro 🐧 | Most of the time those errors have nothing to do with the PR. If so you get a red eval result. More often some GitHub API things have problems. | 15:08:23 |
 hexa | red eval means you broke eval, that's most often the commiters fault, not ofborgs 😀 | 15:25:20 |
| Winter (she/her) | In reply to @sandro:supersandro.de
Because it is an indicator which PRs failed and only people with access to ofborg can figure out why it failed. yes but i never got a red eval, just that label added and an eval didn't start until i pushed again and then everything was fine
so am i keeping it there just because the fact that it failed at one point could be useful to see | 15:41:25 |
| Winter (she/her) | i've seen people remove it when they get it so 🤷♀️ | 15:42:09 |
| Winter (she/her) | actually i guess "people" is cole-h so that makes sense | 15:42:36 |
| hexa | leave it | 15:47:39 |
| hexa | yeah, cole-h is the removal guy 😀 | 15:47:49 |
| hexa | he is important and may do so | 15:47:55 |
 cole-h | In reply to @hexa:lossy.network
he is important and may do so Well... :P | 17:32:34 |
| cole-h | In reply to @winterqt:nixos.dev
yes but i never got a red eval, just that label added and an eval didn't start until i pushed again and then everything was fine
so am i keeping it there just because the fact that it failed at one point could be useful to see FWIW, usually the internal error label is because GitHub sends an API response that, for one reason or another, is missing required fields or is otherwise incomplete | 17:33:45 |
 piegames | The hubcaps library and the GitHub API are slowly diverging | 17:34:18 |
| Winter (she/her) | In reply to @cole-h:matrix.org
FWIW, usually the internal error label is because GitHub sends an API response that, for one reason or another, is missing required fields or is otherwise incomplete figured | 17:35:28 |
| cole-h | In reply to @piegames:matrix.org
The hubcaps library and the GitHub API are slowly diverging I don't doubt that, but it happens not-too-frequently for the volume of PRs we get. Just seems like sometimes their API can't keep up or something... | 17:35:59 |
| cole-h | Given a wealth of free time and, well, wealth, I'd do more investigation... But it hasn't been more harmful than just "you need to restart the eval in one way or another -- no more progress will be made with the current one" | 17:37:04 |
| 2 Feb 2022 |
|  Chinchilla Washington changed their display name from CoilWinder (novus ordo seclorum) to Chuck Winter. | 08:35:51 |
| 13 Feb 2022 |
| hexa | https://github.com/NixOS/nixpkgs/pull/159691 | 21:34:21 |
| hexa | waiting for ofborg for 22h 🙂 | 21:34:25 |
| hexa | probably github webhook shenanigans | 21:35:17 |
| hexa | triggered an eval manually | 21:35:22 |
| 18 Feb 2022 |
| Chinchilla Washington changed their display name from Chuck Winter to Chuck Winter (vi/vim). | 04:11:47 |
| Chinchilla Washington changed their display name from Chuck Winter (vi/vim) to Chuck Winter. | 04:20:14 |
| 20 Feb 2022 |
 Alyssa Ross | cole-h: could you help me figure out why I'm seeing OfBorg fail lib tests on PRs to staging, even though the pass locally? | 01:02:14 |
| Alyssa Ross | e.g. https://github.com/NixOS/nixpkgs/pull/159064 | 01:02:18 |
