-
Notifications
You must be signed in to change notification settings - Fork 1
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
[TEST] specify wrong checksum #4
Conversation
A new Pull Request was created by @kpedro88 (Kevin Pedro) for branch main. @fwyzard, @cmsbuild, @smuzaffar, @makortel, @aandvalenzuela, @iarspider can you please review it and eventually sign? Thanks. |
cms-bot internal usage |
let's wait to test until the next IB is available |
please test |
@kpedro88 , unit test for version check passed but looks like the cfg_versioncheck function is only checking the top level files. You need to recursively check all the sub-directories. Can you please add some print statements about which config files it has processed? |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-62a0db/37123/summary.html Comparison SummarySummary:
|
@smuzaffar it was actually an issue related to symbolic links. I will fix this (and add an option for verbose printouts as requested) in cms-sw/cmssw#43814 (which already includes other various bug fixes). We can leave this PR open to test the fixed script once it's merged. |
please test |
-1 Failed Tests: UnitTests Unit TestsI found 1 errors in the following unit tests: ---> test TestHeterogeneousCoreSonicTritonVersionCheck had ERRORS Comparison SummarySummary:
|
Test works! Output, for the record:
|
Thanks Kevin |
This is a test PR to ensure that the unit test added in cms-sw/cmssw#43696 triggers as intended on cms-data PRs. not to be merged