CP-43755: Increase default max threads for PAM from 1 to 8

We have 16 vCPUs at most in Dom0 and we don't want to take them all up with authentication threads, so limit this at 8. Signed-off-by: Edwin Török <[email protected]>
edwintorok · Oct 11, 2023 · 32d67ec · 32d67ec
1 parent b29e67c
commit 32d67ec
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/ocaml/idl/datamodel_pool.ml b/ocaml/idl/datamodel_pool.ml
@@ -1431,7 +1431,7 @@ let t =
             "coordinator_bias"
             "true if bias against pool master when scheduling vms is enabled, \
              false otherwise"
-        ; field ~qualifier:StaticRO ~ty:Int ~default_value:(Some (VInt 1L))
+        ; field ~qualifier:StaticRO ~ty:Int ~default_value:(Some (VInt 8L))
             ~lifecycle:[] "local_auth_max_threads"
             "Maximum number of threads to use for PAM authentication"
         ; field ~qualifier:StaticRO ~ty:Int ~default_value:(Some (VInt 1L))

diff --git a/ocaml/xapi/dbsync_master.ml b/ocaml/xapi/dbsync_master.ml
@@ -52,7 +52,7 @@ let create_pool_record ~__context =
       ~telemetry_next_collection:Xapi_stdext_date.Date.epoch
       ~last_update_sync:Xapi_stdext_date.Date.epoch
       ~update_sync_frequency:`weekly ~update_sync_day:0L
-      ~update_sync_enabled:false ~local_auth_max_threads:1L
+      ~update_sync_enabled:false ~local_auth_max_threads:8L
       ~ext_auth_max_threads:1L
 
 let set_master_ip ~__context =