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

MSTransferor: resolve RSE expression for pileup location #10041

Merged
merged 1 commit into from
Oct 30, 2020

Conversation

amaltaro
Copy link
Contributor

@amaltaro amaltaro commented Oct 29, 2020

Fixes #10038
Fixes #9969 (not really a fix though, more like a minor enhancement)

Status

ready

Description

  • Improved logging for pileup rules evaluated by MSTransferor;
  • STUCK rules with NO_SOURCES are never used as a valid location;
  • Resolve the pileup RSE expressions in MSTransferor by using the MSTransferor Rucio object, which has an in-memory cache for 12h.

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
  • Pylint check: failed
    • 5 warnings and errors that must be fixed
    • 8 warnings
    • 68 comments to review
  • Pycodestyle check: succeeded
    • 39 comments to review
  • Python3 compatibility checks: succeeded

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

@amaltaro amaltaro requested a review from todor-ivanov October 30, 2020 07:50
@amaltaro
Copy link
Contributor Author

Works fine too!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants