| 16 Sep 2023 |
hexa | TIL: https://github.com/matrix-org/synapse/blob/develop/flake.nix | 17:18:04 |
hexa | created by anoadragon453 | 17:18:38 |
Sumner Evans | Yeah. It's kinda a weird setup for dev tbh. But if you modify a couple things it's usable. | 17:31:45 |
| 18 Sep 2023 |
hexa | worker support 👏 | 17:54:49 |
hexa | thanks everyone | 17:55:09 |
hexa | now, who will manage that nasty nginx vhost for me? 🤡 | 17:55:35 |
ma27 | I'm sure there are at least three people with config for that :> | 17:59:07 |
Dandellion | I think Ma27 seemed quite negative about this generally, But personally I think just putting the maps inside services.nginx.commonHttpConfig is more than adequate even without RFC-42 nginx (Which I think is basically impossible without introducing some kind of DAG system similar to home-manager)
But I'll be changing my module to match the nixpkgs implementation and work towards upstreaming the autoconfig stuff
| 18:01:57 |
hexa | I think it is much more straightforward than nextcloud 😄 | 18:02:44 |
hexa | and maintaining the map in nixpkgs would require a fair amount of testing and attention when updating the package | 18:03:22 |
Dandellion | worst case it just falls back to the main process | 18:03:51 |
Dandellion | it'd be additive in any case really | 18:04:04 |
hexa | Oh yeah, that should work for most cases, except for enable_media_repo = false or send_federation = false | 18:04:43 |
hexa | also stream writers are a bit more complex with the different types they can be configured to handle | 18:05:58 |
Dandellion | stream writers are problematic yep | 18:06:17 |
Dandellion | or, at least, slightly annoying | 18:06:31 |
hexa | events can be striped across multiple workers, most other things (typing, to_device, account_data, receipts, presence) cannot | 18:07:06 |
hexa | synapse will fail to boot when you configure a list where it expects a string, ask me how I know | 18:07:55 |
hexa | * synapse will fail to boot when you configure a list of workers where it expects a just a single one, ask me how I know | 18:08:13 |
Dandellion | that's right, it's also why my module only supports event persisters for now | 18:09:00 |
ma27 | In reply to @hexa:lossy.network synapse will fail to boot when you configure a list of workers where it expects a just a single one, ask me how I know oh, you too :D | 18:09:04 |
ma27 | currently on the phone, will respond later | 18:09:16 |
Dandellion | since abstracting the other stream writers required a little more work at the time | 18:09:33 |
Dandellion | * since abstracting the other stream writers properly required a little more work at the time | 18:09:45 |
hexa | as the worker setup is very brittle, we should probably try to co-maintain most of the thing | 18:12:06 |
hexa | * as the worker setup is very brittle, we should probably try to co-maintain most of the code | 18:12:12 |
hexa | * as the worker setup is very brittle, we should probably try to co-maintain most of the code required for a productive worker setup | 18:12:43 |
hexa | and not let everyone come up with a weird downstream solution | 18:12:50 |
hexa | * and not let everyone come up with an even weirder downstream solution | 18:13:07 |
f0x | In reply to @hexa:lossy.network and not let everyone come up with an even weirder downstream solution ha ha sweats | 18:14:42 |