...
- Every time: Check rule RC-4 (link) that all L or XL commits of the last two weeks have also new unit tests. This is our policy. link-to-reviews (gerrit filter instructions // gerrit t-shirt-sizing).
- https://gerrit.o-ran-sc.org/r/c/ric-plt/rtmgr/+/5990 (Abdulwahid checked this. It's only M size as the commit incorrectly replace many lines with spaces to tabs. Also review 6105 actually fixes some test cases in this area: This should be sufficient as the underlying functionality wasn't changed)
- https://gerrit.o-ran-sc.org/r/c/ric-plt/submgr/+/6132 ok
- https://gerrit.o-ran-sc.org/r/c/ric-plt/vespamgr/+/5987 ok
- https://gerrit.o-ran-sc.org/r/c/ric-plt/xapp-frame/+/6014 ok
- https://gerrit.o-ran-sc.org/r/c/ric-plt/ric-dep/+/5826 (ok, even if no test cases as this is yaml files (Helm chart) repackaged.)
- https://gerrit.o-ran-sc.org/r/c/ric-plt/sdlpy/+/6060 ok
- https://gerrit.o-ran-sc.org/r/c/ric-plt/sdlgo/+/6061 ok
- https://gerrit.o-ran-sc.org/r/c/ric-plt/sdlgo/+/6062 ok
- https://gerrit.o-ran-sc.org/r/c/ric-plt/submgr/+/6082 ok
- https://gerrit.o-ran-sc.org/r/c/ric-plt/submgr/+/6109 ok
- https://gerrit.o-ran-sc.org/r/c/ric-plt/rtmgr/+/6112 ok
- https://gerrit.o-ran-sc.org/r/c/ric-plt/libe2ap/+/6131(still in review, Thoralf commented that unit test cases to be added)
- https://gerrit.o-ran-sc.org/r/c/ric-plt/submgr/+/6081 (replaced by other review, anyway contains test cases, but will likely be abandoned anyway)
- every second time: Review code coverage stats and blocker code smells
- We take a look at the two links in the beginning of this page: Code coverage reports
ric-plt/a1 85,3 // no bugs (2021-01-26)
ric-plt/alarm-go 0,0 // no bugs (2021-01-31)
ric-plt/appmgr 0,0 // 1 major bug (2021-01-31)
ric-plt/asn1-documents not applicable
ric-plt/dbaas 0,0 // 2 blocker bugs in UT code (2021-01-27)
ric-plt/e2 0,0 // no bugs (2021-04-23)
ric-plt/e2mgr 88,4 // no bugs (2021-01-30)
ric-plt/lib/rmr 86,1 // no bugs (2021-05-25)
ric-plt/rtmgr 77,8 // no bugs (2021-01-31)
ric-plt/sdl 66,6 // 1 minor vulnerability bug (2021-01-29)
ric-plt/sdlgo 45,5 // no bugs (2021-02-01)
ric-plt/streaming-protobufs 0,0 // no bugs (2021-01-31)
ric-plt/submgr 33,4 // no bugs (2021-01-31)
ric-plt/tracelibcpp 0,0 // 1 major bug (2021-05-18)
ric-plt/tracelibgo 94,7 // no bugs (2021-01-31)
ric-plt/vespamgr 79,0 // no bugs (2020-12-07)
ric-plt/xapp-frame 0,0 // no bugs (2021-01-31)
ric-plt/xapp-frame-cpp 95,7 // 2 blocker bugs (2021-05-19)
ric-plt/xapp-frame-py 96,3 // 0 blocker bugs (2021-01-26)
- As of 2021-05-25 the following components are missing in the report: ric-plt/alarm-cpp, ric-plt/libe2ap, ric-plt/nodeb-rnib, ric-plt/o1, ric-plt/ric-dep, ric-plt/ric-test, ric-plt/sdlpy, ric-plt/utils. Also ric-plt/jaegeradapter is missing, but we probably abandon that component. So no need to spend time on it.
- How to get reports more frequently?
- ric-plt/e2 is already being handled via RIC-731 (expected to complete in Dawn)
- We take a look at the two links in the beginning of this page: Code coverage reports
- Thoralf updated the CII evaluation criteria. We are now marked as "passed": Core Infrastructure Initiative (CII) Badging and https://bestpractices.coreinfrastructure.org/en/projects/4605
- Note I added RC-5 in Release criteria checklist template "RC-5: Check that "blocker" code smells in Sonartype have been addressed"
- Sunil. RIC-723 further input needed from Thoralf
- demos again?
- From last meeting:
- Thoralf to discuss demos for Nokia features.
- Possible influxDB demo to show how to use from xApps
- Possible demos in devops areas (e.g. RIC-754)
- demo for RIC-747/RIC-748 slidesets
- From last meeting:
- RICAPP agenda: RICAPP Meetings.
...