Code review comment for lp:~allenap/juju-core/azure-open-machine-ports

Revision history for this message
Raphaël Badin (rvb) wrote :

On 2013/07/31 09:00:46, rvb wrote:
> On 2013/07/30 23:16:12, allenap wrote:
> > Please take a look.

> [0]

> Julian has a point, only one deployment should be present in the
service, and it
> has the same name as the service ('azInstance.ServiceName'). That
will save one
> API call.

Actually, since you need to get all the roles for the deployment, I
don't think relying on the fact that we know there is only one
deployment named 'serviceName' will save us an API call…

https://codereview.appspot.com/12102045/

« Back to merge proposal