!UNVBThoJtlIiVwiDjU:nixos.org

Staging

68 Members
Staging merges | Find currently open staging-next PRs: https://github.com/NixOS/nixpkgs/pulls?q=is%3Apr+sort%3Aupdated-desc+head%3Astaging-next+head%3Astaging-next-21.11+head%3Astaging-next-21.05+is%3Aopen | Reports: https://malob.github.io/nix-review-tools-reports/25 Servers

Load older messages


SenderMessageTime
8 Aug 2022
@winterqt:nixos.devWinter (she/her)it looks like it's only ever applied to arm05:48:38
@winterqt:nixos.devWinter (she/her)(see https://github.com/MagicStack/uvloop/issues/412)05:48:50
@fabianhjr:matrix.orgfabianhjr

eventlet failing on hydra: https://hydra.nixos.org/build/186525852 uvloop no attempt on hydra: https://hydra.nixos.org/build/186601863

05:49:23
@fabianhjr:matrix.orgfabianhjrUnsure about what propagated the issue05:49:44
@fabianhjr:matrix.orgfabianhjreventlet has a new test failure but was a single case so disabled it and didn't investigate further05:50:10
@fabianhjr:matrix.orgfabianhjruvloop i identified the hang while building on my machine and upon reading the comments and noticing where it got stuck I just went and tested if I had the same issue and I did have it05:50:52
@winterqt:nixos.devWinter (she/her)on x86_64?05:51:07
@fabianhjr:matrix.orgfabianhjrYeah05:51:13
@fabianhjr:matrix.orgfabianhjrzen ver. 205:51:18
@fabianhjr:matrix.orgfabianhjrAMD 3800X05:51:25
@fabianhjr:matrix.orgfabianhjrCan be also kernel / ipc related05:51:55
@fabianhjr:matrix.orgfabianhjrReading the upstream issue it seems tricky05:57:12
@fabianhjr:matrix.orgfabianhjr

Our builders wait for 120 minute, but the test suite is really stuck there. I cannot reproduce the issue on a Raspberry Pi 4 (Linux 5.11.14, Python 3.8.9), but it is perfectly reproducible on an APM X-Gene 2 (Linux 4.19.188, Python 3.8.9).

05:57:15
@fabianhjr:matrix.orgfabianhjrMaybe some specific hardware + software that creates a really odd edge case.05:57:45
@fabianhjr:matrix.orgfabianhjr

No luck - all tests passed. And the stack info is not really telling a lot. I think it should be safe to disable in your case, as test_remote_shutdown_receives_trailing_data (tests.test_tcp.Test_AIO_TCPSSL) is particularly testing asyncio not uvloop.

05:59:19
@fabianhjr:matrix.orgfabianhjrAlpine also disabled it: https://github.com/alpinelinux/aports/commit/1cbbc2effee69cb63fe596de5ad4f7c8c719144806:00:06
@winterqt:nixos.devWinter (she/her)Alright, LGTM then, as long as we're not the only ones 😅06:07:40
@winterqt:nixos.devWinter (she/her)
In reply to @fabianhjr:matrix.org
eventlet has a new test failure but was a single case so disabled it and didn't investigate further
i don't see why it would suddenly be failing, no recent changes :/ broke pipe doesn't give us much to work with
06:10:55
@winterqt:nixos.devWinter (she/her)s/broke/broken06:11:09
@vcunat:matrix.orgVladimír Čunát Note... for myself? This needs decision before the next staging-next iteration: https://github.com/NixOS/nixpkgs/pull/185613 10:02:21
@sandro:supersandro.deSandro 🐧the decision must not be the same for staging and stable12:00:48
@winterqt:nixos.devWinter (she/her)
In reply to @vcunat:matrix.org
Note... for myself? This needs decision before the next staging-next iteration: https://github.com/NixOS/nixpkgs/pull/185613
I merged it, it's a zlib issue.
15:19:49
@winterqt:nixos.devWinter (she/her)Not really sure what you want to do with that info though, I don't remember if the original commit is staging-next15:20:31
@winterqt:nixos.devWinter (she/her)it isn't -- what do you mean by that, then?15:21:47
@vcunat:matrix.orgVladimír ČunátIt is not.15:21:47
@winterqt:nixos.devWinter (she/her)oh i see what you mean15:22:06
@winterqt:nixos.devWinter (she/her)the patch to zlib to fix this seems like it would be pretty simple but at the same time i'm only looking at an isolated piece of the code so who knows if that would cause other things to blow up15:22:50
@vcunat:matrix.orgVladimír ČunátI'd hope that something is proposed upstream very soon. (not necessarily by madler)15:25:28
@winterqt:nixos.devWinter (she/her)https://github.com/madler/zlib/pull/688 this makes all the curl tests pass, though who knows what i've missed -- hopefully it gets merged or madler makes a proper fix16:56:29
@sandro:supersandro.deSandro 🐧you should add s description to that17:04:42

There are no newer messages yet.


Back to Room ListRoom Version: 6