T7175: VPP add sFlow smoketest

Author: sever-sever

smoketest/scripts/cli/test_vpp.py

@@ -1393,6 +1393,30 @@ def test_17_vpp_nat(self):
         _, out = rc_cmd('sudo vppctl show nat44 summary')
         self.assertIn(f'max translations per thread: {sess_limit} fib 0', out)
 
+    def test_18_vpp_sflow(self):
+        base_sflow = ['system', 'sflow']
+
+        self.cli_set(base_path + ['sflow', 'interface', interface])
+        self.cli_set(base_sflow + ['interface', interface])
+        self.cli_set(base_sflow + ['server', '127.0.0.1'])
+        self.cli_set(base_sflow + ['vpp'])
+        self.cli_commit()
+
+        # Check sFlow
+        _, out = rc_cmd('sudo vppctl show sflow')
+
+        expected_entries = (
+            'sflow sampling-direction ingress',
+            f'sflow enable {interface}',
+            'interfaces enabled: 1',
+        )
+
+        for expected_entry in expected_entries:
+            self.assertIn(expected_entry, out)
+
+        self.cli_delete(base_sflow)
+        self.cli_commit()
+
 
 if __name__ == '__main__':
     unittest.main(verbosity=2)

src/conf_mode/vpp_sflow.py

@@ -91,7 +91,9 @@ def verify(config):
     # Verify that all interfaces specified exist in VPP
     for interface in config['interface']:
         if interface not in config['vpp_ifaces']:
-            raise ConfigError(f'{interface} must be a VPP interface for sFlow monitoring')
+            raise ConfigError(
+                f'{interface} must be a VPP interface for sFlow monitoring'
+            )
 
     # Verify sample rate is a positive integer
     if 'sample_rate' in config:
@@ -104,7 +106,9 @@ def verify(config):
 
     # Verify that system sflow has enable-vpp defined
     if 'system_sflow' not in config or 'vpp' not in config.get('system_sflow', {}):
-        raise ConfigError('sFlow enable-vpp must be defined under system sflow configuration')
+        raise ConfigError(
+            'sFlow enable-vpp must be defined under system sflow configuration'
+        )
 
 
 def generate(config):