Fix DQM warnings from FSQ client during cosmics workflows

[fruboes]

No description provided.

[cmsbuild]

A new Pull Request was created by @fruboes for CMSSW_7_4_X.

Fix DQM warnings from FSQ client during cosmics workflows

It involves the following packages:

Configuration/StandardSequences

@cmsbuild, @franzoni, @nclopezo, @davidlange6 can you please review it and eventually sign? Thanks.
@ghellwig, @appeltel, @makortel, @GiacomoSguazzoni, @rovere, @VinInn, @Martin-Grunewald, @cerati, @dgulhan this is something you requested to watch as well.
You can sign-off by replying to this message having '+1' in the first line of your reply.
You can reject by replying to this message having '-1' in the first line of your reply.
If you are a L2 or a release manager you can ask for tests by saying 'please test' in the first line of a comment.

[ktf]

@cmsbuild please test

[ghost]

@cmsbuild please test

[cmsbuild]

The tests are being triggered in jenkins.

[deguio]

thanks @fruboes
why don't you remove the fsq module from:
https://github.com/cms-sw/cmssw/blob/CMSSW_7_4_X/DQMOffline/Configuration/python/DQMOfflineCosmics_SecondStep_cff.py

instead from
Configuration/StandardSequences/python/HarvestingCosmics_cff.py
?

in this way we don't spread the sequence definition all around CMSSW.

sounds good?
thanks,
F.

[ktf]

@Degano what happened to the tests?

[ghost]

@ktf this PR is suffering from the missing refs bug. I restarted the tests pointing to the branch instead.

[fruboes]

Dear @deguio,

sorry for a late reply. I should be able to look at this again early next week

Cheers,
Tomasz

[deguio]

ciao @fruboes
74x is closing today. do you think you can converge on this in the next hours?

thanks,
F.

[cmsbuild]

+1
Tested at: f7b2f94
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-7321/2319/summary.html

[cmsbuild]

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

[ktf]

Any updates on this?

[fruboes]

should be ready soon (working on it), apologies for a very late followup

[fruboes]

Created a new PR: #7570

[ktf]

Please close this one.
On Thu 5 Feb 2015 at 10:09 fruboes notifications@github.com wrote:

Created a new PR: #7570 #7570

—
Reply to this email directly or view it on GitHub
#7321 (comment).