Fix for Tracker Maps and Shifter Summary

[fioriNTU]

Fix in SiStripUtility to take into account the new IsolatedBunches directory in SiStrip workspace
Taking the opportunity to update the tk map script and add a template cfg to be used by Tracker Offline Shifters

[cmsbuild]

A new Pull Request was created by @fioriNTU for CMSSW_8_0_X.

It involves the following packages:

DQM/SiStripMonitorClient

@cmsbuild, @dmitrijus, @vanbesien, @davidlange6 can you please review it and eventually sign? Thanks.
@idebruyn, @threus, @hdelanno, @venturia this is something you requested to watch as well.
@slava77, @smuzaffar you are the release manager for this.

cms-bot commands are list here #13028

[fioriNTU]

If possible this should go in the first patch realese for production, it fixes the Top Summary for shifters that at the moment in Offline DQM shows a fake status of 100% green

[cmsbuild]

Pull request #15515 was updated. @cmsbuild, @dmitrijus, @vanbesien, @davidlange6 can you please check and sign again.

[davidlange6]

@fioriNTU - needs to be in 81x first - is there a PR?

[boudoul]

yes it was already referenced in the conversation : I can remind it here : PR #15517

[fioriNTU]

yes, here #15517 sorry for not linking

Il 19/08/2016 10.48, David Lange ha scritto:

@fioriNTU https://github.com/fioriNTU - needs to be in 81x first -
is there a PR?

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#15515 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKYUWoOQ0yLU90ePkFg8d-htmPVKtAUmks5qhW3RgaJpZM4JnjRL.

[davidlange6]

thanks - I just missed it

On Aug 19, 2016, at 11:23 AM, fioriNTU notifications@github.com wrote:

yes, here #15517 sorry for not linking

Il 19/08/2016 10.48, David Lange ha scritto:

@fioriNTU https://github.com/fioriNTU - needs to be in 81x first -
is there a PR?

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#15515 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKYUWoOQ0yLU90ePkFg8d-htmPVKtAUmks5qhW3RgaJpZM4JnjRL.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

[venturia]

when we (Tracker) change the DQM code we should check the impact up to the shifter Tracker map production. I don't think this can be included in any (semi-)automatic validation procedure

[dmitrijus]

+1

[cmsbuild]

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

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_8_0_X IBs after it passes the integration tests. This pull request requires discussion in the ORP meeting before it's merged. @slava77, @davidlange6, @smuzaffar

[cmsbuild]

+1
Tested at: 920f0a6
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-15515/14679/summary.html

[cmsbuild]

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

[davidlange6]

+1