Code review comment for lp://qastaging/~mbruzek/charms/precise/rabbitmq-server/tests

Revision history for this message
James Page (james-page) wrote :

https://codereview.appspot.com/51230045/diff/20001/config.yaml
File config.yaml (right):

https://codereview.appspot.com/51230045/diff/20001/config.yaml#newcode16
config.yaml:16: default: ""
I'd rather not see this - "" is not the same as unset and the charm uses
config-get --format=json ssl_key and relies on return values being
"None" in python.

https://codereview.appspot.com/51230045/

« Back to merge proposal