remove zookeeper config requirement

[bdoyle0182]

Description

First step to getting rid of the zookeeper dependency on the invokers. Invokers currently can already be started without the zookeeper config if --id flag is set which is already the case in ansible. The only case that needs to be covered still before removing the dependency is adding support for kubernetes deployment. If started without config and need to connect to zookeeper still, the invoker will fail and shut down anyways from failing to connect to zookeeper in the instance id assigner on startup so this shouldn't be a breaking change.

Related issue and scope

• [ ] I opened an issue to propose and discuss this change (#????)

My changes affect the following components

• [ ] API
• [ ] Controller
• [ ] Message Bus (e.g., Kafka)
• [ ] Loadbalancer
• [X] Scheduler
• [X] Invoker
• [ ] Intrinsic actions (e.g., sequences, conductors)
• [ ] Data stores (e.g., CouchDB)
• [ ] Tests
• [ ] Deployment
• [ ] CLI
• [ ] General tooling
• [ ] Documentation

Types of changes

• [ ] Bug fix (generally a non-breaking change which closes an issue).
• [ ] Enhancement or new feature (adds new functionality).
• [ ] Breaking change (a bug fix or enhancement which changes existing behavior).

Checklist:

• [X] I signed an Apache CLA.
• [X] I reviewed the style guides and followed the recommendations (Travis CI will check :).
• [ ] I added tests to cover my changes.
• [ ] My changes require further changes to the documentation.
• [ ] I updated the documentation where necessary.

[codecov-commenter]

Codecov Report

Merging #5375 (703c3a8) into master (096dba4) will decrease coverage by 61.94%. The diff coverage is n/a.

:exclamation: Current head 703c3a8 differs from pull request most recent head 3a3407b. Consider uploading reports for the commit 3a3407b to get more accurate results

@@            Coverage Diff             @@
##           master   #5375       +/-   ##
==========================================
- Coverage   66.46%   4.53%   -61.94%     
==========================================
  Files         240     240               
  Lines       14569   14567        -2     
  Branches      642     647        +5     
==========================================
- Hits         9684     660     -9024     
- Misses       4885   13907     +9022     

Impacted Files	Coverage Δ
...la/org/apache/openwhisk/core/invoker/Invoker.scala	0.00% <ø> (-72.50%)	:arrow_down:
...rg/apache/openwhisk/core/scheduler/Scheduler.scala	0.00% <ø> (-1.90%)	:arrow_down:
...ain/scala/org/apache/openwhisk/spi/SpiLoader.scala	0.00% <0.00%> (-100.00%)	:arrow_down:
...n/scala/org/apache/openwhisk/utils/JsHelpers.scala	0.00% <0.00%> (-100.00%)	:arrow_down:
...scala/org/apache/openwhisk/common/Prometheus.scala	0.00% <0.00%> (-100.00%)	:arrow_down:
...scala/org/apache/openwhisk/core/FeatureFlags.scala	0.00% <0.00%> (-100.00%)	:arrow_down:
...scala/org/apache/openwhisk/http/CorsSettings.scala	0.00% <0.00%> (-100.00%)	:arrow_down:
...ala/org/apache/openwhisk/common/ConfigMXBean.scala	0.00% <0.00%> (-100.00%)	:arrow_down:
...la/org/apache/openwhisk/http/BasicRasService.scala	0.00% <0.00%> (-100.00%)	:arrow_down:
...a/org/apache/openwhisk/common/ConfigMapValue.scala	0.00% <0.00%> (-100.00%)	:arrow_down:
... and 184 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more