| 9 Feb 2022 |
 Amanda (she/her) | so I'm working under the assumtion that means commit just was the last query ran | 15:18:27 |
 @grahamc:nixos.org | yeah | 15:18:32 |
| @grahamc:nixos.org | it sounds like it started a worker thread with a connection to handle a build and now it is waiting | 15:18:42 |
| Amanda (she/her) | "At the same time" +- some milliseconds, all within the same couple seconds at least | 15:19:32 |
| @grahamc:nixos.org | I am hesitant to call it a leak exactly, because the # of connections is very stable for both h.n.o and ajs124 's, but does seem like it would be good to be able to tune that | 15:22:24 |
| Amanda (she/her) | Just checked hydra-queue-runner, lots of messages like this, stedially increasing: warning: 48 concurrent database updates; PostgreSQL may be stalled | 15:24:28 |
| @grahamc:nixos.org | yeah, there's some tuning that would be good | 15:24:52 |
| @grahamc:nixos.org | I wonder if there are some already | 15:24:56 |
| @grahamc:nixos.org | yes! | 15:25:15 |
| @grahamc:nixos.org | max_db_connections defaults to 128 | 15:25:29 |
| Amanda (she/her) | I'm not sure what postgres defaults to, but I've not changed it | 15:28:17 |
| Amanda (she/her) | ( Nor even how to do so ) | 15:29:44 |
| Amanda (she/her) | max_db_connections seems to have been the knob I needed though, danke | 15:32:45 |
| Amanda (she/her) | the default may need to be tweaked, not sure if the postgres connection limit is global or per-database | 15:33:15 |
 cransom | it's a global setting. | 15:37:24 |
 das_j | In reply to @grahamc:nixos.org
das_j: I'm coming around to test that ldap PR and I am just struggling to get excited about digging in lol No worries ;) | 16:03:24 |
| @grahamc:nixos.org | okay but some worries because I'm supposed to :P | 16:06:28 |
 hexa | hopefully we can soon slap kanidm in there and get rid of slapd 🙂 | 16:08:51 |
| @grahamc:nixos.org | I'm already 20% through copy-pasting ldif files around | 16:12:01 |
| @grahamc:nixos.org | I could abort if kanidm is easy :) | 16:12:09 |
| @grahamc:nixos.org | I was 5% tempted to make subs to create users and groups and instead I think I'll just have the test writer copy paste their own ldif around | 16:13:37 |
| hexa | In reply to @grahamc:nixos.org
I could abort if kanidm is easy :) Good opportunity to test kanidm, it looks simple, but only speaks LDAP to a certain degree | 16:37:30 |
| hexa | There is an open pr that i did a first review on a few days ago | 16:38:18 |
 Arian | Kanidm people are on matrix btw and pretty helpful | 16:40:00 |
| Arian | https://matrix.to/#/!fzbFDgtoaOQXSSxOZc:gitter.im?via=matrix.org&via=gitter.im&via=lossy.network | 16:40:28 |
| @grahamc:nixos.org | I'm already so far in to getting slapd going | 16:45:58 |
| Amanda (she/her) | hydra-queue-runner might need some poking for connections handling. It seems that the max_db_connections as a result also ends up limiting the concurrency of the steps of a build. I had a build stalled for 30m with no new output, I suspect because I set the max_db_connections too low | 16:49:43 |
| Amanda (she/her) | In reply to @amanda:camnet.site
hydra-queue-runner might need some poking for connections handling. It seems that the max_db_connections as a result also ends up limiting the concurrency of the steps of a build. I had a build stalled for 30m with no new output, I suspect because I set the max_db_connections too low In any case, that default should be looked at, judging from the sample config postgres's default is 100 | 17:32:52 |
| @grahamc:nixos.org | das_j: https://github.com/NixOS/hydra/blob/13b6308581a36b52e92221273f533b3844469def/t/Hydra/Controller/User/ldap.t | 19:37:52 |
| das_j | Is it possible that the localStore check is broken? I can see the variable being referenced in templates but not in perl code: [% IF localStore %] | 23:45:25 |
