Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Changing era to Run2022B #4706

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.

Sign up for GitHub

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

Jump to bottom
Merged
germanfgv merged 1 commit into from
Jul 5, 2022
Merged

Changing era to Run2022B #4706

germanfgv merged 1 commit into from
Jul 5, 2022

Conversation

@jhonatanamado
Copy link
Contributor

@jhonatanamado jhonatanamado commented Jul 4, 2022 •
edited by germanfgv
Loading

Set raw_to_disk=False for everything, but HLTPhysics*, EphemeralHLTPhysics* and ZeroBias*

@francescobrivio
Copy link
Contributor

francescobrivio commented Jul 4, 2022

Hi @jhonatanamado!
Maybe I misunderstood Dima's message, but in https://cms-talk.web.cern.ch/t/raw-on-disk-in-run2022b-and-later/12317 I read:

The only exception that we are aware of is the TSG PDs, i.e. HLTPhysics and ZeroBias PDs in all their different forms, which will be subscribed by default to CERN for 3 months.

Which means that RAW (and all other forms) for HLTPhysics and ZeroBias PDs should stay on Disk for 3 months, am I missing something?
Thanks a lot!

@drkovalskyi
Copy link

drkovalskyi commented Jul 4, 2022

Thanks Francesco. @germanfgv please review the pull request. There is no need to set raw_to_disk to False for each call of addDataset. We set raw_to_disk to False by default. So it should be enough to remove all unnecessary raw_to_disk=True flags.

@tvami
Copy link
Contributor

tvami commented Jul 4, 2022

Hi All,
I just wanted to report that tonight we'll still have 900 GeV collisions, so this era change should really happen tomorrow morning.

@drkovalskyi
Copy link

drkovalskyi commented Jul 4, 2022

The change of the configuration in the repo doesn't have direct effect on the production node. Beside we will need to switch the machine for new era. Basically it's a preparation for the era change.

germanfgv
germanfgv reviewed Jul 4, 2022
View reviewed changes
Copy link
Contributor

@germanfgv germanfgv left a comment

Choose a reason for hiding this comment

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

@jhonatanamado What we want from the PR is to set raw_to_disk=False for all PRs, with the exception of HLTPhysics and ZeroBias PDs.

Simply keep the default to false, and remove all the unneeded raw_to_disk=True and it should be fine

@francescobrivio
Copy link
Contributor

francescobrivio commented Jul 4, 2022

FYI @ttedeschi as this week's ORM :)

@jhonatanamado
Changing era to Run2022B
e6a75a3 
Set raw_to_disk=False for everything, with the exception of HLTPhysics*, EphemeralHLTPhysics*, ZeroBias*
@jhonatanamado
Copy link
Contributor Author

jhonatanamado commented Jul 4, 2022

Thanks guys, I misunderstand the message. @germanfgv German, could you check again?

@jhonatanamado jhonatanamado requested a review from germanfgv July 4, 2022 13:43
germanfgv
germanfgv approved these changes Jul 4, 2022
View reviewed changes
Copy link
Contributor

@germanfgv germanfgv left a comment

Choose a reason for hiding this comment

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

Everything looks fine now. We will prepare a machine with this configuration and make the change to it tomorrow morning, as discussed in https://cms-talk.web.cern.ch/t/era-change/12416

@germanfgv germanfgv merged commit bd7ba31 into dmwm:master Jul 5, 2022
@jhonatanamado jhonatanamado deleted the Run2022B branch July 5, 2022 20:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@germanfgv germanfgv germanfgv approved these changes

@drkovalskyi drkovalskyi Awaiting requested review from drkovalskyi

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@jhonatanamado @francescobrivio @drkovalskyi @tvami @germanfgv