[nova][neutron][cyborg] Bandwidth (and accel) providers are broken if CONF.host is set
smooney at redhat.com
Fri Nov 22 12:39:43 UTC 2019
On Fri, 2019-11-22 at 02:11 +0000, Nadathur, Sundar wrote:
> > -----Original Message-----
> > From: Eric Fried <openstack at fried.cc>
> > Sent: Thursday, November 21, 2019 3:28 PM
> > Action Summary
> > ==============
> > If the above sounds reasonable, it would entail the following actions:
> > - Neutron(/Cyborg?): backportable patch to
> > s/CONF.host/socket.gethostname()/
> > - Nova: GET /os-hypervisors*?service_host=X in a new microversion.
> > - Neutron/Cyborg: master-only patch to do the logic described in `Upgrade
> > Concerns`_ (though for now without the `elif` branch).
> Cyborg does use CONF.host today. We can use socket.gethostname() instead.
> On a related node, the patch for ARQ binding  uses instance.host, which comes from CONF.host. I'll change it to
> instance.node, which comes from , which in turn comes from get_hostname() .
for comunication with neutorn and cinder nova uses CONF.host so cyborg shoudl still use the CONF.host for acclerators
but for comunicating with placemet we use the hypervior_hostname. so in the cyborg case we choudl chage form
socket.getFQDN to socket.gethostname as the defualt for CONF.host but simply change the get_root_prover function so that
it uses the hypervisor api to determin the uuid of the RP
>  https://review.opendev.org/#/c/631244/46/nova/compute/manager.py@2634
>  https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L9137
>  https://github.com/openstack/nova/blob/1cd5563f2dd2b218db2422397c8aab394d484626/nova/virt/libvirt/driver.py#L9614
> > Thanks,
> > efried
> Thanks & Regards,
More information about the openstack-discuss