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

Revision history for this message
Gavin Panella (allenap) wrote :

On 2013/07/31 09:04:02, rvb wrote:
> 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…

The code as I've written it will work now and if the
service:deployment:role ratio changes so I think it's better to leave
it.

Thank you both for the reviews!

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

« Back to merge proposal