joshfischer1108 opened a new pull request #3632: URL: https://github.com/apache/incubator-heron/pull/3632
Previous to this PR `helm install` would fail because helm was trying to substitute a number instead of string. The error would look something like below. ``` $ helm install ./heron -g -n heron Error: ConfigMap in version "v1" cannot be handled as a ConfigMap: v1.ConfigMap.Data: ReadString: expects " or n, but found 3, error found in #10 byte of ...|xSizeMB":300,"BK_led|..., bigger context ...|"BK_journalMaxBackups":"3","BK_journalMaxSizeMB":300,"BK_ledgerDirectories":"/bookkeeper/data/ledger|... ``` There is still some work to do on the helm chart and bookkeeper. This is a start of that work. I'll follow up with details of what else needs to be done via github issues and the mailing list. ---------------------------------------------------------------- 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: [email protected]
