From 33ae19a57dc12ffb38973de5666b428e11187602 Mon Sep 17 00:00:00 2001
From: Daniil Baturin <daniil@baturin.org>
Date: Wed, 17 Jul 2024 11:45:35 +0100
Subject: op-mode: T6586: add a distinct exception for unconfigured objects as
 opposed to entire subsystems

---
 src/op_mode/bridge.py | 2 +-
 src/op_mode/dhcp.py   | 2 +-
 src/op_mode/ssh.py    | 2 +-
 src/op_mode/zone.py   | 4 ++--
 4 files changed, 5 insertions(+), 5 deletions(-)

(limited to 'src/op_mode')

diff --git a/src/op_mode/bridge.py b/src/op_mode/bridge.py
index d04f1541f..e80b1c21d 100755
--- a/src/op_mode/bridge.py
+++ b/src/op_mode/bridge.py
@@ -70,7 +70,7 @@ def _get_raw_data_fdb(bridge):
     # From iproute2 fdb.c, fdb_show() will only exit(-1) in case of
     # non-existent bridge device; raise error.
     if code == 255:
-        raise vyos.opmode.UnconfiguredSubsystem(f"no such bridge device {bridge}")
+        raise vyos.opmode.UnconfiguredObject(f"bridge {bridge} does not exist in the system")
     data_dict = json.loads(json_data)
     return data_dict
 
diff --git a/src/op_mode/dhcp.py b/src/op_mode/dhcp.py
index 6f57f22a5..e5455c8af 100755
--- a/src/op_mode/dhcp.py
+++ b/src/op_mode/dhcp.py
@@ -332,7 +332,7 @@ def _verify_client(func):
 
         # Check if config does not exist
         if not config.exists(f'interfaces {interface_path} address dhcp{v}'):
-            raise vyos.opmode.UnconfiguredSubsystem(unconf_message)
+            raise vyos.opmode.UnconfiguredObject(unconf_message)
         return func(*args, **kwargs)
     return _wrapper
 
diff --git a/src/op_mode/ssh.py b/src/op_mode/ssh.py
index 102becc55..0c51576b0 100755
--- a/src/op_mode/ssh.py
+++ b/src/op_mode/ssh.py
@@ -65,7 +65,7 @@ def show_fingerprints(raw: bool, ascii: bool):
 def show_dynamic_protection(raw: bool):
     config = ConfigTreeQuery()
     if not config.exists(['service', 'ssh', 'dynamic-protection']):
-        raise vyos.opmode.UnconfiguredSubsystem("SSH server dynamic-protection is not enabled.")
+        raise vyos.opmode.UnconfiguredObject("SSH server dynamic-protection is not enabled.")
 
     attackers = []
     try:
diff --git a/src/op_mode/zone.py b/src/op_mode/zone.py
index d24b1065b..49fecdf28 100644
--- a/src/op_mode/zone.py
+++ b/src/op_mode/zone.py
@@ -104,7 +104,7 @@ def _convert_config(zones_config: dict, zone: str = None) -> list:
         if zones_config:
             output = [_convert_one_zone_data(zone, zones_config)]
         else:
-            raise vyos.opmode.DataUnavailable(f'Zone {zone} not found')
+            raise vyos.opmode.UnconfiguredObject(f'Zone {zone} not found')
     else:
         if zones_config:
             output = _convert_zones_data(zones_config)
@@ -212,4 +212,4 @@ if __name__ == '__main__':
             print(res)
     except (ValueError, vyos.opmode.Error) as e:
         print(e)
-        sys.exit(1)
\ No newline at end of file
+        sys.exit(1)
-- 
cgit v1.2.3