Fix new scheduler error#5206
Conversation
| "CONFIG_whisk_etcd_pool_threads": "{{ etcd.pool_threads }}" | ||
| "CONFIG_whisk_scheduler_grpc_tls": "{{ scheduler.grpc.tls | default('false') | lower }}" | ||
| "CONFIG_whisk_scheduler_maxPeek": "{{ scheduler.maxPeek }}" | ||
| "CONFIG_whisk_spi_LoadBalancerProvider": "org.apache.openwhisk.core.loadBalancer.FPCPoolBalancer" |
There was a problem hiding this comment.
Currently, this is guided to configure them alternatively.
https://github.com/apache/openwhisk/tree/master/ansible#optional-enable-the-new-scheduler
If we take this approach, how about changing the documentation too?
There was a problem hiding this comment.
this pr is only useful for ansible way(we may need other prs for k8s and standalone), so I think it's better to keep the document
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #5206 +/- ##
===========================================
+ Coverage 44.67% 72.16% +27.49%
===========================================
Files 238 238
Lines 13937 14227 +290
Branches 581 588 +7
===========================================
+ Hits 6226 10267 +4041
+ Misses 7711 3960 -3751 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
| AuthenticationDirectiveProvider = org.apache.openwhisk.core.controller.BasicAuthenticationDirective | ||
| InvokerProvider = org.apache.openwhisk.core.invoker.InvokerReactive | ||
| InvokerServerProvider = org.apache.openwhisk.core.invoker.DefaultInvokerServer | ||
| DurationCheckerProvider = org.apache.openwhisk.core.scheduler.queue.NoopDurationCheckerProvider |
There was a problem hiding this comment.
Regardless of whether the ansible changes make it in, I think we should merge in the changes to this file so that service can at least run with all of the necessary providers. If you want to put this file's changes in a separate PR I'd approve that. Don't have an opinion on the ansible change
5 participants
Add some missing configuration for new scheduler
Description
Related issue and scope
My changes affect the following components
Types of changes
Checklist: