bug: Correct wrong specification for 'Create a new replication rule' #…

…6515 This commit replaces the wrong specification for dids for 'Create a new replication rule', it was previously specified as a list of strings. It is now specified as a list of dictionaries containing two properties scope and name.
rucio · Apr 17, 2024 · 92dbade · 92dbade
1 parent 71d75d5
commit 92dbade
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/lib/rucio/client/ruleclient.py b/lib/rucio/client/ruleclient.py
@@ -12,6 +12,7 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
+from collections.abc import Sequence
 from json import dumps, loads
 from typing import Any, Optional, Union
 from urllib.parse import quote_plus
@@ -30,7 +31,7 @@ class RuleClient(BaseClient):
 
     def add_replication_rule(
         self,
-        dids: list[str],
+        dids: Sequence[dict[str, str]],
         copies: int,
         rse_expression: str,
         priority: int = 3,

diff --git a/lib/rucio/web/rest/flaskapi/v1/rules.py b/lib/rucio/web/rest/flaskapi/v1/rules.py
@@ -276,7 +276,14 @@ def post(self):
                     description: The list of data identifiers.
                     type: array
                     items:
-                      type: string
+                      type: object
+                      properties:
+                        scope:
+                          description: The scope of the data identifier
+                          type: string
+                        name:
+                          description: The name of the data identifier
+                          type: string
                   account:
                     description: The account of the issuer.
                     type: string