Code review comment for lp:~gz/juju-core/openstack_network_config_1241674

Revision history for this message
Martin Packman (gz) wrote :

Reviewers: mp+203114_code.launchpad.net,

Message:
Please take a look.

Description:
provider/openstack: Add network config option

Resolve issue with running juju on openstack setups with
multiple networks, by allowing the user to supply a
network label or id that juju should use for its machines.

Note: this still requires a dependencies.tsv goose bump

https://code.launchpad.net/~gz/juju-core/openstack_network_config_1241674/+merge/203114

(do not edit description out of merge proposal)

Please review this at https://codereview.appspot.com/52710048/

Affected files (+146, -8 lines):
   A [revision details]
   M provider/openstack/config.go
   M provider/openstack/config_test.go
   M provider/openstack/export_test.go
   M provider/openstack/local_test.go
   M provider/openstack/provider.go

« Back to merge proposal