Merge lp://qastaging/~ntt-pf-lab/nova/network-service into lp://qastaging/~midokura/nova/network-service

Normally these kinds of things should probably have bugs associated with them for tracking.

Looks good, though

lgtm

why are we using environment variables here instead of command-line arguments?

It's using the env variables mainly for consistency with novaclient tool. I think it would work either way.

Thanks for catching this. Will merge.

Shouldn't we be being consistent with nova, not novaclient?

In this case, this client, even though it's called nova-manage-network, it's doing something a bit different from nova-manage in that it's simply making REST API calls to the nova-api server as opposed to updating DB records directly from the script. The only other client tool in Nova that I was aware of that does something similar was novaclient so I just implemented it the same way. This branch is also experimental, and it's here just to showcase that the network services can be pluggable The new (less disruptive) way to do this is being developed in lp:~midokura/nova/network-refactoring.

And I completely agree that we need to make everything as consistent as possible to Nova, which is one of the main focuses in the new branch.