)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"3051fd7d55b06a2750ad7e5390904079a9549cf9","unresolved":true,"context_lines":[{"line_number":9,"context_line":"In some deployments, hostnames can be different between hosts and"},{"line_number":10,"context_line":"resource provider records. For example in deployment managed by TripleO"},{"line_number":11,"context_line":"we use short host name (without domain name) in host, while we use FQDN"},{"line_number":12,"context_line":"for resource provider records (this comes from the FQDN set to the host"},{"line_number":13,"context_line":"option in nova.conf)."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"This causes an issue with the way how currently neutron looks up the"},{"line_number":16,"context_line":"root resource provider because placment API requires the exact hostname"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"e149a048_84911e19","line":13,"range":{"start_line":12,"start_character":30,"end_line":13,"end_character":21},"updated":"2020-11-24 13:19:09.000000000","message":"this is not correct.\nwe do not use the host paramter in the nova.conf to set the RP name.\n\nwe use the hypervior_hostname filed in the compute node recored that is populated form teh virt driver.\nin the case of libvirt we call libvirt to get teh host name which internally call gethostname in libc\n\nthe resouce provider name should always match what is returon by socket.gethostname","commit_id":"2f57b56c6dd7bf65009c5ab8c9befdc2b5039715"}],"neutron/agent/common/utils.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"367b1cf6f2968f5a7bed20c684f271d99851998a","unresolved":true,"context_lines":[{"line_number":86,"context_line":"        format."},{"line_number":87,"context_line":"    :param default_hypervisor: Default hypervisor hostname."},{"line_number":88,"context_line":"    \"\"\""},{"line_number":89,"context_line":"    _default_hypervisor \u003d default_hypervisor or socket.gethostname()"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"    rv \u003d {}"},{"line_number":92,"context_line":"    for _physnet, devices in device_mappings.items():"}],"source_content_type":"text/x-python","patch_set":4,"id":"653da02e_9292abe3","line":89,"updated":"2020-11-23 12:26:12.000000000","message":"But the goal here was to sync Nova and Neutron way of naming the RPs, using the hostname. In case you are not using the hostname and you need, for example, the fqdn (as in your case), why don\u0027t you populate \"resource_provider_hypervisors\"?\n\n\n[1]https://review.opendev.org/c/openstack/neutron/+/696600/3/neutron/agent/common/utils.py#86","commit_id":"59575cbd351446485f4c82f8a09ff9b2b56cad34"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"e858efad3e9ef5986c16022d397ca0d507d710ed","unresolved":true,"context_lines":[{"line_number":86,"context_line":"        format."},{"line_number":87,"context_line":"    :param default_hypervisor: Default hypervisor hostname."},{"line_number":88,"context_line":"    \"\"\""},{"line_number":89,"context_line":"    _default_hypervisor \u003d default_hypervisor or socket.gethostname()"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"    rv \u003d {}"},{"line_number":92,"context_line":"    for _physnet, devices in device_mappings.items():"}],"source_content_type":"text/x-python","patch_set":4,"id":"dc17ad3b_4a9db7ed","line":89,"in_reply_to":"653da02e_9292abe3","updated":"2020-11-23 13:13:17.000000000","message":"Actually we can use resource_provider_hypervisors to override hypervisor names.\nHowever to use the resource_provider_hypervisors option, we need to pass a list of all bridges or interface. This looks very redundant when we expect only a single hypervisor name.","commit_id":"59575cbd351446485f4c82f8a09ff9b2b56cad34"}]}