Skip to content
This repository was archived by the owner on Jul 17, 2026. It is now read-only.
This repository was archived by the owner on Jul 17, 2026. It is now read-only.

Retry download not working #45

Description

@gabrielefronze

Apparently there's an error within the code wrapping rucio download, which was previously commented to avoid issues.
If a download fails it should be retried by the pipeline:

if(output->freturn_code != MAX_ATTEMPTS){
fastlog(INFO,"Trying again did %s download in %s.", output->fdid.data(), output->full_cache_path().data());
// fInputQ->append(*output);

The idea is nice, but the issue is with the if condition. In fact testing:

output->freturn_code != MAX_ATTEMPTS

should be:

output->freturn_code != TOO_MANY_ATTEMPTS

or:

output->fattempt > MAX_ATTEMPTS

based on the definition here:

rucio_download_info* rucio_download_wrapper(rucio_download_info& info){
if(not server_exists(info.fserver_name)) {
fastlog(ERROR, "Server %s not found. Aborting!", info.fserver_name.data());
info.freturn_code = SERVER_NOT_FOUND;
return &info;
} else {
if (info.fattempt <= MAX_ATTEMPTS and info.freturn_code != SETTINGS_NOT_FOUND) {
info.fattempt++;
info.freturn_code = rucio_download_wrapper(info.fserver_name, info.fserver_config, info.scopename(),
info.filename());
info.fdownloaded = (info.freturn_code != FILE_NOT_FOUND and info.freturn_code != SETTINGS_NOT_FOUND);
} else {
info.freturn_code = TOO_MANY_ATTEMPTS;
info.fdownloaded = false;
}
}
return &info;
}

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Fields

    Size

    None yet

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions