Code review comment for lp:~dimitern/juju-core/147-apiprovisioner-blank-env-secrets

Revision history for this message
John A Meinel (jameinel) wrote :

On 2013/09/26 13:10:37, jameinel wrote:
> LGTM given it is the same as the previous submisison which had already
been
> approved.

I do still have the question why we set the values to "not available"
rather than just deleting them. I guess so Env creation doesn't complain
about missing entries?

https://codereview.appspot.com/13964043/

« Back to merge proposal