StopInstances requires Instances, causing unnecessary provider API overhead

Bug #1260171 reported by Andrew Wilkins on 2013-12-12
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
juju-core
Low
Andrew Wilkins

Bug Description

The StopInstances method of environs.InstanceBroker requires a slice of instance.Instance. For the most part we do not require instances, and instead could rely solely on the instance ID for destroying. Requiring the full Instance causes unnecessary provider API overhead, which may lead to rate limiting.

Related branches

John A Meinel (jameinel) wrote :

Generally we don't use "Medium", as it is either stuff we're working on (planned work, High) or it isn't (Low).

Changed in juju-core:
importance: Medium → Low
Andrew Wilkins (axwalk) wrote :

I'm fixing this to fix lp:1316272

Changed in juju-core:
assignee: nobody → Andrew Wilkins (axwalk)
status: Triaged → In Progress
Ian Booth (wallyworld) on 2014-05-13
Changed in juju-core:
milestone: none → 1.19.3
Andrew Wilkins (axwalk) on 2014-05-14
Changed in juju-core:
status: In Progress → Fix Committed
Curtis Hovey (sinzui) on 2014-05-30
Changed in juju-core:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers