#1022 backend: drop unused option max_workers
Merged 4 years ago by msuchy. Opened 4 years ago by praiskup.
Unknown source drop-max-workers-config  into  master

@@ -214,9 +214,6 @@

                  "terminate_playbook": _get_conf(

                      cp, "backend", "group{0}_terminate_playbook".format(group_id),

                      default="/srv/copr-work/provision/terminatepb-PC.yml"),

-                 "max_workers": _get_conf(

-                     cp, "backend", "group{0}_max_workers".format(group_id),

-                     default=32, mode="int"),

                  "max_vm_total": _get_conf(

                      cp, "backend", "group{}_max_vm_total".format(group_id),

                      # default=16, mode="int"),

@@ -31,7 +31,6 @@

  #   archs - architectures to build by this group

  #   spawn_playbook - path to an ansible playbook which spawns a builder

  #   terminate_playbook - path to an ansible playbook to terminate the builder

- #   max_workers - maximum number of workers in this group

  #   max_vm_total - maximum number of VM which can run in parallel

  #   max_vm_per_user - maximum number of VM which can use one user in parallel

  #   max_spawn_processes=2 - max number of spawning playbooks run in parallel
@@ -50,12 +49,10 @@

  #   group0_archs=i386,x86_64

  #   group0_spawn_playbook=/srv/copr-work/provision/builderpb-PC.yml

  #   group0_terminate_playbook=/srv/copr-work/provision/terminatepb-PC.yml

- #   group0_max_workers=8

  group0_name=dockergroup

  group0_archs=i386,x86_64

  group0_spawn_playbook=/root/provision/spawn_local.yml

  group0_terminate_playbook=/root/provision/terminate_local.yml

- group0_max_workers=1

  group0_max_vm_total=1

  

  # directory where results are stored

This was just misleading in code and example config file, this is not
used for some time (since 1c51da6 I think).

Merges: #1022

So, the maximum number of workers is limited by group0_max_vm_total?
LGTM

Commit 3e7c9a9 fixes this pull-request

Pull-Request has been merged by msuchy

4 years ago

Pull-Request has been merged by msuchy

4 years ago