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

Support caching of RSE expressions in the Rucio wrapper #9892

Merged
merged 2 commits into from
Sep 4, 2020

Conversation

amaltaro
Copy link
Contributor

@amaltaro amaltaro commented Sep 3, 2020

Fixes #9891

Status

not-tested

Description

New client API to evaluate an RSE expression and parse the output. The output gets cached in memory for a configurable amount of seconds.
In addition to that, applied camelCase to the pickRSE implementation provided here: #9884

Is it backward compatible (if not, which system it affects?)

yes

Related PRs

none

External dependencies / deployment changes

none

@cmsdmwmbot
Copy link

Jenkins results:

  • Unit tests: succeeded
    • 1 tests added
  • Pylint check: failed
    • 2 warnings and errors that must be fixed
    • 12 warnings
    • 7 comments to review
  • Pycodestyle check: succeeded
    • 1 comments to review
  • Python3 compatibility checks: succeeded
    • there are suggested fixes for newer python3 idioms

Details at https://cmssdt.cern.ch/dmwm-jenkins/view/All/job/DMWM-WMCore-PR-test/10388/artifact/artifacts/PullRequestReport.html

make cache expiration configurable

added reset cache method; return a shallow copy of the value
@cmsdmwmbot
Copy link

Jenkins results:

  • Unit tests: succeeded
    • 1 tests added
  • Pylint check: failed
    • 2 warnings and errors that must be fixed
    • 12 warnings
    • 7 comments to review
  • Pycodestyle check: succeeded
    • 1 comments to review
  • Python3 compatibility checks: succeeded
    • there are suggested fixes for newer python3 idioms

Details at https://cmssdt.cern.ch/dmwm-jenkins/view/All/job/DMWM-WMCore-PR-test/10389/artifact/artifacts/PullRequestReport.html

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.

Cache results from the Rucio list_rses() method
2 participants