CORTX-33673: updated lock unlock interfaces

Signed-off-by: Tanuja Shinde <[email protected]>

Author: Tanuja Shinde

.github/auto_assign.yml

@@ -0,0 +1,19 @@
+# Set to true to add reviewers to pull requests
+  addReviewers: true
+
+# Set to true to add assignees to pull requests
+  addAssignees: true
+
+# A list of reviewers to be added to pull requests (GitHub user name)
+  reviewers: 
+   - sachinpunadikar
+   - ujjwalpl
+   - cdeshmukh
+
+# A list of keywords to be skipped the process that add reviewers if pull requests include it 
+  skipKeywords:
+   - wip
+
+# A number of reviewers added to the pull request
+# Set 0 to add all the reviewers (default: 0)
+  numberOfReviewers: 0

py-utils/src/utils/conf_store/conf_store.py

@@ -291,9 +291,11 @@ def lock(self, index: str, **kwargs):
                 index)
 
         allowed_keys = { 'domain', 'owner', 'duration' }
-        for key in kwargs.keys():
+        for key, val in kwargs.items():
             if key not in allowed_keys:
                 raise ConfError(errno.EINVAL, "Invalid parameter %s", key)
+            if key == 'duration' and not isinstance(val, int):
+                raise ConfError(errno.EINVAL, "Invalid value %s for parameter %s", val, key)
 
         owner = self._lock_owner if 'owner' not in kwargs.keys() else kwargs['owner']
         domain = self._lock_domain if 'domain' not in kwargs.keys() else kwargs['domain']
@@ -330,11 +332,11 @@ def unlock(self, index: str, **kwargs):
 
         force = False
         allowed_keys = { 'domain', 'owner', 'force' }
-        for key in kwargs.keys():
+        for key, val in kwargs.items():
             if key not in allowed_keys:
                 raise ConfError(errno.EINVAL, "Invalid parameter %s", key)
-            if key == 'force' and not isinstance(kwargs['force'], bool):
-                raise ConfError(errno.EINVAL, "Invalid value %s for parameter %s", kwargs['force'], key)
+            if key == 'force' and not isinstance(val, bool):
+                raise ConfError(errno.EINVAL, "Invalid value %s for parameter %s", val, key)
 
         owner = self._lock_owner if 'owner' not in kwargs.keys() else kwargs['owner']
         domain = self._lock_domain if 'domain' not in kwargs.keys() else kwargs['domain']
@@ -362,7 +364,7 @@ def test_lock(self, index: str, **kwargs):
                 index)
 
         allowed_keys = { 'domain' , 'owner'}
-        for key in kwargs.keys():
+        for key, _ in kwargs.items():
             if key not in allowed_keys:
                 raise ConfError(errno.EINVAL, "Invalid parameter %s", key)