Code review comment for lp:~openstack-gd/nova/nova-avail-zones

Revision history for this message
Vish Ishaya (vishvananda) wrote :

This is looking pretty good. It appears that volumes are not handled. Did you plan on doing those in another patch? I'm also wondering if this could actually be a subclass of Simple Scheduler, so we at least get the added functionality of going to the least loaded host. Another possibility is just to ignore the zone if it is None, and add it to simple scheduler.

review: Needs Information

« Back to merge proposal