Remove now-unused AthRegSeq
As discussed on the atlas-sw-core mailing list: https://groups.cern.ch/group/atlas-sw-core/Lists/Archive/Flat.aspx?RootFolder=%2Fgroup%2Fatlas%2Dsw%2Dcore%2FLists%2FArchive%2FAthRegSeq%20used%20anywhere&FolderCTID=0x012002002A6E23175D832F4DAD957CEB680A4D57 AthReqSeq is not used for anything. This MR will remove it. From grepping the repo I see a lot of unit test references: let's see what the CI returns.
Merge request reports
Activity
added full-unit-tests label
This merge request affects 2 packages:
- Control/AthenaCommon
- Control/AthenaConfiguration
Adding @ssnyder as watcher
added Core master review-pending-level-1 labels
CI Result FAILURE (hash c705343f)Athena AthSimulation AthGeneration AnalysisBase externals cmake make required tests optional tests Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 18158]added 55 commits
-
c705343f...026b2e6a - 53 commits from branch
atlas:master
- 1fb307fd - Merge remote-tracking branch 'upstream/master' into remove_AthRegSeq
- 1afa9357 - Update references
-
c705343f...026b2e6a - 53 commits from branch
This merge request affects 3 packages:
- Control/AthenaCommon
- Control/AthenaConfiguration
- Control/AthenaServices
Adding @ssnyder as watcher
CI Result FAILURE (hash 1afa9357)Athena AthSimulation AthGeneration AnalysisBase externals cmake make required tests optional tests Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 18165]added review-user-action-required label and removed review-pending-level-1 label
added 151 commits
-
1afa9357...e7514668 - 149 commits from branch
atlas:master
- 6a1b7e67 - Merge remote-tracking branch 'upstream/master' into remove_AthRegSeq
- c6dd3ace - Update reference
-
1afa9357...e7514668 - 149 commits from branch
Hi Rongkun,
Core changes like this are likely to disrupt unit tests, which is why I added the full-unit-tests label. Once we get down to the level of failures in the nightly (7 failures last night): http://atlas-computing.web.cern.ch/atlas-computing/links/distDirectory/gitwww/GITWebArea/nightlies/master/2020-08-02T2137/Athena/x86_64-centos7-gcc8-opt/Athena//UT/index.html
it should be good to go.
Cheers,
Stewart
This merge request affects 4 packages:
- Control/AthenaCommon
- Control/AthenaConfiguration
- Control/AthenaServices
- Control/DataModelTest/DataModelRunTests
Adding @ssnyder as watcher
added review-pending-level-1 label and removed review-user-action-required label
CI Result FAILURE (hash c6dd3ace)Athena AthSimulation AthGeneration AnalysisBase externals cmake make required tests optional tests Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 18217]added review-user-action-required label and removed review-pending-level-1 label
added 97 commits
-
c6dd3ace...c1b84f44 - 95 commits from branch
atlas:master
- e7a2b18f - Merge remote-tracking branch 'upstream/master' into remove_AthRegSeq
- 5ba2ce81 - Remove mention of AthRegSeq in logfiles
-
c6dd3ace...c1b84f44 - 95 commits from branch
added Calorimeter JetEtmiss review-pending-level-1 labels and removed review-user-action-required label