Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Transfer time and wait time metrics aren't correctly recorded #6482

Closed
rcarpa opened this issue Jan 24, 2024 · 0 comments
Closed

Transfer time and wait time metrics aren't correctly recorded #6482

rcarpa opened this issue Jan 24, 2024 · 0 comments
Assignees
@rcarpa
Labels
bug Transfers
Milestone
33.4.0

Comments

@rcarpa
Copy link
Contributor

rcarpa commented Jan 24, 2024

We don't use the correct fields to compute the metrics. We take them from the "request" object which comes from the database,
but these fields are NULL because they are fetched from the DB before actually setting them there a couple of lines later.
@rcarpa rcarpa self-assigned this Jan 24, 2024
rcarpa added a commit to rcarpa/rucio that referenced this issue Jan 24, 2024
@rcarpa
Transfer: fix transfer time and wait time metrics. Closes rucio#6482
aba6463 
Getting the started_at and transferred_at from the request
is useless, because this is the request _before_ its update
in the database. The information must be retrieved from the
transfertool message.

Also, record the transfer time without including the wait time.
@bari12 bari12 closed this as completed in 2fe0690 Jan 26, 2024
bari12 pushed a commit that referenced this issue Jan 26, 2024
@rcarpa @bari12
Transfer: fix transfer time and wait time metrics. Closes #6482
2ed4cdd 
Getting the started_at and transferred_at from the request
is useless, because this is the request _before_ its update
in the database. The information must be retrieved from the
transfertool message.

Also, record the transfer time without including the wait time.
@bari12 bari12 added this to the 33.3.1 / 33.4.0 milestone Jan 26, 2024
voetberg pushed a commit to voetberg/rucio that referenced this issue Mar 21, 2024
@rcarpa @voetberg
Transfer: fix transfer time and wait time metrics. Closes rucio#6482
42bd8ac 
Getting the started_at and transferred_at from the request
is useless, because this is the request _before_ its update
in the database. The information must be retrieved from the
transfertool message.

Also, record the transfer time without including the wait time.
voetberg pushed a commit to voetberg/rucio that referenced this issue Apr 15, 2024
@rcarpa @voetberg
Transfer: fix transfer time and wait time metrics. Closes rucio#6482
60eadd7 
Getting the started_at and transferred_at from the request
is useless, because this is the request _before_ its update
in the database. The information must be retrieved from the
transfertool message.

Also, record the transfer time without including the wait time.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rcarpa rcarpa

Labels
bug Transfers
Projects
None yet
Milestone
33.4.0
Development

No branches or pull requests
2 participants
@bari12 @rcarpa