Skip to content
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 TestDQMOfflineConfigurationGotAll failing due to extra sequences #45486

Closed
aandvalenzuela opened this issue Jul 17, 2024 · 6 comments
Closed

Comments

@aandvalenzuela
Copy link
Contributor

Hello,

Since CMSSW_14_1_X_2024-07-16-2300, unit test TestDQMOfflineConfigurationGotAll module (DQMOffline/Configuration) is failing with:

+ echo 'Sequences run by final DQMOfflineConfiguration: 9'
Sequences run by final DQMOfflineConfiguration: 9
+ '[' 9 -gt 0 ']'
+ echo 'Final DQMOfflineConfiguration should not run any sequences.'
Final DQMOfflineConfiguration should not run any sequences.
+ echo 'Please update parameters for TestDQMOfflineConfiguration unittest to run the extra sequences.'
Please update parameters for TestDQMOfflineConfiguration unittest to run the extra sequences.
+ exit 1

---> test TestDQMOfflineConfigurationGotAll had ERRORS

I do not see any recent merges that could have caused the appearance of these extra sequences, but I am tagging @mmusich since we had a similar issue three weeks ago (fixed in #45287). Should we just increase the number of sequences as well?

Thanks,
Andrea

@cmsbuild
Copy link
Contributor

cmsbuild commented Jul 17, 2024

cms-bot internal usage

@cmsbuild
Copy link
Contributor

A new Issue was created by @aandvalenzuela.

@Dr15Jones, @antoniovilela, @makortel, @mandrenguyen, @rappoccio, @sextonkennedy, @smuzaffar can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here

@mmusich
Copy link
Contributor

mmusich commented Jul 17, 2024

Should we just increase the number of sequences as well?

That will silence the failure. It's up to @cms-sw/dqm-l2 to clarify if the increase in the number of sequences is justified.

@makortel
Copy link
Contributor

assign dqm

@cmsbuild
Copy link
Contributor

New categories assigned: dqm

@rvenditti,@syuvivida,@tjavaid,@nothingface0,@antoniovagnerini you have been requested to review this Pull request/Issue and eventually sign? Thanks

@iarspider
Copy link
Contributor

@cmsbuild please close
Fixed by #45507

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants