Juju Charms Collection
landscape-client package

Code review comment for lp:~free.ekanayaka/charms/trusty/landscape-client/drop-unit-name-from-juju-info

  1. Trusty Tahr (14.04)
  2. drop-unit-name-from-juju-info
  3. Merge into trunk
Revision history for this message
Alberto Donato (ack) wrote :

LGTM, +1

One question inline.

#1:
I think we could drop "private-address" too. It's optional in the message and we don't use it on the server.

review: Approve

« Back to merge proposal