Code review comment for lp://staging/~jimmiebtlr/juju-core/add_machines_tag

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

On 2014/05/28 05:43:32, jimmiebtlr wrote:
> Please take a look.

So I confirmed it with William, that we don't actually want the CLI to
be exposing tags to the user. The actual user facing things is intended
to be things like "0" for machine-0 and mysql/2 for unit-mysql-2.

I'm quite happy to see the tests get updated, but I think we actually
want the original syntax.

https://codereview.appspot.com/98610044/

« Back to merge proposal