-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
remove deprecated heavyIon scenario workflows #44665
Conversation
A new Pull Request was created by @mandrenguyen for CMSSW_14_0_X. It involves the following packages:
@subirsarkar, @AdrianoDee, @miquork, @sunilUIET, @srimanob, @cmsbuild can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
cms-bot internal usage |
please test |
-1 Failed Tests: RelVals RelVals-INPUT RelValsValueError: Undefined workflows: 24834.0, 24834.911, 4.53, 4.22, 5.1, 7.3, 8, 9.0, 135.4, 136.731, 136.7611, 136.8311, 136.88811, 136.793, 136.874, 139.001, 12434.0, 140.023, 140.043, 140.063, 141.044, 25, 1306.0, 14234.0, 158.01, 12834.0, 12834.7, 12846.0, 1330, 13234.0, 312.0, 24896.0, 140.53, 23234.0, 140.56, 2500.4, 24900.0, 141.042, 141.046, 25034.999, 14034.0, 250202.181, 101.0, 1000, 1001, 13034.0, 10224.0, 25202.0, 11634.0 RelVals-INPUT |
Pull request #44665 was updated. @AdrianoDee, @sunilUIET, @cmsbuild, @srimanob, @miquork, @subirsarkar can you please check and sign again. |
please test |
-1 Failed Tests: RelVals RelVals-INPUT RelValsValueError: Undefined workflows: 24834.0, 24834.911, 4.53, 4.22, 5.1, 7.3, 8, 9.0, 135.4, 136.731, 136.7611, 136.8311, 136.88811, 136.793, 136.874, 139.001, 12434.0, 140.023, 140.043, 140.063, 141.044, 25, 1306.0, 14234.0, 158.01, 12834.0, 12834.7, 12846.0, 1330, 13234.0, 312.0, 24896.0, 140.53, 23234.0, 140.56, 2500.4, 24900.0, 141.042, 141.046, 25034.999, 14034.0, 250202.181, 101.0, 1000, 1001, 13034.0, 10224.0, 25202.0, 11634.0 RelVals-INPUT |
Pull request #44665 was updated. @miquork, @AdrianoDee, @srimanob, @subirsarkar, @sunilUIET, @cmsbuild can you please check and sign again. |
please test |
Pull request #44665 was updated. @sunilUIET, @AdrianoDee, @srimanob, @miquork, @cmsbuild, @subirsarkar can you please check and sign again. |
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-54e12b/38714/summary.html Comparison SummarySummary:
|
@cms-sw/pdmv-l2 @cms-sw/upgrade-l2 |
+pdmv |
I was wondering if it would make sense to back port all #43824 ? Or is it too much? |
+1 |
merge |
PR description:
There are some knock-off effects from the removal of workflows based on the old
heavyIons
scenario, done in:#43824
The removal of these in the master is causing issues in CMSSW_14_0_X, as described here:
#44612 (comment)
This PR simply removes these workflows in 14_0_X as well.
PR validation:
compiles
If this PR is a backport please specify the original PR and why you need to backport that PR. If this PR will be backported please specify to which release cycle the backport is meant for:
This is a manual cherry-pick of
01c75db