Code review comment for lp:~dimitern/juju-core/400-lp-1307513-multiple-nics-same-mac

Revision history for this message
Dimiter Naydenov (dimitern) wrote :

On 2014/04/18 14:41:17, fwereade wrote:
> mostly looking good, but (1) IsVirtual on network still seems
ill-formed to me;
> and (2) are we explicitly ignoring compatibility with earlier 1.19?
these new
> fields in state worry me a little, the people who want to work with
networks
> will likely be upgrading from there...

1) I'll drop IsVirtual from networks and keep it on interfaces only.
2) 1.19.0 is a dev release which was out recently, so I think we can get
away with saying "if you used networks, please redeploy your environment
when upgrading to 1.19.1" - I'll add a note to the bug and ping sinzui.

https://codereview.appspot.com/89260044/

« Back to merge proposal