making HLTConfigProvider correctly handle ignored modules

[Sam-Harper]

HLTConfigProvider sees module names which are ignored as "-modname" rather than "modname". This means when it tries to retrieve the PSet (in which the name is "modname"), it fails and doesnt recognise that it has saveTags=True. The code is now fixed to remove the front "-" if it exists when trying to retrieve the module parameters.

A second fix is for paths which has an ignored HLTLevel1GtSeed module. Currently it things there are two L1Seed modules and as such can not calculate the pre-scale. However there is really only one active one so before pushing back to the list of active L1GTSeeds, it checks if the module is ignore by looking for the "-" in front of the module name.

[cmsbuild]

A new Pull Request was created by @Sam-Harper for CMSSW_8_0_X.

It involves the following packages:

HLTrigger/HLTcore

@Martin-Grunewald, @perrotta, @cmsbuild, @davidlange6, @fwyzard can you please review it and eventually sign? Thanks.
@Martin-Grunewald this is something you requested to watch as well.
@slava77, @Degano, @smuzaffar you are the release manager for this.

cms-bot commands are list here #13028

[Martin-Grunewald]

please test

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/11066/console

[cmsbuild]

+1
Tested at: 2e1b560
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-13215/11066/summary.html

[cmsbuild]

Comparison is ready
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-13215/11066/summary.html

[Martin-Grunewald]

+1
Thanks again, Sam!

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_8_0_X IBs (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @slava77, @davidlange6, @Degano, @smuzaffar

[davidlange6]

+1