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.

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

Don't resubmit intermediate transfers #4809

Closed
rcarpa opened this issue Aug 27, 2021 · 0 comments
Closed

Don't resubmit intermediate transfers #4809

rcarpa opened this issue Aug 27, 2021 · 0 comments
Assignees
Milestone

Comments

@rcarpa
Copy link
Contributor

rcarpa commented Aug 27, 2021

Motivation

When a multi-hop transfer fails, each of its hops is marked as failed. The issue is that the finisher will then re-schedule each of the hops separately. This can result in some undesired behaviors:

  • deleting the rule which triggered the initial multihop transfer will not stop rucio from attempting the intermediate hops. Things will solve by themselves after 3 failed attempts for the first hop (other hops have the same issue, but they usually will fail fast without actually submitting anything). This can be expensive when this are 3 attempts to stage-in from a tape.
  • re-scheduling the initial multihop will fail until those 3 attempts are actually executed because of
    logger(logging.WARNING, 'Multihop : A request already exists for the transfer between %s and %s. Will cancel all the parent requests', transfer_path[0].src, transfer_path[-1].dst)

Modification

Fail fast the intermediate transfers. It's probably enough to

@rcarpa rcarpa changed the title Transfers: don't resubmit intermediate replicas Transfers: don't resubmit intermediate transfers Aug 27, 2021
rcarpa added a commit to rcarpa/rucio that referenced this issue Aug 31, 2021
rcarpa added a commit to rcarpa/rucio that referenced this issue Aug 31, 2021
rcarpa added a commit to rcarpa/rucio that referenced this issue Sep 14, 2021
rcarpa added a commit to rcarpa/rucio that referenced this issue Sep 17, 2021
rcarpa added a commit to rcarpa/rucio that referenced this issue Sep 24, 2021
rcarpa added a commit to rcarpa/rucio that referenced this issue Sep 30, 2021
rcarpa added a commit to rcarpa/rucio that referenced this issue Oct 1, 2021
rcarpa added a commit to rcarpa/rucio that referenced this issue Oct 1, 2021
rcarpa added a commit to rcarpa/rucio that referenced this issue Oct 6, 2021
rcarpa added a commit to rcarpa/rucio that referenced this issue Oct 6, 2021
@bari12 bari12 closed this as completed in 3e27e96 Oct 18, 2021
bari12 added a commit that referenced this issue Oct 18, 2021
…diate

Transfers: don't resubmit intermediate transfers. Closes #4809
@bari12 bari12 changed the title Transfers: don't resubmit intermediate transfers Don't resubmit intermediate transfers Oct 18, 2021
@bari12 bari12 added this to the 1.26.6 milestone Oct 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants