rucio-clients 37.7.1__tar.gz → 38.0.0__tar.gz

{rucio_clients-37.7.1 → rucio_clients-38.0.0}/AUTHORS.rst

@@ -69,6 +69,7 @@ Individual contributors to the source code
 - Maximilian Linhoff, <maximilian.linhoff@tu-dortmund.de>, 2024
 - Eric Banzuzi, <eric.banzuzi@gmail.com>, 2024
 - Paul Millar, <paul.millar@desy.de>, 2025
+- Vimalan S <vimalan.github@gmail.com>, 2025
 
 Organisations employing contributors
 ------------------------------------

{rucio_clients-37.7.1 → rucio_clients-38.0.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: rucio-clients
-Version: 37.7.1
+Version: 38.0.0
 Summary: Rucio Client Lite Package
 Home-page: https://rucio.cern.ch/
 Author: Rucio
@@ -22,7 +22,7 @@ License-File: AUTHORS.rst
 Requires-Dist: click
 Requires-Dist: requests
 Requires-Dist: urllib3
-Requires-Dist: dogpile-cache<=1.2.2
+Requires-Dist: dogpile-cache
 Requires-Dist: packaging
 Requires-Dist: tabulate
 Requires-Dist: jsonschema

{rucio_clients-37.7.1 → rucio_clients-38.0.0}/bin/rucio

@@ -54,6 +54,7 @@ def map_legacy_command() -> Optional[list[str]]:
         command_map = {
             "list-file-replicas": ["replica", "list", "file"],
             "list-dataset-replicas": ["replica", "list", "dataset"],
+            "list-datasets-rse": ["replica", "list", "dataset", "--rse"],
             "add-dataset": ["did", "add", "--type dataset"],
             "add-container": ["did", "add", "--type container"],
             "attach": ["did", "content", "add"],
@@ -96,7 +97,7 @@ def map_legacy_command() -> Optional[list[str]]:
 
 
 if __name__ == "__main__":
-    commands = ("account", "config", "did", "replica", "rse", "rule", "scope", "subscription", "ping", "whoami", "test-server", "lifetime-exception", "upload", "download")
+    commands = ("account", "config", "did", "replica", "rse", "rule", "scope", "subscription", "ping", "whoami", "test-server", "lifetime-exception", "upload", "download", "opendata")
 
     parser = argparse.ArgumentParser(add_help=False)
     # Check for legacy flag

{rucio_clients-37.7.1 → rucio_clients-38.0.0}/etc/rucio.cfg.template

@@ -73,8 +73,8 @@ default_mail_from = spamspamspam@cern.ch
 
 [database]
 default = sqlite:////tmp/rucio.db
-#default = oracle://_____________:___________@(DESCRIPTION=(ADDRESS=(PROTOCOL=TCP)(HOST=_________)(PORT=______))(ADDRESS=(PROTOCOL=TCP)(HOST=_________)(PORT=_____))(CONNECT_DATA=(SERVER=DEDICATED)(SERVICE_NAME=__________)))
-#default = oracle://_____________:___________@(DESCRIPTION=(ADDRESS=(PROTOCOL=TCP)(HOST=127.0.0.1)(PORT=______))(CONNECT_DATA=(SERVER=DEDICATED)(SERVICE_NAME=_____________)))
+#default = oracle+oracledb://_____________:___________@(DESCRIPTION=(ADDRESS=(PROTOCOL=TCP)(HOST=_________)(PORT=______))(ADDRESS=(PROTOCOL=TCP)(HOST=_________)(PORT=_____))(CONNECT_DATA=(SERVER=DEDICATED)(SERVICE_NAME=__________)))
+#default = oracle+oracledb://_____________:___________@(DESCRIPTION=(ADDRESS=(PROTOCOL=TCP)(HOST=127.0.0.1)(PORT=______))(CONNECT_DATA=(SERVER=DEDICATED)(SERVICE_NAME=_____________)))
 #schema=atlas_rucio  # only for cern oracle
 #default = mysql+pymysql://rucio:rucio@localhost/rucio
 #default = postgresql+psycopg://rucio:rucio@localhost/rucio

{rucio_clients-37.7.1 → rucio_clients-38.0.0}/lib/rucio/alembicrevision.py

@@ -12,4 +12,4 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-ALEMBIC_REVISION = '30d5206e9cad'  # the current alembic head revision
+ALEMBIC_REVISION = 'a62db546a1f1'  # the current alembic head revision

{rucio_clients-37.7.1 → rucio_clients-38.0.0}/lib/rucio/cli/bin_legacy/rucio.py

@@ -20,12 +20,10 @@ import os
 import signal
 import sys
 import time
-import traceback
 import unittest
 import uuid
 from copy import deepcopy
 from datetime import datetime
-from logging import DEBUG
 from typing import TYPE_CHECKING, Optional
 
 from rich.console import Console
@@ -45,13 +43,13 @@ from rucio.common.client import detect_client_location
 from rucio.common.config import config_get, config_get_float
 from rucio.common.constants import ReplicaState
 from rucio.common.exception import (
-    DIDFilterSyntaxError,
-    DuplicateCriteriaInDIDFilter,
     DuplicateRule,
     InputValidationError,
     InvalidObject,
     InvalidType,
+    RSENotFound,
     RucioException,
+    ScopeNotFound,
     UnsupportedOperation,
 )
 from rucio.common.extra import import_extras
@@ -109,8 +107,7 @@ def ping(args, client, logger, console, spinner):
     if server_info:
         print(server_info['version'])
         return SUCCESS
-    logger.error('Ping failed')
-    return FAILURE
+    raise RucioException('Ping failed')
 
 
 @exception_handler
@@ -232,11 +229,9 @@ def list_file_replicas(args, client, logger, console, spinner):
     table_data = []
     dids = []
     if args.missing and not args.rses:
-        print('Cannot use --missing without specifying a RSE')
-        return FAILURE
+        raise InputValidationError('Cannot use --missing without specifying a RSE')
     if args.link and ':' not in args.link:
-        print('The substitution parameter must equal --link="/pfn/dir:/dst/dir"')
-        return FAILURE
+        raise ValueError('The substitution parameter must equal --link="/pfn/dir:/dst/dir"')
 
     if cli_config == 'rich':
         spinner.update(status='Fetching file replicas')
@@ -398,14 +393,13 @@ def attach(args, client, logger, console, spinner):
 
     if args.fromfile:
         if len(dids) > 1:
-            logger.error("If --fromfile option is active, only one file is supported. The file should contain a list of dids, one per line.")
-            return FAILURE
+            raise ValueError('If --fromfile option is active, only one file is supported. The file should contain a list of dids, one per line.')
         try:
             f = open(dids[0], 'r')
             dids = [did.rstrip() for did in f.readlines()]
-        except OSError:
+        except OSError as error:
             logger.error("Can't open file '" + dids[0] + "'.")
-            return FAILURE
+            raise OSError from error
 
     dids = [{'scope': get_scope(did, client)[0], 'name': get_scope(did, client)[1]} for did in dids]
     if len(dids) <= limit:
@@ -468,35 +462,16 @@ def list_dids(args, client, logger, console, spinner):
         name = '*'
 
     if scope not in client.list_scopes():
-        logger.error('Scope not found.')
-        return FAILURE
+        raise ScopeNotFound
 
     if args.recursive and '*' in name:
-        logger.error('Option recursive cannot be used with wildcards.')
-        return FAILURE
+        raise InputValidationError('Option recursive cannot be used with wildcards.')
     else:
         if filters:
             if ('name' in filters) and (name != '*'):
-                logger.error('Must have a wildcard in did name if filtering by name.')
-                return FAILURE
+                raise ValueError('Must have a wildcard in did name if filtering by name.')
 
-    try:
-        filters, type_ = parse_did_filter_from_string_fe(args.filter, name)
-    except InvalidType as error:
-        logger.error(error)
-        return FAILURE
-    except DuplicateCriteriaInDIDFilter as error:
-        logger.error(error)
-        return FAILURE
-    except DIDFilterSyntaxError as error:
-        logger.error(error)
-        return FAILURE
-    except ValueError as error:
-        logger.error(error)
-        return FAILURE
-    except Exception as e:
-        logger.error(e)
-        return FAILURE
+    filters, type_ = parse_did_filter_from_string_fe(args.filter, name)
 
     if cli_config == 'rich':
         spinner.update(status='Fetching DIDs')
@@ -531,8 +506,7 @@ def list_dids_extended(args, client, logger, console, spinner):
 
     List the data identifiers for a given scope (DEPRECATED).
     """
-    logger.error("This command has been deprecated. Please use list_dids instead.")
-    return FAILURE
+    raise UnsupportedOperation('This command has been deprecated. Please use list_dids instead.')
 
 
 @exception_handler
@@ -542,15 +516,17 @@ def list_scopes(args, client, logger, console, spinner):
 
     List scopes.
     """
-    # For the moment..
-
-    if cli_config == 'rich':
+    if (cli_config == 'rich') or (not args.csv):
         spinner.update(status='Fetching scopes')
         spinner.start()
 
-    scopes = client.list_scopes()
-    if cli_config == 'rich':
-        table = generate_table([[scope] for scope in sorted(scopes)], headers=['SCOPE'], col_alignments=['left'])
+    if args.account:
+        scopes = client.list_scopes_for_account(args.account)
+    else:
+        scopes = client.list_scopes()
+    if (cli_config == 'rich') and (not args.csv):
+        scopes = [[scope] for scope in sorted(scopes)]
+        table = generate_table(scopes, headers=['SCOPE'], col_alignments=['left'])
         spinner.stop()
         print_output(table, console=console, no_pager=args.no_pager)
     else:
@@ -813,8 +789,7 @@ def list_parent_dids(args, client, logger, console, spinner):
         else:
             print(tabulate(table_data, tablefmt=tablefmt, headers=['SCOPE:NAME', '[DID TYPE]']))
     else:
-        print('At least one option has to be given. Use -h to list the options.')
-        return FAILURE
+        raise InputValidationError('At least one option has to be given. Use -h to list the options.')
     return SUCCESS
 
 
@@ -931,13 +906,11 @@ def upload(args, client, logger, console, spinner):
     Upload files into Rucio
     """
     if args.lifetime and args.expiration_date:
-        logger.error("--lifetime and --expiration-date cannot be specified at the same time.")
-        return FAILURE
+        raise InputValidationError("--lifetime and --expiration-date cannot be specified at the same time.")
     elif args.expiration_date:
         expiration_date = datetime.strptime(args.expiration_date, "%Y-%m-%d-%H:%M:%S")
         if expiration_date < datetime.utcnow():
-            logger.error("The specified expiration date should be in the future!")
-            return FAILURE
+            raise ValueError("The specified expiration date should be in the future!")
         args.lifetime = (expiration_date - datetime.utcnow()).total_seconds()
 
     dsscope = None
@@ -1023,23 +996,18 @@ def download(args, client, logger, console, spinner):
     """
     # Input validation
     if not args.dids and not args.filter and not args.metalink_file:
-        logger.error('At least one did is mandatory')
-        return FAILURE
+        raise InputValidationError('At least one did is mandatory')
     elif not args.dids and args.filter and not args.scope:
-        logger.error('The argument scope is mandatory')
-        return FAILURE
+        raise InputValidationError('The argument scope is mandatory')
 
     if args.filter and args.metalink_file:
-        logger.error('Arguments filter and metalink cannot be used together.')
-        return FAILURE
+        raise InputValidationError('Arguments filter and metalink cannot be used together.')
 
     if args.dids and args.metalink_file:
-        logger.error('Arguments dids and metalink cannot be used together.')
-        return FAILURE
+        raise InputValidationError('Arguments dids and metalink cannot be used together.')
 
     if args.ignore_checksum and args.check_local_with_filesize_only:
-        logger.error('Arguments ignore-checksum and check-local-with-filesize-only cannot be used together.')
-        return FAILURE
+        raise InputValidationError('Arguments ignore-checksum and check-local-with-filesize-only cannot be used together.')
 
     trace_pattern = {}
 
@@ -1083,16 +1051,12 @@ def download(args, client, logger, console, spinner):
             filters, type_ = parse_did_filter_from_string(args.filter)
             if args.scope:
                 filters['scope'] = args.scope
-        except InvalidType as error:
+        except (InvalidType, ValueError) as error:
             logger.error(error)
-            return FAILURE
-        except ValueError as error:
-            logger.error(error)
-            return FAILURE
+            raise error
         except Exception as error:
-            logger.error(error)
             logger.error("Invalid Filter. Filter must be 'key=value', 'key>=value', 'key>value', 'key<=value', 'key<value'")
-            return FAILURE
+            raise error
         item_defaults['filters'] = filters
 
     if not args.pfn:
@@ -1148,8 +1112,7 @@ def download(args, client, logger, console, spinner):
 
             download_rse = _get_rse_for_pfn(replicas, args.pfn)
             if download_rse is None:
-                logger.error("Could not find RSE for pfn %s", args.pfn)
-                return FAILURE
+                raise RSENotFound("Could not find RSE for pfn %s" % args.pfn)
             else:
                 item_defaults['rse'] = download_rse
 
@@ -1354,8 +1317,7 @@ def delete_rule(args, client, logger, console, spinner):
     except ValueError:
         # Otherwise, trying to extract the scope, name from args.rule_id
         if not args.rses:
-            logger.error('A RSE expression must be specified if you do not provide a rule_id but a DID')
-            return FAILURE
+            raise InputValidationError('A RSE expression must be specified if you do not provide a rule_id but a DID')
         scope, name = get_scope(args.rule_id, client)
         rules = client.list_did_rules(scope=scope, name=name)
         if args.rule_account is None:
@@ -1372,8 +1334,7 @@ def delete_rule(args, client, logger, console, spinner):
                 client.delete_replication_rule(rule_id=rule['id'], purge_replicas=args.purge_replicas)
                 deletion_success = True
         if not deletion_success:
-            logger.error('No replication rule was deleted from the DID')
-            return FAILURE
+            raise RucioException('No replication rule was deleted from the DID')
     return SUCCESS
 
 
@@ -1394,8 +1355,8 @@ def update_rule(args, client, logger, console, spinner):
         elif args.locked.title() == "False":
             options['locked'] = False
         else:
-            logger.error('Locked must be True or False')
-            return FAILURE
+            raise InputValidationError('Locked must be True or False')
+
     if args.comment:
         options['comment'] = args.comment
     if args.rule_account:
@@ -1410,8 +1371,7 @@ def update_rule(args, client, logger, console, spinner):
         options['source_replica_expression'] = None if args.source_replica_expression.lower() == 'none' else args.source_replica_expression
     if args.cancel_requests:
         if 'state' not in options:
-            logger.error('--stuck or --suspend must be specified when running --cancel-requests')
-            return FAILURE
+            raise InputValidationError('--stuck or --suspend must be specified when running --cancel-requests')
         options['cancel_requests'] = True
     if args.priority:
         options['priority'] = int(args.priority)
@@ -1578,12 +1538,11 @@ def list_rules(args, client, logger, console, spinner):
     elif args.rule_account:
         rules = client.list_account_rules(account=args.rule_account)
     elif args.subscription:
-        account = args.subscription[0]
-        name = args.subscription[1]
+        account = args.rule_account if args.rule_account else client.account
+        name = args.subscription
         rules = client.list_subscription_rules(account=account, name=name)
     else:
-        print('At least one option has to be given. Use -h to list the options.')
-        return FAILURE
+        raise InputValidationError('At least one option has to be given. Use -h to list the options.')
     if args.csv:
         for rule in rules:
             print(rule['id'],
@@ -1990,21 +1949,18 @@ def add_lifetime_exception(args, client, logger, console, spinner):
     """
 
     if not args.reason:
-        logger.error('reason for the extension is mandatory')
-        return FAILURE
+        raise InputValidationError('reason for the extension is mandatory')
     reason = args.reason
     if not args.expiration:
-        logger.error('expiration is mandatory')
-        return FAILURE
+        raise InputValidationError('expiration is mandatory')
     try:
         expiration = datetime.strptime(args.expiration, "%Y-%m-%d")
     except Exception as err:
-        logger.error(err)
-        return FAILURE
+        msg = f'Cannot parse expiration date: {err}'
+        raise ValueError(msg)
 
     if not args.inputfile:
-        logger.error('inputfile is mandatory')
-        return FAILURE
+        raise InputValidationError('inputfile is mandatory')
     with open(args.inputfile) as infile:
         # Deduplicate the content of the input file and ignore empty lines.
         dids = set(did for line in infile if (did := line.strip()))
@@ -2067,22 +2023,8 @@ def add_lifetime_exception(args, client, logger, console, spinner):
     if not datasets:
         logger.error('Nothing to submit')
         return SUCCESS
-    try:
-        client.add_exception(dids=datasets, account=client.account, pattern='', comments=reason, expires_at=expiration)
-    except UnsupportedOperation as err:
-        logger.error(err)
-        return FAILURE
-    except Exception:
-        error_message = 'Failure to submit exception. Please retry.'
-        if cli_config == 'rich':
-            if logger.level == DEBUG:
-                logger.exception(error_message)
-            else:
-                logger.error(error_message)
-        else:
-            logger.error(error_message)
-            logger.debug(traceback.format_exc())
-        return FAILURE
+
+    client.add_exception(dids=datasets, account=client.account, pattern='', comments=reason, expires_at=expiration)
 
     logger.info('Exception successfully submitted. Summary below:')
     for key, data in error_summary.items():
@@ -2499,6 +2441,8 @@ You can filter by key/value, e.g.::
     ''')
 
     scope_list_parser.set_defaults(function=list_scopes)
+    scope_list_parser.add_argument("--csv", action="store_true", default=False, help="Comma Separated Value output.")
+    scope_list_parser.add_argument('--account', help='Filter scopes by account')
 
     # The close command
     close_parser = subparsers.add_parser('close', help='Close a dataset or container.')
@@ -2699,7 +2643,7 @@ You can filter by account::
     list_rules_parser.add_argument('--csv', dest='csv', action='store_true', default=False, help='Comma Separated Value output')
     list_rules_parser.add_argument('--file', dest='file', action='store', help='List associated rules of an affected file')
     list_rules_parser.add_argument('--account', dest='rule_account', action='store', help='List by account')
-    list_rules_parser.add_argument('--subscription', dest='subscription', action='store', help='List by account and subscription name', metavar=('ACCOUNT', 'SUBSCRIPTION'), nargs=2)
+    list_rules_parser.add_argument('--subscription', dest='subscription', action='store', help='List by subscription name')
 
     # The list_rules_history command
     list_rules_history_parser = subparsers.add_parser('list-rules-history', help='List replication rules history for a DID.')

{rucio_clients-37.7.1 → rucio_clients-38.0.0}/lib/rucio/cli/bin_legacy/rucio_admin.py

@@ -37,8 +37,11 @@ from rucio.cli.utils import exception_handler, get_client, setup_gfal2_logger, s
 from rucio.client.richclient import MAX_TRACEBACK_WIDTH, MIN_CONSOLE_WIDTH, CLITheme, generate_table, get_cli_config, get_pager, print_output, setup_rich_logger
 from rucio.common.constants import RseAttr
 from rucio.common.exception import (
+    InputValidationError,
+    InvalidObject,
     ReplicaNotFound,
     RSEOperationNotSupported,
+    RucioException,
 )
 from rucio.common.extra import import_extras
 from rucio.common.utils import StoreAndDeprecateWarningAction, chunks, clean_pfns, construct_non_deterministic_pfn, extract_scope, get_bytes_value_from_string, parse_response, render_json, setup_logger, sizefmt
@@ -222,8 +225,10 @@ def set_limits(args, client, logger, console, spinner):
             try:
                 byte_limit = int(limit_input)
             except ValueError:
-                logger.error('The limit could not be set. Either you misspelled infinity or your input could not be converted to integer or you used a wrong pattern. Please use a format like 10GB with B,KB,MB,GB,TB,PB as units (not case sensitive)')
-                return FAILURE
+                msg = f'\
+                    The limit could not be set. Either you misspelled infinity or your input ({args.bytes}) could not be converted to integer or you used a wrong pattern. \
+                    Please use a format like 10GB with B,KB,MB,GB,TB,PB as units (not case sensitive)'
+                raise InputValidationError(msg)
 
     client.set_account_limit(account=args.account, rse=args.rse, bytes_=byte_limit, locality=locality)
     print('Set account limit for account %s on RSE %s: %s' % (args.account, args.rse, sizefmt(byte_limit, True)))
@@ -266,12 +271,10 @@ def identity_add(args, client, logger, console, spinner):
 
     """
     if args.email == "":
-        logger.error('Error: --email argument can\'t be an empty string. Failed to grant an identity access to an account')
-        return FAILURE
+        raise InputValidationError('Error: --email argument can\'t be an empty string. Failed to grant an identity access to an account')
 
     if args.authtype == 'USERPASS' and not args.password:
-        logger.error('missing --password argument')
-        return FAILURE
+        raise InputValidationError('Missing --password argument')
 
     client.add_identity(account=args.account, identity=args.identity, authtype=args.authtype, email=args.email, password=args.password)
     print('Added new identity to account: %s-%s' % (args.identity, args.account))
@@ -634,8 +637,7 @@ def add_protocol_rse(args, client, logger, console, spinner):
     if args.ext_attr_json:
         proto['extended_attributes'] = args.ext_attr_json
     if proto['scheme'] == 'srm' and not args.web_service_path:
-        print('Error: space-token and web-service-path must be provided for SRM endpoints.')
-        return FAILURE
+        raise InputValidationError('Error: space-token and web-service-path must be provided for SRM endpoints.')
     if args.space_token:
         proto['extended_attributes']['space_token'] = args.space_token
     if args.web_service_path:
@@ -764,7 +766,7 @@ def list_scopes(args, client, logger, console, spinner):
     List scopes.
 
     """
-    if cli_config == 'rich':
+    if (cli_config == 'rich') and (not args.csv):
         spinner.update(status='Fetching scopes')
         spinner.start()
 
@@ -772,15 +774,14 @@ def list_scopes(args, client, logger, console, spinner):
         scopes = client.list_scopes_for_account(args.account)
     else:
         scopes = client.list_scopes()
-    if cli_config == 'rich':
-        scopes = [[scope] for scope in sorted(scopes) if 'mock' not in scope]
+    if (cli_config == 'rich') and (not args.csv):
+        scopes = [[scope] for scope in sorted(scopes)]
         table = generate_table(scopes, headers=['SCOPE'], col_alignments=['left'])
         spinner.stop()
         print_output(table, console=console, no_pager=args.no_pager)
     else:
         for scope in scopes:
-            if 'mock' not in scope:
-                print(scope)
+            print(scope)
     return SUCCESS
 
 
@@ -1042,8 +1043,7 @@ def __declare_bad_file_replicas_by_lfns(args: object, client) -> object:
     Declare a list of bad replicas using RSE name, scope and list of LFNs.
     """
     if not args.scope or not args.rse:
-        print("--lfns requires using --rse and --scope")
-        return FAILURE
+        raise InputValidationError("--lfns requires using --rse and --scope")
     reason = args.reason
     scope = args.scope
     rse = args.rse
@@ -1094,8 +1094,8 @@ def declare_bad_file_replicas(args, client, logger, console, spinner):
             scope, name = get_scope(bad_file, client)
             did_info = client.get_did(scope, name)
             if did_info['type'].upper() != 'FILE' and not args.allow_collection:
-                print('DID %s:%s is a collection and --allow-collection was not specified.' % (scope, name))
-                return FAILURE
+                msg = f'DID {scope}:{name} is a collection and --allow-collection was not specified.'
+                raise InputValidationError(msg)
             replicas = [replica for rep in client.list_replicas([{'scope': scope, 'name': name}])
                         for replica in list(rep['pfns'].keys())]
             bad_files_pfns.extend(replicas)
@@ -1173,15 +1173,15 @@ def declare_temporary_unavailable_replicas(args, client, logger, console, spinne
             for line in infile:
                 bad_file = line.rstrip('\n')
                 if '://' not in bad_file:
-                    print('%s is not a valid PFN. Aborting', bad_file)
-                    return FAILURE
+                    msg = f'{bad_file} is not a valid PFN. Aborting'
+                    raise InvalidObject(msg)
                 if bad_file != '':
                     bad_files.append(bad_file)
     else:
         bad_files = args.listbadfiles
 
     if args.duration is None:
-        raise ValueError("Duration should have been set, something went wrong!")
+        raise InputValidationError("Duration should have been set, something went wrong!")
 
     expiration_date = (datetime.datetime.utcnow() + datetime.timedelta(seconds=args.duration)).isoformat()
 
@@ -1246,8 +1246,7 @@ def list_pfns(args, client, logger, console, spinner):
                                            path if not path.startswith('/') else path[1:]])
                             print(pfn)
                 else:
-                    logger.error('Unexpected error')
-                    return FAILURE
+                    raise RucioException
             else:
                 print(result)
     return SUCCESS
@@ -1279,7 +1278,7 @@ def import_data(args, client, logger, console, spinner):
         else:
             print('There was problem with decoding your file.')
             print(error)
-        return FAILURE
+        raise ValueError from error
     except OSError as error:
         if cli_config == 'rich':
             spinner.stop()
@@ -1288,7 +1287,7 @@ def import_data(args, client, logger, console, spinner):
         else:
             print('There was a problem with reading your file.')
             print(error)
-        return FAILURE
+        raise OSError from error
 
     if data:
         client.import_data(data)
@@ -1304,7 +1303,7 @@ def import_data(args, client, logger, console, spinner):
             print_output('Nothing to import.', console=console, no_pager=True)
         else:
             print('Nothing to import.')
-        return FAILURE
+        raise ValueError
 
 
 @exception_handler
@@ -1342,7 +1341,7 @@ def export_data(args, client, logger, console, spinner):
         else:
             print('There was a problem with reading your file.')
             print(error)
-        return FAILURE
+        raise OSError from error
 
 
 @exception_handler
@@ -2131,6 +2130,7 @@ def get_parser():
                                                           '\n')
     list_scope_parser.set_defaults(which='list_scopes')
     list_scope_parser.add_argument('--account', dest='account', action='store', help='Account name')
+    list_scope_parser.add_argument('--csv', action='store_true', help='Output a list of scopes as a csv')
 
     # The config subparser
     config_parser = subparsers.add_parser('config',

{rucio_clients-37.7.1 → rucio_clients-38.0.0}/lib/rucio/cli/command.py

@@ -76,6 +76,7 @@ class LazyGroup(click.Group):
         "scope": "rucio.cli.scope.scope",
         "subscription": "rucio.cli.subscription.subscription",
         "upload": "rucio.cli.upload.upload_command",
+        "opendata": "rucio.cli.opendata.opendata",
     },
     context_settings={"help_option_names": ["-h", "--help"]}
 )  # TODO: Implement https://click.palletsprojects.com/en/stable/options/#dynamic-defaults-for-prompts for args from config or os

{rucio_clients-37.7.1 → rucio_clients-38.0.0}/lib/rucio/cli/did.py

@@ -117,7 +117,7 @@ def content():
 
 
 @content.command("history")
-@click.argument("dids", nargs=-1)
+@click.argument("dids", nargs=-1, required=True)
 @click.pass_context
 def content_history(ctx, dids):
     """List the content history of a collection-type DID"""
@@ -146,7 +146,7 @@ def content_remove(ctx, dids, from_did):
 
 
 @content.command("list")
-@click.argument("dids", nargs=-1)
+@click.argument("dids", nargs=-1, required=True)
 @click.option("--short", is_flag=True, default=False, help="Just dump the list of DIDs.")
 @click.pass_context
 def content_list_(ctx, dids, short):