Add additional paths to customizeHLTIter0ToMkFit

[Dr15Jones]

PR description:

The other paths also include hltIter0PFlowCkfTrackCandidates and need the change in order to be self-consistent.
The inconsistency was found by #34735

PR validation:

I ran workflow 11634.7 with this change and the changes from #34735 and it now runs fine.

[cmsbuild]

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-34793/24501

• This PR adds an extra 12KB to repository

[cmsbuild]

A new Pull Request was created by @Dr15Jones (Chris Jones) for master.

It involves the following packages:

• RecoTracker/MkFit (reconstruction)

@perrotta, @jpata, @cmsbuild, @slava77 can you please review it and eventually sign? Thanks.
@makortel, @felicepantaleo, @GiacomoSguazzoni, @JanFSchulte, @rovere, @VinInn, @ebrondol, @gpetruc, @mmusich, @mtosi, @dgulhan this is something you requested to watch as well.
@silviodonato, @dpiparo, @qliphy, @perrotta you are the release manager for this.

cms-bot commands are listed here

[Dr15Jones]

please test

[Dr15Jones]

code-checks

[slava77]

it looks like the bot is out for lunch or dinner

[slava77]

it looks like the bot is out for lunch or dinner

@smuzaffar
please take a look, to get the bot to react to code-checks

[smuzaffar]

@slava77 , most of code check jobs run on lxplus nodes and for last few days we have issues with some of lxplus nodes where singularity can not run [a]. Unfortunately code checks for this PR was run 4 times on buggy nodes.

Anyway, code checks are running now and results will be available soon

singularity -s exec /cvmfs/unpacked.cern.ch/registry.hub.docker.com/cmssw/cc7:amd64 sh -c 'export ORIGINAL_SINGULARITY_BIND=${SINGULARITY_BIND}; export SINGULARITY_BIND='\'''\'';  source /cvmfs/cms.cern.ch/cmsset_default.sh ; export PATH=$PATH:/usr/sbin;export PATH=$HOME/bin:$PATH; voms-proxy-init -voms cms -valid 24:00|| true ; cd /tmp/cmsbuild/jenkins/workspace/run-pr-code-checks; echo $PATH; /tmp/cmsbuild/jenkins/workspace/run-pr-code-checks/run.sh'
00:26:01 �[91mERROR  : Failed to create user namespace: user namespace disabled

[cmsbuild]

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-34793/24536

• This PR adds an extra 12KB to repository

[cmsbuild]

Pull request #34793 was updated. @perrotta, @jpata, @cmsbuild, @slava77 can you please check and sign again.

[Dr15Jones]

please test

[Dr15Jones]

@smuzaffar is the comparison test for this pr stuck in some endless loop? Seems to be doing the same thing over and over for the past 12 hours.

[smuzaffar]

@Dr15Jones not really, https://cmssdt.cern.ch/jenkins/job/compare-root-files-short-matrix/44643/ should finish in 10/20 mins. Normally it takes 2 hours when job starts.

[Dr15Jones]

@smuzaffar I checked this build before going to bed last night and the camparison had already been running for more than 2 hours. I then get up this morning and it still says it is running.

[smuzaffar]

Here is the sequence of jobs

• 12h ago started the pr build job https://cmssdt.cern.ch/jenkins/job/ib-run-pr-tests/17599/
• 12 h ago started the deployment of cvmfs https://cmssdt.cern.ch/jenkins/job/ib-run-pr-wait-deployment/11966/
• 5h ago started the RelVals job https://cmssdt.cern.ch/jenkins/job/ib-run-pr-relvals/8810/ (took 1h to finish)
• 2h ago started the Comparison job https://cmssdt.cern.ch/jenkins/job/compare-root-files-short-matrix/44643/

all look good in terms of jobs. As jenkins was very busy last night that is why there were delays beteween various jobs

[cmsbuild]

+1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-235cd6/17599/summary.html
COMMIT: a297db5
CMSSW: CMSSW_12_1_X_2021-08-05-1100/slc7_amd64_gcc900
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-sw/cmssw/34793/17599/install.sh to create a dev area with all the needed externals and cmssw changes.

Comparison Summary

Summary:

• No significant changes to the logs found
• Reco comparison results: 4 differences found in the comparisons
• DQMHistoTests: Total files compared: 39
• DQMHistoTests: Total histograms compared: 2999410
• DQMHistoTests: Total failures: 7
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 2999381
• DQMHistoTests: Total skipped: 22
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 0.0 KiB( 38 files compared)
• Checked 165 log files, 37 edm output root files, 39 DQM output files
• TriggerResults: no differences found

[slava77]

+reconstruction

for #34793 a297db5

• code changes are in line with the PR description and the follow up review
• jenkins tests pass; the specific fail case targeted by this PR was tested as detailed in the PR description

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @silviodonato, @dpiparo, @qliphy, @perrotta (and backports should be raised in the release meeting by the corresponding L2)

[qliphy]

+1