Skip to content

Commit

Permalink
CP-43755: Increase default max threads for PAM from 1 to 8
Browse files Browse the repository at this point in the history
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]>
  • Loading branch information
edwintorok committed Oct 12, 2023
1 parent 8d4ff98 commit d90369d
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion ocaml/idl/datamodel_pool.ml
Original file line number Diff line number Diff line change
Expand Up @@ -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))
Expand Down
2 changes: 1 addition & 1 deletion ocaml/xapi/dbsync_master.ml
Original file line number Diff line number Diff line change
Expand Up @@ -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 =
Expand Down

0 comments on commit d90369d

Please sign in to comment.