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

HLT-compatible LHCInfoPer* PopCon #45395

Draft
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

JanChyczynski
Copy link
Contributor

PR description:

Update of the duringFill mode of LHCInfoPer* O2Os to be able to write to HLT and express tags complying to the synchronization rules. It writes only one (or none) payload at execution and writes to an IOV corresponding to a lumisection not yet processed by HLT. It's achieved by using OnlineDBOutputService::writeIOVForNextLumisection instead of PoolDBOutputService::writeMany. The O2Os in duringFill now uses OnlinePopCon and OnlinePopConAnalyzer (implemented in #44927) instead of the original ones.

TODO:

  • Get access to data needed by OnlineDBOutputService (oms-eventing.cms, run number and frontier key from cDAQ) outside of P5
  • Finish setting up OnlineDBOutputService in python configurations of the O2Os
  • Add unit test mode to python configurations of the O2Os to allow for testing without access to oms-eventing.cms etc.

Depends on #44927 (has to be tested on top of it and merged after it)

PR validation:

Work in progress. Some tests were run but cannot be fully tested before resolving issues from the TODO list above

Backport

Not yet determined

@cmsbuild
Copy link
Contributor

cmsbuild commented Jul 7, 2024

cms-bot internal usage

@cmsbuild
Copy link
Contributor

cmsbuild commented Jul 7, 2024

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-45395/40845

@cmsbuild
Copy link
Contributor

cmsbuild commented Jul 7, 2024

A new Pull Request was created by @JanChyczynski for master.

It involves the following packages:

  • CondTools/RunInfo (db)

@cmsbuild, @consuegs, @francescobrivio, @perrotta, @saumyaphor4252 can you please review it and eventually sign? Thanks.
@JanChyczynski, @PonIlya, @mmusich, @rsreds, @yuanchao this is something you requested to watch as well.
@antoniovilela, @rappoccio, @sextonkennedy you are the release manager for this.

cms-bot commands are listed here

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

Successfully merging this pull request may close these issues.

None yet

2 participants