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
__________________________________________________________________________
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
