search

fermitools / declad

BSD 3-Clause "New" or "Revised" License
1 stars 2 forks source link

sam not fully blocked #30

Closed rlcee closed 1 month ago

rlcee commented 1 month ago

mover.py;320

        do_declare_to_sam = self.Config.get("declare_to_sam", True)
        if sclient is not None:

need to add do_declare_to_sam to the if

rlcee commented 1 month ago

I think the same problem exists other places in mover, for rucio, at least. I'd argue that if sam or rucio is turned off, nothing related to them should be collected or executed. Thanks

marcmengel commented 1 month ago

This will require a more thorough review... In particular since we support either metadata format as input; we still need conversion calls, even if we are not doing one or the other backend...