rucio 35.4.0__tar.gz → 35.4.1__tar.gz

{rucio-35.4.0 → rucio-35.4.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: rucio
-Version: 35.4.0
+Version: 35.4.1
 Summary: Rucio Package
 Home-page: https://rucio.cern.ch/
 Author: Rucio

{rucio-35.4.0 → rucio-35.4.1}/bin/rucio-replica-recoverer

@@ -227,11 +227,11 @@ Note that attempting the use the ``--vos`` argument when in single-VO mode will
   $ rucio-replica-recoverer --run-once --vos abc xyz
   2020-07-28 15:21:33,349 5488    WARNING Ignoring argument vos, this is only applicable in a multi-VO setup.
 ''', formatter_class=argparse.RawDescriptionHelpFormatter)  # NOQA: E501
-    parser.add_argument("--nattempts", action="store", default=5, help='Minimum count of suspicious file replica appearance in bad_replicas table. Default value is 5.')
-    parser.add_argument("--younger-than", action="store", default=5, help='Consider all file replicas logged in bad_replicas table since specified number of younger-than days. Default value is 5.')
+    parser.add_argument("--nattempts", action="store", default=5, type=int, help='Minimum count of suspicious file replica appearance in bad_replicas table. Default value is 5.')
+    parser.add_argument("--younger-than", action="store", default=5, type=int, help='Consider all file replicas logged in bad_replicas table since specified number of younger-than days. Default value is 5.')
     parser.add_argument('--vos', nargs='+', type=str, help='Optional list of VOs to consider. Only used in multi-VO mode.')
     parser.add_argument("--run-once", action="store_true", default=False, help='One iteration only.')
-    parser.add_argument("--limit-suspicious-files-on-rse", action="store", default=5, help='Maximum number of suspicious replicas on an RSE before that RSE is considered problematic and the suspicious replicas on that RSE are declared "TEMPORARY_UNAVAILABLE". Default value is 5.')
+    parser.add_argument("--limit-suspicious-files-on-rse", action="store", default=5, type=int, help='Maximum number of suspicious replicas on an RSE before that RSE is considered problematic and the suspicious replicas on that RSE are declared "TEMPORARY_UNAVAILABLE". Default value is 5.')
     parser.add_argument('--json-file-name', action="store", default="/opt/rucio/etc/suspicious_replica_recoverer.json", type=str, help='Name of the json file that that contains the policies which will be used by the suspicious replica recoverer.')
     parser.add_argument('--sleep-time', action="store", default=3600, type=int, help='Concurrency control: Thread sleep time after each chunk of work.')
     parser.add_argument('--active-mode', action="store_true", default=False, help='If NOT specified, the daemon will run without taking any actions on any files. In either case, the log file will be produced normally.')

{rucio-35.4.0 → rucio-35.4.1}/lib/rucio/client/didclient.py

@@ -92,7 +92,7 @@ class DIDClient(BaseClient):
             self,
             scope: str,
             name: str,
-            did_type: Literal['FILE', 'DATASET', 'CONTAINER'],
+            did_type: Literal['DATASET', 'CONTAINER'],
             statuses: Optional["Mapping[str, Any]"] = None,
             meta: Optional["Mapping[str, Any]"] = None,
             rules: Optional["Sequence[Mapping[str, Any]]"] = None,
@@ -105,7 +105,7 @@ class DIDClient(BaseClient):
 
         :param scope: The scope name.
         :param name: The data identifier name.
-        :param did_type: The data identifier type (file|dataset|container).
+        :param did_type: The data identifier type (dataset|container).
         :param statuses: Dictionary with statuses, e.g. {'monotonic':True}.
         :param meta: Meta-data associated with the data identifier is represented using key/value pairs in a dictionary.
         :param rules: Replication rules associated with the data identifier. A list of dictionaries, e.g., [{'copies': 2, 'rse_expression': 'TIERS1'}, ].

{rucio-35.4.0 → rucio-35.4.1}/lib/rucio/core/subscription.py

@@ -17,7 +17,7 @@ import logging
 import re
 from configparser import NoOptionError, NoSectionError
 from json import dumps
-from typing import TYPE_CHECKING
+from typing import TYPE_CHECKING, Any, Optional
 
 from sqlalchemy import and_, delete, func, select
 from sqlalchemy.exc import IntegrityError, NoResultFound, StatementError
@@ -26,12 +26,11 @@ from sqlalchemy.orm import aliased
 from rucio.common.config import config_get_bool
 from rucio.common.exception import RucioException, SubscriptionDuplicate, SubscriptionNotFound
 from rucio.db.sqla import models
-from rucio.db.sqla.constants import SubscriptionState
+from rucio.db.sqla.constants import RuleState, SubscriptionState
 from rucio.db.sqla.session import read_session, stream_session, transactional_session
 
 if TYPE_CHECKING:
     from collections.abc import Callable, Iterator
-    from typing import Any, Optional
 
     from sqlalchemy.orm import Session
 
@@ -46,10 +45,10 @@ def add_subscription(name: str,
                      filter_: str,
                      replication_rules: str,
                      comments: str,
-                     lifetime: "Optional[int]" = None,
-                     retroactive: "Optional[bool]" = False,
-                     dry_run: "Optional[bool]" = False,
-                     priority: "Optional[int]" = 3,
+                     lifetime: Optional[int] = None,
+                     retroactive: Optional[bool] = False,
+                     dry_run: Optional[bool] = False,
+                     priority: Optional[int] = 3,
                      *, session: "Session") -> str:
     """
     Adds a new subscription which will be verified against every new added file and dataset
@@ -151,7 +150,7 @@ def update_subscription(name: str,
         keep_history = config_get_bool('subscriptions', 'keep_history')
     except (NoOptionError, NoSectionError, RuntimeError):
         keep_history = False
-    values = {'state': SubscriptionState.UPDATED}
+    values: dict[str, Any] = {'state': SubscriptionState.UPDATED}
     if 'filter' in metadata and metadata['filter']:
         values['filter'] = dumps(metadata['filter'])
     if 'replication_rules' in metadata and metadata['replication_rules']:
@@ -212,9 +211,9 @@ def update_subscription(name: str,
 
 
 @stream_session
-def list_subscriptions(name: "Optional[str]" = None,
+def list_subscriptions(name: Optional[str] = None,
                        account: "Optional[InternalAccount]" = None,
-                       state: "Optional[SubscriptionState]" = None,
+                       state: Optional[SubscriptionState] = None,
                        *, session: "Session",
                        logger: "LoggerFunction" = logging.log) -> "Iterator[SubscriptionType]":
     """
@@ -282,7 +281,13 @@ def delete_subscription(subscription_id: str, *, session: "Session") -> None:
 
 
 @stream_session
-def list_subscription_rule_states(name=None, account=None, *, session: "Session", logger=logging.log):
+def list_subscription_rule_states(
+    name: Optional[str] = None,
+    account: Optional["InternalAccount"] = None,
+    *,
+    session: "Session",
+    logger: "LoggerFunction" = logging.log
+) -> "Iterator[tuple[InternalAccount, str, RuleState, int]]":
     """Returns a list of with the number of rules per state for a subscription.
 
     :param name:               Name of the subscription
@@ -311,7 +316,7 @@ def list_subscription_rule_states(name=None, account=None, *, session: "Session"
             )
 
         if account:
-            if '*' in account.internal:
+            if account.internal and '*' in account.internal:
                 account_str = account.internal.replace('*', '%')
                 stmt = stmt.where(
                     subscription.account.like(account_str)
@@ -332,11 +337,11 @@ def list_subscription_rule_states(name=None, account=None, *, session: "Session"
     )
 
     for row in session.execute(stmt).all():
-        yield row
+        yield row._tuple()
 
 
 @read_session
-def get_subscription_by_id(subscription_id, *, session: "Session"):
+def get_subscription_by_id(subscription_id: str, *, session: "Session") -> "SubscriptionType":
     """
     Get a specific subscription by id.
 

{rucio-35.4.0 → rucio-35.4.1}/lib/rucio/db/sqla/sautils.py

@@ -17,18 +17,26 @@ SQLAlchemy utilities
 
 '''
 
+from typing import TYPE_CHECKING, Any, Union
+
 from sqlalchemy.ext.compiler import compiles
 from sqlalchemy.sql.expression import ClauseElement, Executable
 
+if TYPE_CHECKING:
+    from sqlalchemy import Table
+    from sqlalchemy.engine.interfaces import SQLCompiler
+    from sqlalchemy.sql.schema import Column
+    from sqlalchemy.sql.selectable import Select
+
 
 class InsertFromSelect(Executable, ClauseElement):
-    def __init__(self, insert_spec, select):
+    def __init__(self, insert_spec: "Union[Table, list[Column]]", select: "Select") -> None:
         self.insert_spec = insert_spec
         self.select = select
 
 
 @compiles(InsertFromSelect)
-def visit_insert_from_select(element, compiler, **kw):
+def visit_insert_from_select(element: InsertFromSelect, compiler: "SQLCompiler", **kw: Any) -> str:
     if isinstance(element.insert_spec, list):
         columns = []
         for column in element.insert_spec:

{rucio-35.4.0 → rucio-35.4.1}/lib/rucio/vcsversion.py

@@ -4,8 +4,8 @@ This file is automatically generated; Do not edit it. :)
 '''
 VERSION_INFO = {
     'final': True,
-    'version': '35.4.0',
+    'version': '35.4.1',
     'branch_nick': 'release-35-LTS',
-    'revision_id': '3fb618d968f2efbe7431b303ef916d5cdf07d133',
-    'revno': 13156
+    'revision_id': 'b041c5990126a6987d2f884680869fc93b609acc',
+    'revno': 13162
 }