neeraj-laad commented on a change in pull request #372: Helm chart providers
URL:
https://github.com/apache/incubator-openwhisk-deploy-kube/pull/372#discussion_r236987858
##########
File path: helm/openwhisk/values.yaml
##########
@@ -242,14 +254,14 @@ providers:
db:
external: false
# Define the rest of these values if you are using external couchdb
instance
- # host: "10.10.10.10"
- # port: 5984
- # protocol: "http"
- # username: "admin"
- # password: "secret"
+ host: "10.10.10.10"
+ port: 5984
+ protocol: "http"
+ username: "admin"
+ password: "secret"
# Alarm provider configurations
alarm:
- enabled: false
+ enabled: true
Review comment:
Given we are switching back to docker, and we do not have an automated way
to enable them using docker container factory. I'll disable them for now.. We
can review this again once #370 is merged in.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
