Change L1 menu for HLT tests with phase1 2017 geometry

[silviodonato]

This PR is needed to run the HLT GRun menu with the new phase1 2017 geometry.
It overwrites the L1 menu in the global tag to use the correct one.

[cmsbuild]

A new Pull Request was created by @silviodonato (Silvio Donato) for CMSSW_8_1_X.

It involves the following packages:

Configuration/HLT

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

cms-bot commands are listed here #13028

[silviodonato]

@cmsbuild please test

[cmsbuild]

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

[cmsbuild]

+1
Tested at: 55d499a
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-16332/15931/summary.html

The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
1c463fd
1661fff
You can see more details here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-16332/15931/git-log-recent-commits
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-16332/15931/git-merge-result

[cmsbuild]

Comparison job queued.

[cmsbuild]

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

[Martin-Grunewald]

@silviodonato @mmusich
autoCondHLT.py contains our "own" TSG symbolic GTs. We should not change existing official GTs, those should be changed by the AlCa team only. Marco, can you have a look at updating the L1T menu in the 2017 GTs?

[mmusich]

hold

[cmsbuild]

Pull request has been put on hold by @mmusich
They need to issue an unhold command to remove the hold state or L1 can unhold it for all

[cmsbuild]

Pull request #16332 was updated. @perrotta, @cmsbuild, @silviodonato, @Martin-Grunewald, @fwyzard, @davidlange6 can you please check and sign again.

[silviodonato]

I renamed the symbolic GT in order to avoid to overwrite the existing GT.
@cmsbuild please test

[cmsbuild]

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

[Martin-Grunewald]

@mmusich
In principle yes, but at one point we'll leave 2016 behind, esp. when we move to the real 2017 menu
in 2017.

[mmusich]

@Martin-Grunewald done in #16339

[mmusich]

@silviodonato please rename this PR to reflect the actual meaning of it - I'll remove the hold

[silviodonato]

@mmusich , thanks. I've changed the title

[mmusich]

unhold

[cmsbuild]

+1
Tested at: 2b04b1b
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-16332/15937/summary.html

The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
1c463fd
1661fff
You can see more details here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-16332/15937/git-log-recent-commits
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-16332/15937/git-merge-result

[cmsbuild]

Comparison job queued.

[cmsbuild]

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

[silviodonato]

+1

[cmsbuild]

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

[davidlange6]

i take it that #16339 supersedes this PR?

[mmusich]

@davidlange6 IMHO, TSG people use case i.e. to create their own customized autoCond keys is a legitimate one, provided they never EVER end up being used in production

[davidlange6]

+1

@mmusich - thanks - indeed the PR was not doing what I thought it was before.