Fix buffer_profile GCU validator to pass scope to rdma_config_update_validator

validate_field_operation invokes validators as (scope, patch_element).
buffer_profile_config_update_validator only took patch_element and called
rdma_config_update_validator with one argument, causing TypeError on
field-level BUFFER_PROFILE patches. Accept scope and forward both arguments.

Signed-off-by: anamehra <anamehra@cisco.com>

Author: anamehra

generic_config_updater/field_operation_validators.py

@@ -62,11 +62,13 @@ def get_asic_name():
     return asic
 
 
-def buffer_profile_config_update_validator(patch_element):
+def buffer_profile_config_update_validator(scope, patch_element):
     """
     Enhanced buffer profile validator that handles both field-level and object-level operations.
     - Field-level operations (e.g., /BUFFER_PROFILE/profile/dynamic_th) follow existing rules
     - Object-level operations (e.g., /BUFFER_PROFILE/profile) allow remove operations
+
+    scope: same as other GCU validators (e.g. localhost, asic0); forwarded to rdma_config_update_validator.
     """
     path = patch_element["path"]
     path_parts = jsonpointer.JsonPointer(path).parts
@@ -87,7 +89,7 @@ def buffer_profile_config_update_validator(patch_element):
             return False  # Disallow unsupported operations
 
     # For field-level operations, use the existing validation logic
-    return rdma_config_update_validator(patch_element)
+    return rdma_config_update_validator(scope, patch_element)
 
 
 def rdma_config_update_validator(scope, patch_element):

tests/generic_config_updater/field_operation_validator_test.py

@@ -379,7 +379,8 @@ def test_buffer_profile_config_update_validator_object_level_remove(self):
         }
 
         # Object-level remove should be allowed without ASIC/version validation
-        assert fov.buffer_profile_config_update_validator(patch_element) is True
+        for scope in ["localhost", "asic0"]:
+            assert fov.buffer_profile_config_update_validator(scope, patch_element) is True
 
     def test_buffer_profile_config_update_validator_object_level_add(self):
         """Test that object-level add operations are allowed"""
@@ -390,7 +391,8 @@ def test_buffer_profile_config_update_validator_object_level_add(self):
         }
 
         # Object-level add should be allowed
-        assert fov.buffer_profile_config_update_validator(patch_element) is True
+        for scope in ["localhost", "asic0"]:
+            assert fov.buffer_profile_config_update_validator(scope, patch_element) is True
 
     def test_buffer_profile_config_update_validator_object_level_unsupported_op(self):
         """Test that unsupported operations on object-level are denied"""
@@ -400,7 +402,8 @@ def test_buffer_profile_config_update_validator_object_level_unsupported_op(self
             "from": "/BUFFER_PROFILE/old_profile"
         }
 
-        assert fov.buffer_profile_config_update_validator(patch_element) is False
+        for scope in ["localhost", "asic0"]:
+            assert fov.buffer_profile_config_update_validator(scope, patch_element) is False
 
     def test_buffer_profile_config_update_validator_field_level_uses_existing_validation(self):
         """Test that field-level operations use existing validation logic"""
@@ -413,4 +416,5 @@ def test_buffer_profile_config_update_validator_field_level_uses_existing_valida
         # Mock the existing validation to return True
         with patch("generic_config_updater.field_operation_validators.rdma_config_update_validator",
                    return_value=True):
-            assert fov.buffer_profile_config_update_validator(patch_element) is True
+            for scope in ["localhost", "asic0"]:
+                assert fov.buffer_profile_config_update_validator(scope, patch_element) is True