Refs #29817 - Implement a resizable hosts_queue pool #364
Conversation
|
theforeman/puppet-foreman#843 allows users to control the concurrency and the number of workers, here I only see number of workers, is that an intended design to restrict it for this use case? |
|
Not intentional, just a rushed patch here. @ianballou could you have a look about the desired config options and if this properly replaces your patch? Also looks like I messed up the fixtures. Will update both once there's clarity about which params are expected. |
|
@ekohl, it looks like this PR sufficiently covers my old one 👍 |
|
I'm curious what's going on with this PR, I haven't checked-in for a while. As far as I know we'd still like this functionality for Pulp 3 applicability calculations, but someone can let me know if we don't. |
|
This can now be updated with theforeman/puppet-foreman#843 having been merged |
ekohl
force-pushed
the
29817-dynflow-pool
branch
from
7530ece
to
3bf628a
Compare
|
Rebased to resolve merge conflicts and it depends on theforeman/foreman 17.x to ensure the correct API is present. |
This uses the new foreman::dynflow::pool defined type which allows users to easily add additional workers or scale them back.
ekohl
force-pushed
the
29817-dynflow-pool
branch
from
3bf628a
to
44fe676
Compare
This uses the new foreman::dynflow::pool defined type which allows users to easily add additional workers or scale them back.
Currently a draft since it's pointing to my branch in the fixtures to allow testing. Blocked on theforeman/puppet-foreman#843.