rcarpa

Race condition in multithreading environment due to usage of os.chdir

4

Hi, I recently ran into a race condition when using two different instances of `git.Repo` objects in two different threads. Inside `repo.index.add()`, some functions are decorated with the `git_working_dir` decorator....

Motivation ---------- ``` conveyor-finisher[ac37097:19/47]: Something unexpected happened when handling replicas on RequestType.TRANSFER rule 2e8ca6bfd9fe44ff94eafee7b2dd36f9 'type' File "/usr/local/lib/python3.6/site-packages/rucio/daemons/conveyor/finisher.py", line 349, in __handle_terminated_replicas __update_bulk_replicas(replicas[req_type][rule_id], logger=logger) File "/usr/local/lib/python3.6/site-packages/rucio/db/sqla/session.py", line 388, in new_funct result...

Motivation ---------- Because of this change: https://github.com/rucio/rucio/commit/183b46a0f7d1ac67bca8233013770021967da442#diff-1085ef0a38068abd1b394d8a69b43fa31b34a3a3eb1686fc3727f3638853c7cdL3000 we get the following exception ``` Database exception. Details: (cx_Oracle.DatabaseError) ORA-32460: maximum number of private temporary tables per session exceeded [SQL: CREATE PRIVATE...

reaper fails when run with --scheme on rses without given scheme

1

Motivation ---------- Recently this error was reported. reaper was running on this rse with --scheme != 'https', the only scheme supported by this rse. ``` 2022-08-09 16:23:54,929 root 10 DEBUG...

foreign key error on deleting dids in reaper

4

Motivation ---------- ``` 2022-07-13 08:32:24,129 root 1 CRITICAL reaper[2b317444f2:0/41]: Exception Traceback (most recent call last): File "/usr/local/lib64/python3.6/site-packages/sqlalchemy/engine/base.py", line 1803, in _execute_context cursor, statement, parameters, context File "/usr/local/lib64/python3.6/site-packages/sqlalchemy/engine/default.py", line 732, in...

Motivation ---------- Conveyor throttler is functional and pretty well tested. However, there are many improvements which must be done: - [ ] there is no user-facing cli/api to configure it...

Motivation ---------- Rucio has tons of metrics collected all over the code, but the naming convention is not consistent. Design a metric naming convention, and rename all metrics.

Motivation ---------- This ticket comes from a discussion with Katy from CMS. Currently, rucio has a hard-coded orded in which sources are evaluated https://github.com/rucio/rucio/blob/15cfab2ad37ec0584e93884f99bee48b333174d1/lib/rucio/core/transfer.py#L934 This order may not necessarily be...

Motivation ---------- ``` Internal Error Duplicated timeseries in CollectorRegistry: {'rucio_trace_trace_total', 'rucio_trace_trace', 'rucio_trace_trace_created'} File "/usr/local/lib/python3.6/site-packages/rucio/web/rest/flaskapi/v1/common.py", line 54, in dispatch_request return super(ErrorHandlingMethodView, self).dispatch_request(*args, **kwargs) File "/usr/local/lib/python3.6/site-packages/flask/views.py", line 158, in dispatch_request return current_app.ensure_sync(meth)(*args,...