| 9 Feb 2022 |
 @grahamc:nixos.org | hmmm | 15:15:44 |
| @grahamc:nixos.org | like the query is still running, or? | 15:15:51 |
| @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 | 15:16:19 |
 Amanda (she/her) | grahamc (he/him): Yes, I think that hydra-queue-runner might be leaking the connection. There's 10-20 connections all started at the same time, then never closed | 15:17:27 |
| @grahamc:nixos.org | if the COMMIT is still running then it would not be a leaked connection, it is waiting for postgres to finish | 15:17:50 |
| Amanda (she/her) | pg_stat_actibity has it listed as idle state | 15:18:07 |
| @grahamc:nixos.org | gotcha | 15:18:12 |
| 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 |
