-
Notifications
You must be signed in to change notification settings - Fork 4.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Unit test failure in DQM/Integration #31592
Comments
assign dqm |
New categories assigned: dqm @jfernan2,@andrius-k,@fioriNTU,@kmaeshima,@ErnestaP you have been requested to review this Pull request/Issue and eventually sign? Thanks |
A new Issue was created by @silviodonato Silvio Donato. @Dr15Jones, @dpiparo, @silviodonato, @smuzaffar, @makortel, @qliphy can you please review it and eventually sign/assign? Thanks. cms-bot commands are listed here |
No idea how this issue relates to my cleaning PR, but I can't deny it looks like there is a connection because the issue was not present before my PR got merged. Hence, I jumped in to investigate and found a fix. |
Sorry, actually the connection is obvious. The relevant package that was mistakenly a library instead of a plugin was previously included in a different plugin: https://github.com/cms-sw/cmssw/pull/31548/files#diff-d35b5dbd3e30c40bee4cc0c5bd6ae022L13. That's how the plugin ended up in the register before, I suppose. |
+1 |
fixed by #31593 |
This issue is fully signed and ready to be closed. |
We are getting
in the test
TestDQMOnlineClient-scal_dqm_sourceclient
ofDQM/Integration
.I suspect it is related to #31548 (@guitargeek)
The text was updated successfully, but these errors were encountered: