| 29 Sep 2023 |
@asymmetric:matrix.dapp.org.uk | seems that:
- it's seconds
- hydra sets a default of 10 hrs?
https://github.com/NixOS/hydra/blob/master/src/hydra-eval-jobs/hydra-eval-jobs.cc#L195
| 09:24:28 |
@asymmetric:matrix.dapp.org.uk | so if i set it to e.g. 2 hours for ofborg i'd actually be lowering relative to the default for hydra? | 09:25:34 |
@asymmetric:matrix.dapp.org.uk | should this be set to the same value as hydra's 36000? https://github.com/NixOS/ofborg/blob/de415d372959b7e6fc6b2f6c95f0c21e5010348d/config.public.json#L124 | 09:26:53 |
@asymmetric:matrix.dapp.org.uk | the package only times out on ofborg, so changing meta.timeout doesn't seem like the cleanest solution: https://hydra.nixos.org/job/nixpkgs/trunk/polkadot.aarch64-darwin | 09:43:40 |
@asymmetric:matrix.dapp.org.uk | i wonder if alternatively we could tell ofborg not to build the package on certain archs at all. rn it's just wasting precious cycles | 09:45:03 |
@asymmetric:matrix.dapp.org.uk | * i wonder if alternatively we could tell ofborg not to build the package on certain arches at all. rn it's just wasting precious cycles | 09:45:12 |
raitobezarius | badPlatforms ? | 12:57:56 |
7c6f434c | The question seems to be distinguishing «broken there», «don't build on Hydra for resource reasons including space» and «will timeout on ofBorg» | 12:59:37 |
Lily Foster | Yeah there's a meta attribute for controlling hydra platforms, but there is not for ofborg currently | 13:17:50 |
Lily Foster | I mean I think ofborg does pure eval, so you could try to detect it and those bad platforms in meta.broken 🙈 | 14:02:21 |
Lily Foster | Redacted or Malformed Event | 14:02:25 |
Lily Foster | * I mean I think ofborg does impure eval, so you could try to detect it and those bad platforms in
meta.broken 🙈 | 14:02:31 |
Lily Foster | * I mean I think ofborg does impure eval, so you could try to detect it and those bad platforms in `meta.broken` 🙈 | 14:02:44 |
Lily Foster | In reply to @lily:lily.flowers I mean I think ofborg does impure eval, so you could try to detect it and those bad platforms in meta.broken 🙈 (Pls don't actually do that, I will cry if you do) | 14:03:56 |
7c6f434c | I don't think ofBorg and normal impure eval are distinguishable, and we don't want to mark the package broken for normal use. | 14:07:11 |
Lily Foster | In reply to @7c6f434c:nitro.chat I don't think ofBorg and normal impure eval are distinguishable, and we don't want to mark the package broken for normal use. Well if it's impure, you could use builtins.readFile and detect hostnames or the ofborg runner executable and such 😉 | 14:30:18 |
7c6f434c | I wonder if this will be an error-out during pure eval, though | 14:33:22 |
Lily Foster | (But again please never do that, everything about it is so cursed and frail 😭) | 14:33:25 |
Lily Foster | In reply to @7c6f434c:nitro.chat I wonder if this will be an error-out during pure eval, though lib.isPureEval | 14:33:37 |
Lily Foster | * `lib.inPureEvalMode` | 14:34:49 |
7c6f434c | Fortunately existence of stable branches would make tying to details of ofBorg deployments just too painful and impractical… | 14:38:34 |
Lily Foster | Stables branches or not, it's a bad and frail idea | 14:41:41 |
Lily Foster | Please don't seriously consider this in any capacity.... | 14:42:19 |
cole-h | My problem with increasing the ofborg timeout on darwin is that the darwin builders are pretty slow as it is. I worry that that would cause the darwin queue to blow up (as has happened in the past even without a longer timeout).
It would be interesting to explore an "ofborgWillTimeoutOnTheseSystems" predicate, though. | 15:40:09 |
@infinisil:matrix.org | cole-h: What about a dynamic approach: When the queue is too long, time out the longest-running job until it's short enough again | 15:58:15 |
@asymmetric:matrix.dapp.org.uk | could we add a command to ofborg, so that one could do @ofborg set timeout 2h or something, on a case by case basis, as a github comment? this would at least be a stopgap, and would mean we don't have to "pollute" meta with ofborg-specific attributes | 15:59:46 |
@asymmetric:matrix.dapp.org.uk | * could we add another command to ofborg, so that one could do @ofborg set timeout 2h or something, on a case by case basis, as a github comment? this would at least be a stopgap, and would mean we don't have to "pollute" meta with ofborg-specific attributes | 15:59:54 |
@asymmetric:matrix.dapp.org.uk | * could we add another command to ofborg, so that one could do @ofborg set timeout 2h aarch64-darwin or something, on a case by case basis, as a github comment? this would at least be a stopgap, and would mean we don't have to "pollute" meta with ofborg-specific attributes | 16:00:06 |
@asymmetric:matrix.dapp.org.uk | * could we add another command to ofborg, so that one could do @ofborg set timeout 2h aarch64-darwin or something, on a case by case basis, as a github comment on the pr of a specific package? this would at least be a stopgap, and would mean we don't have to "pollute" meta with ofborg-specific attributes | 16:00:24 |
@asymmetric:matrix.dapp.org.uk | * could we add another command to ofborg, so that one could do @ofborg set timeout 2h polkadot aarch64-darwin or something, on a case by case basis, as a github comment on the pr of a specific package? this would at least be a stopgap, and would mean we don't have to "pollute" meta with ofborg-specific attributes | 16:00:31 |