ddragosd opened a new pull request #2256: surface errors in runtimes.manifest configuration in the controller logs URL: https://github.com/apache/incubator-openwhisk/pull/2256 This PR ensures than when `runtimes.manifest` property is incorrectly configured that the controller prints what the error is in the logs. Without this PR we would only see in the logs: ``` [2017-05-17T21:21:53.457Z] [ERROR] [??] [Controller] Bad configuration, cannot start. ``` With this PR we can see: ``` Exception in thread "main" java.lang.IllegalStateException: Runtimes manifest is set but it's invalid: spray.json.DeserializationException: Object is missing required member 'image' at whisk.core.entity.ExecManifest$.initialize(ExecManifest.scala:55) at whisk.core.controller.Controller$.main(Controller.scala:185) at whisk.core.controller.Controller.main(Controller.scala) ``` This is very helpful when debugging issues with the configuration such as the one mentioned at https://github.com/apache/incubator-openwhisk-devtools/issues/29 ---------------------------------------------------------------- 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: [email protected]
With regards, Apache Git Services
