feat: disable activity workers

[cv65kr]

Reason for This PR

roadrunner-server/roadrunner#2089

Description of Changes

Disable activity workers

License Acceptance

By submitting this pull request, I confirm that my contribution is made under
the terms of the MIT license.

PR Checklist

[Author TODO: Meet these criteria.]
[Reviewer TODO: Verify that these criteria are met. Request changes if not]

• All commits in this PR are signed (git commit -s).
• The reason for this PR is clearly provided (issue no. or explanation).
• The description of changes is clear and encompassing.
• Any required documentation changes (code and docs) are included in this PR.
• Any user-facing changes are mentioned in CHANGELOG.md.
• All added/changed functionality is tested.

[cv65kr]

Changes need to be provided to https://github.com/roadrunner-server/pool/blob/master/pool/config.go#L42

[rustatian]

Hey @cv65kr 👋🏻 Thank you for the PR 👍🏻

Changes need to be provided to

To skip workers allocation, after the initializing the pool in the config.go, check if that option to disable activity workers is active (set to true) and set NumWorkers to 0: link.

[cv65kr]

@rustatian I don't think it will work
https://github.com/temporalio/roadrunner-temporal/blob/master/common/interfaces.go#L66 ->
https://github.com/roadrunner-server/server/blob/master/plugin.go#L143 ->
https://github.com/roadrunner-server/pool/blob/612ec9672eb621bab67004e8dc517d396a0bfd10/pool/static_pool/pool.go#L61

Anyway it will override the default values

[rustatian]

We can't change the default behavior for the pool.

In the pool (NewPool), last arg are options. You may introduce a new option, like, WithOverrideWorkersNum (something like that) and pass this option in the pool in the rr-temporal plugin. Options are applied after InitDefaults and before actual workers allocation.

[codecov-commenter]

⚠️ Please install the to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 36.71%. Comparing base (c5c2e9a) to head (c2aa12d).
Report is 33 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #592      +/-   ##
==========================================
- Coverage   40.06%   36.71%   -3.36%     
==========================================
  Files          14       14              
  Lines         936     1095     +159     
==========================================
+ Hits          375      402      +27     
- Misses        518      652     +134     
+ Partials       43       41       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[rustatian]

LGTM, thanks @cv65kr 👍🏻