[GitHub] [openwhisk-deploy-kube] dgrove-oss commented on pull request #606: Fix to check if affinity is enabled

[GitBox <gi...@apache.org>]

dgrove-oss commented on pull request #606:
URL: https://github.com/apache/openwhisk-deploy-kube/pull/606#issuecomment-635648369


   DockerContainerFactory is of dwindling usefulness, but I do think we should either require affinity to be enabled when using DockerContainerFactory or make some changes to the documentation to make it clear what is happening.  In particular we emphasize labeling the subset of nodes for invokers to use (https://github.com/apache/openwhisk-deploy-kube#initial-setup). If we are going to ignore that when affinity is disabled, we should clearly document that so that people understand that unless you are just kicking the tires on a single node cluster, its not a good configuration to use.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org