cms-sw / cms-bot

A few scripts to automate approval / testing process
28 stars 245 forks source link

Only report each RelVal failure once; fix formatting #2382

Closed iarspider closed 4 days ago

iarspider commented 4 days ago

Example of broken formatting:

image

cmsbuild commented 4 days ago

A new Pull Request was created by @iarspider for branch master.

@aandvalenzuela, @cmsbuild, @iarspider, @smuzaffar can you please review it and eventually sign? Thanks. @antoniovilela, @mandrenguyen, @rappoccio, @sextonkennedy you are the release manager for this. cms-bot commands are listed here

cmsbuild commented 4 days ago

cms-bot internal usage

iarspider commented 4 days ago

@smuzaffar this reports first error found (i.e. first line containing ERROR executing), or DAS_ERROR. Let me know if that's not the right approach.

cmsbuild commented 4 days ago

Pull request #2382 was updated.

smuzaffar commented 4 days ago

please test

looks good though

cmsbuild commented 4 days ago

+1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-de0dce/43062/summary.html COMMIT: d99f08c17fca606ec054dbfef4fc65629a7f9462 CMSSW: CMSSW_15_0_X_2024-11-25-1100/el8_amd64_gcc12 User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week1/cms-sw/cms-bot/2382/43062/install.sh to create a dev area with all the needed externals and cmssw changes.

Comparison Summary

Summary:

smuzaffar commented 4 days ago

please test workflow 11834.0,11834.13

lets test some workflow which are failing in IBs

cmsbuild commented 4 days ago

-1

Failed Tests: RelVals Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-de0dce/43069/summary.html COMMIT: d99f08c17fca606ec054dbfef4fc65629a7f9462 CMSSW: CMSSW_15_0_X_2024-11-25-1100/el8_amd64_gcc12 User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week1/cms-sw/cms-bot/2382/43069/install.sh to create a dev area with all the needed externals and cmssw changes.

RelVals

smuzaffar commented 4 days ago

+externals

cmsbuild commented 4 days ago

This pull request is fully signed and it will be integrated in one of the next master IBs (but tests are reportedly failing). This pull request will now be reviewed by the release team before it's merged. @mandrenguyen, @sextonkennedy, @antoniovilela, @rappoccio (and backports should be raised in the release meeting by the corresponding L2)