[GitHub] chetanmeh opened a new pull request #3369: Use pureconfig for CouchDbRestStore

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

chetanmeh opened a new pull request #3369: Use pureconfig for CouchDbRestStore
URL: https://github.com/apache/incubator-openwhisk/pull/3369
 
 
   PR for #3355 Now the CouchDB config is read from application.conf with following structure
   
   ```
   whisk {
       couchdb {
           protocol = "http"
           host     = "localhost"
           port     = 5984
           username = "user"
           password = "password"
           provider = "CouchDB"
       }
     }
   ```
   
   The config can be provided via following environment variables
   
   ```
   CONFIG_whisk_couchdb_protocol
   CONFIG_whisk_couchdb_host
   CONFIG_whisk_couchdb_port
   CONFIG_whisk_couchdb_username
   CONFIG_whisk_couchdb_password
   CONFIG_whisk_couchdb_provider
   ```
   
   This would be a backward incompatible change for existing deployments where they would need to switch to new environment variables instead of existing DB_PORT etc
   
   For now `ArtifactStoreProvider#makeStore` still takes WhiskConfig for determining DB name. That would be addressed in a separate PR

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services