Software Shifter Summary week 3 and 4
Summary:
Unfortunately the previous 2 weeks suffered a few different network problems which played havoc with the Jenkins nightly and ci-test build systems. These should all now be resolved. Some additional fixes/improvements to ci-test are being progressed to address issues that came up in running tests over the last week or so.
Status of MR on Mon 3rd Feb:
149 MR targeting master 81 WIP, 10 ready for merging, several MR in need of new references, ~15 in nightly testing with most builds/tests working as expected.
Comments to next shifter:
The maintainer App appears to have a few gotchas but is useful: http://apearce.web.cern.ch/apearce/maintainer-app/
I've tried to setup a service to track MRs via a flowchart with some automatic gitlab integration: http://rta-mon.cern.ch/board/1 This automatically tracks the status of MRs wrt open/close WIP/non-WIP and tracks different projects by different colours.
MR summary:
Ready to Merge
link(s) | status |
---|---|
Rec!1858 (merged) !345 (merged) Brunel!943 (merged) | Should be ready for merging |
Lbcom!412 (merged) | Should be good to merge |
Lbcom!411 (merged) | Should be good to merge |
Lbcom!402 (merged) | Should be good to merge |
LHCb!2298 (merged) LHCb!2297 (merged) | Should be good to merge as soon as LHCb!2297 (merged) is approved |
LHCb!2293 (merged) | Should be good to merge |
LHCb!2299 (merged) | Should be good to merge |
Once a MR is good to be merged, does it make sense to keep it tagged in a nightly slot just in-case something gets committed which breaks an MR between it getting signed off and passing a ci-test?
MRs in nightly testing
link(s) | tag | status |
---|---|---|
Gaudi!942 | all-slots | doc changes |
Gaudi!1063 | lhcb-gaudi-head | changes for ROOT-6.18 |
Phys!633 (merged) | lhcb-gaudi-head | Looks good, can be merged? |
Rec!1903 (merged) | lhcb-test-throughput | throughput drop appears understood |
LHCb!2270 (merged) (WIP) LHCb!2338 (merged) LHCb!2339 (merged) Rec!1892 (merged) Rec!2893 (merged) Rec!1895 (merged) LHCb!2343 (merged) LHCb!2341 (merged) LHCb!2342 (merged) Phys!650 (merged) Rec!1896 (merged) | Lots of inter-dependent MRs (LHCb#73 (closed)) some WIP, some bugs need identifying |
MRs needing review from coding expert
link(s) | status |
---|---|
!354 (merged) | looks good for merging, needs approval |
Rec!1879 (merged) | Needs testing/approving, was waiting on gaudi release |
Rec!1893 (merged) | Review pending, there has been recent development |
Rec!1904 (merged) Brunel!947 (merged) | Needs comment and possibly testing to make sure no tests break |
Lbcom!416 (merged) | Needs review but passes tests. OK to merge? |
Lbcom!415 (merged) | Needs review but passes tests. OK to merge? |
Lbcom!413 (merged) | Needs review but passes tests. OK to merge? |
Lbcom!399 (merged) LHCb!2290 (merged) | Many failing Boole tests does this need work or new refs? |
LHCb!2343 (merged) | needs to be approved but close to being ready |
MRs with new References
MR link(s) | new Reference | status |
---|---|---|
Rec!1858 (merged) !345 (merged) | Brunel!943 (merged) | Should be ready for merging |
LHCb!2303 (merged) | Rec!1869 (merged) | needs testing with new references |
!381 (closed) | needs sensitivities updating | |
Rec!1903 (merged) | !383 (merged) | looks correct to merge if new ref fixes tests |
MRs which need testing
link(s) | status |
---|---|
Rec!1882 (merged) | needs new ref from new test |
Rec!1861 (merged) | needs new test to get new refs |
Lbcom!410 (merged) | needs new Boole Ref (probably new ci-test needs to be run) |
Lbcom!409 (merged) | needs ref updating, logs lost |
Lbcom!408 (merged) | needs ref updating, logs lost |
Lbcom!401 (merged) Boole!255 (merged) | Tests in Boole are still failing |
Rec!1894 (merged) Phys!658 (merged) | needs testing with ci-test |
LHCb!2297 (merged) | needs testing with ci-test |
!377 (merged) LHCb!2270 (merged) | needs testing with ci-test |
LHCb!2354 (merged) Rec!1902 (merged) | testing ongoing |
LHCb!2295 (merged) | Appears to need Brunel!933 (closed) |
LHCb!2290 (merged) | Not clear if needs update for failing Boole tests |
Active MRs in development
link(s) | status |
---|---|
Rec!1728 (merged) | appears in development |
Rec!1772 (closed) | suspected problems with thoughput |
Rec!1775 (closed) | lots of open discussions |
Rec!1831 (closed) | lots of open discussions |
Rec!1870 (merged) | tests revealed problems, being reviewed |
Rec!1872 (merged) | under review |
Rec!1902 (merged) | ready for testing |
!361 (merged) | work to fix bugs |
LHCb!2309 (merged) | close to being ready for testing |
LHCb!2289 (merged) | discussions ongoing |
LHCb!2257 (closed) | many open discussions |
Brunel!947 (merged) Rec!1904 (merged) | Added new test which is failing, this should be checked |
New MRs
link(s) | status |
---|---|
Phys!658 (merged) | |
Mooore!383 | new Ref for Rec!1903 (merged) |