StopInstances requires Instances, causing unnecessary provider API overhead

Bug #1260171 reported by Andrew Wilkins
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
juju-core
Fix Released
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.

Tags: tech-debt

Related branches

Revision history for this message
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
Revision history for this message
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)
Changed in juju-core:
milestone: none → 1.19.3
Andrew Wilkins (axwalk)
Changed in juju-core:
status: In Progress → Fix Committed
Curtis Hovey (sinzui)
Changed in juju-core:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.