Merge "openvswitch: Use ansible_hostname for system-id"

ansible/roles/openvswitch/defaults/main.yml

@@ -75,4 +75,4 @@ openvswitch_vswitchd_extra_volumes: "{{ openvswitch_extra_volumes }}"
 # OpenvSwitch
 #############
 
-openvswitch_system_id: "{{ inventory_hostname_short }}"
+openvswitch_system_id: "{{ ansible_hostname }}"

releasenotes/notes/bug-1884734-ec054799d3300778.yaml

@@ -0,0 +1,10 @@
+---
+fixes:
+  - |
+    Fixes an issue when using ip addresses instead of hostnames in Ansible
+    inventory. OpenvSwitch role sets system-id based on inventory_hostname,
+    which in case of ip addresses in is first ip octet.
+    Such a deployment would result in multiple OVN chassis with duplicate name
+    e.g. "10" connecting to OVN Southbound database - which spawns high numbers
+    of create/delete events in Encap database table - leading to near 100% CPU
+    usage of OVN/OVS/Neutron processes.

zuul.d/base.yaml

@@ -168,7 +168,7 @@
     parent: kolla-ansible-base
     voting: false
     files:
-      - ^ansible/roles/(ovn|neutron)/
+      - ^ansible/roles/(neutron|openvswitch|ovn)/
       - ^tests/test-ovn.sh
       - ^tests/test-core-openstack.sh
     vars: