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

Add Phase-2 with HLT:75e33 wf #37175

Closed
wants to merge 2 commits into from

Conversation

srimanob
Copy link
Contributor

@srimanob srimanob commented Mar 8, 2022

PR description:

To add Phase-2 wf with HLT 75e33 menu. This is still a draft, to be used with #37158 and #37162.

PR validation:

runTheMatrix.py --what upgrade -l 35034.111 -t 8 --wm init
give expected workflow.

if this PR is a backport please specify the original PR and why you need to backport that PR:

If merge in 12_4_X, then backport is needed for 12_3_X.

@cmsbuild
Copy link
Contributor

cmsbuild commented Mar 8, 2022

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-37175/28741

@cmsbuild
Copy link
Contributor

cmsbuild commented Mar 8, 2022

A new Pull Request was created by @srimanob (Phat Srimanobhas) for master.

It involves the following packages:

  • Configuration/PyReleaseValidation (pdmv, upgrade)

@jordan-martins, @bbilin, @wajidalikhan, @cmsbuild, @AdrianoDee, @srimanob, @kskovpen can you please review it and eventually sign? Thanks.
@makortel, @kpedro88, @Martin-Grunewald, @missirol, @fabiocos, @slomeo this is something you requested to watch as well.
@perrotta, @dpiparo, @qliphy you are the release manager for this.

cms-bot commands are listed here

@srimanob
Copy link
Contributor Author

srimanob commented Mar 8, 2022

No need to trigger the test at the moment. I see an error when running DigiTrigger step, mentioned in
#37162 (comment)

@srimanob srimanob marked this pull request as ready for review March 8, 2022 14:49
@srimanob
Copy link
Contributor Author

srimanob commented Mar 8, 2022

test parameters:

@trtomei
Copy link
Contributor

trtomei commented Mar 22, 2022

Cross-posting from #37162 : how does this need to be modified to become the workflow discussed there?

@Martin-Grunewald
Copy link
Contributor

I understand this PR currently defines part of the required workflow (only), but not the complete one, nor does it add it to the PR, IB and relval tests. @srimanob could you please help us to achieve this?

@srimanob
Copy link
Contributor Author

Hi @Martin-Grunewald @trtomei
I will update the PR to address the additional step of HLT.

@srimanob srimanob force-pushed the 123_AddPhase2HLT75e33wf branch from aab5cde to e6812d9 Compare March 23, 2022 15:23
@srimanob
Copy link
Contributor Author

@cmsbuild code-checks

@srimanob
Copy link
Contributor Author

Reopen the PR in 124 branch, #37324

@srimanob srimanob closed this Mar 23, 2022
@cmsbuild
Copy link
Contributor

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-37175/28967

  • This PR adds an extra 60KB to repository

@cmsbuild
Copy link
Contributor

Pull request #37175 was updated. @jordan-martins, @bbilin, @wajidalikhan, @cmsbuild, @AdrianoDee, @srimanob, @kskovpen can you please check and sign again.

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.

5 participants