Open Stack

Hi,

we have this latent bug where we pass objects for subnetpools into  
extension functions. (That was a mistake made when switching to using  
objects in base db plugin for the resource.) Now, we want to revert to  
passing db models as we did before, so that our plugin api is consistent  
and reflects what we had before.

The patch that would achieve that is at:  
https://review.openstack.org/#/c/348279/

The patch is still a slight WIP, and also has concerns from @zzzeek around  
using populate_existing, same concerns as found in Kevin’s patch at:  
https://review.openstack.org/#/c/364301/

While we are still shaping the patches, it would be wise to land some of  
dependencies that are just cosmetic changes to objects API and test  
framework. Specifically, we’ll need:

https://review.openstack.org/#/c/348271/
https://review.openstack.org/#/c/366052/

Those pass CI right now, and would be actually valuable irrelevant to the  
result of the discussion with @zzzeek.

Thanks for reviews,
Ihar