~axwalk/juju-core/lp1260171-stopinstances-ids

Viewing all changes in revision 2723.

  • Committer: Andrew Wilkins
  • Date: 2014-05-13 11:37:40 UTC
  • Revision ID: andrew.wilkins@canonical.com-20140513113740-xghytp9prigo5add
Change StopInstances to take []instance.Id

StopInstances previously took []instance.Instance,
which is unnecessary for most providers, and imposes
an unnecessary cost for callers that only have IDs
initially. We change StopInstances to 

The openstack provider's StopInstances is now slightly
more expensive than before, requiring an additional call
to Instances. This cost will disappear when we fix the
security groups/open-port implementation.

Drive-by: the maas provider now uses a bulk "release"
in StopInstances.

Live tested on ec2, local, virtual MAAS, canonistack, azure.
I lack a Joyent account, but I think it should be fine.

Fixes lp:1260171
Fixes lp:1316272

expand all expand all

Show diffs side-by-side

added added

removed removed

Lines of Context: