Skip to content

Conversation

fstagni
Copy link
Contributor

@fstagni fstagni commented Jul 15, 2025

Only for FTS transfers (I do not think we want others...?).

BEGINRELEASENOTES

*DMS
NEW: add scitag to FTS transfers

ENDRELEASENOTES


PS: the scitag "API" is https://scitags.org/api.json. The "activityName" as coded should be match the "FTS3 activity", which is of course VO-specific. In LHCb it's coded as simple as:

    def inferFTSActivity(self, ftsOperation, rmsRequest, rmsOperation):
        """
        Tries to infer the FTS Activity
        """

        ### Data Challenge activity
        # All the tests with data challenges are done
        # on SE with '-DC-' in their name
        targetSEs = rmsOperation.targetSEList
        if any("-DC-" in se for se in targetSEs):
            return "Data Challenge"

        return super().inferFTSActivity(ftsOperation, rmsRequest, rmsOperation)

(pay attention that here it's spelled "Data Challenge" while in the scitag API is "DataChallenge".

@fstagni fstagni marked this pull request as ready for review September 4, 2025 11:58
@fstagni fstagni force-pushed the 90_scitags branch 6 times, most recently from 59137f9 to da61742 Compare September 5, 2025 08:28
@fstagni fstagni force-pushed the 90_scitags branch 2 times, most recently from bcf926a to e5624e5 Compare September 8, 2025 09:47
# Create a TTL cache: max 1 item, expires after 84400 seconds (1 day)
json_cache = TTLCache(maxsize=1, ttl=86400)

@cached(json_cache, lock=Lock())
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TODO for me: fix it with cachedmethod

@chaen chaen marked this pull request as draft October 14, 2025 07:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants