Add ovn-monitor-all configuration option to ovn-controller

This commit adds the option 'ovn-monitor-all' to the OVN
charm, allowing the operator to enable or disable monitoring of all
records in the OVN_Southbound database by the ovn-controller.

- Updated config.yaml to include 'ovn-monitor-all' (default: false)
- Updated lib/charms/ovn_charm.py to handle the option in
  configure_bridges()
- Added unit tests

Closes-Bug: 2138758

Author: goldberl

config.yaml

@@ -371,3 +371,11 @@ options:
 
       If you are upgrading across LTS boundaries you may need to use version
       pinning to avoid data plane outage during the upgrade.
+  ovn-monitor-all:
+    type: boolean
+    default: false
+    description: |
+      A boolean value that tells if ovn-controller should monitor all records 
+      of tables in the OVN_Southbound. If this option is set to false, 
+      ovn-controller will conditionally monitor only the records that are 
+      needed for the local chassis.

lib/charms/ovn_charm.py

@@ -1344,6 +1344,11 @@ def configure_bridges(self):
         else:
             opvs.remove('.', 'external_ids', 'ovn-cms-options')
 
+        if self.options.ovn_monitor_all:
+            opvs.set('.', 'external_ids:ovn-monitor-all', 'true')
+        else:
+            opvs.remove('.', 'external_ids', 'ovn-monitor-all')
+
     def _get_ovn_cms_options(self):
         """Get options to be passed into ovn-cms-options"""
         cms_opts = []

unit_tests/test_lib_charms_ovn_charm.py

@@ -498,6 +498,7 @@ def _fake_config(x=None):
                 'enable-dpdk': False,
                 'bridge-interface-mappings': 'br-ex:eth0',
                 'prefer-chassis-as-gw': False,
+                'ovn-monitor-all': False,
                 'vpd-device-spec':
                 '[{"bus": "pci", "vendor_id": "beef", "device_id": "cafe"}]',
             }
@@ -580,6 +581,7 @@ def setUp(self):
             'ovn-bridge-mappings': (
                 'provider:br-ex other:br-data'),
             'prefer-chassis-as-gw': False,
+            'ovn-monitor-all': False,
             'dpdk-runtime-libraries': '',
             'vpd-device-spec': '',
             'ovn-source': 'distro',
@@ -766,6 +768,7 @@ def setUp(self):
             'ovn-bridge-mappings': (
                 'provider:br-ex other:br-data'),
             'prefer-chassis-as-gw': False,
+            'ovn-monitor-all': False,
             'dpdk-runtime-libraries': '',
             'vpd-device-spec': '',
             'pmd-cpu-set': '',
@@ -1099,6 +1102,7 @@ def setUp(self):
             'ovn-bridge-mappings': (
                 'provider:br-provider other:br-other'),
             'prefer-chassis-as-gw': True,
+            'ovn-monitor-all': True,
             'vpd-device-spec':
             '[{"bus": "pci", "vendor_id": "beef", "device_id": "cafe"}]',
             'ovn-source': 'distro',
@@ -1440,7 +1444,9 @@ def test_configure_bridges(self):
             mock.call('.', 'external_ids:ovn-bridge-mappings',
                       'other:br-other,provider:br-provider'),
             mock.call('.', 'external_ids:ovn-cms-options',
-                      'enable-chassis-as-gw,card-serial-number=c4rd-53r14l'),
+                      'enable-chassis-as-gw,card-serial-number=c4rd-53r14l,'),
+            mock.call('.', 'external_ids',
+                      'ovn-monitor-all=true'),
         ], any_order=True)
 
     def test_wrong_configure_bridges(self):