This is only used by the file based drivers, who need to call nova for assistance with attached snapshots. Most drivers won't use it, including the default, LVM.
I can't comment on the test coverage, sorry. On 14 February 2017 at 13:01, Sean Dague <s...@dague.net> wrote: > After some of the confusion around endpoints in devstack, we decided to > simplify the endpoints registered in devstack to only the ones needed > for development. Basically only register "public" interfaces unless > there is something special about the service. > > https://review.openstack.org/#/c/433272/ is the change. > > Matt Riedemann pointed out that this would break Cinder because there is > a hardcoded concept of nova_catalog_admin_info - > https://github.com/openstack/cinder/blob/cfc617b0cea99ed6994f08e5337fd5 > b65ea9fd1c/cinder/compute/nova.py#L39-L41 > > Except... it didn't (see results on > https://review.openstack.org/#/c/433272/). > > What is more confusing, is the oslo.config dump at the beginning of > cinder service starts there don't have any reference to any of these > nova_ config variables. > > How is this code loaded and used? Is there no testing of cinder -> nova > happening in the gate? Is this missing testing, or are there reasons > these configurations would never load that code? > > -Sean > > -- > Sean Dague > http://dague.net > > __________________________________________________________________________ > 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 > -- -- Duncan Thomas
__________________________________________________________________________ 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
