Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.

...

  1. 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).
    1. 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)
    2. https://gerrit.o-ran-sc.org/r/c/ric-plt/submgr/+/6132 ok
    3. https://gerrit.o-ran-sc.org/r/c/ric-plt/vespamgr/+/5987 ok
    4. https://gerrit.o-ran-sc.org/r/c/ric-plt/xapp-frame/+/6014 ok
    5. 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.)
    6. https://gerrit.o-ran-sc.org/r/c/ric-plt/sdlpy/+/6060 ok
    7. https://gerrit.o-ran-sc.org/r/c/ric-plt/sdlgo/+/6061 ok
    8. https://gerrit.o-ran-sc.org/r/c/ric-plt/sdlgo/+/6062 ok
    9. https://gerrit.o-ran-sc.org/r/c/ric-plt/submgr/+/6082 ok
    10. https://gerrit.o-ran-sc.org/r/c/ric-plt/submgr/+/6109 ok
    11. https://gerrit.o-ran-sc.org/r/c/ric-plt/rtmgr/+/6112 ok
    12. https://gerrit.o-ran-sc.org/r/c/ric-plt/libe2ap/+/6131(still in review, Thoralf commented that unit test cases to be added)
    13. 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)
  2. every second time: Review code coverage stats and blocker code smells
    1. We take a look at the two links in the beginning of this page: Code coverage reports
      1. ric-plt/a1 85,3 // no bugs (2021-01-26)

      2. ric-plt/alarm-go 0,0 // no bugs (2021-01-31)

      3. ric-plt/appmgr 0,0 // 1 major bug (2021-01-31)

      4. ric-plt/asn1-documents not applicable

      5. ric-plt/dbaas 0,0 // 2 blocker bugs in UT code (2021-01-27)

      6. ric-plt/e2 0,0 // no bugs (2021-04-23)

      7. ric-plt/e2mgr 88,4 // no bugs (2021-01-30)

      8. ric-plt/lib/rmr 86,1 // no bugs (2021-05-25)

      9. ric-plt/rtmgr 77,8 // no bugs (2021-01-31)

      10. ric-plt/sdl  66,6 // 1 minor vulnerability bug (2021-01-29)

      11. ric-plt/sdlgo 45,5 // no bugs (2021-02-01)

      12. ric-plt/streaming-protobufs 0,0 // no bugs (2021-01-31)

      13. ric-plt/submgr 33,4 // no bugs (2021-01-31)

      14. ric-plt/tracelibcpp 0,0 // 1 major bug (2021-05-18)

      15. ric-plt/tracelibgo 94,7 // no bugs (2021-01-31)

      16. ric-plt/vespamgr 79,0 // no bugs (2020-12-07)

      17. ric-plt/xapp-frame 0,0 // no bugs (2021-01-31)

      18. ric-plt/xapp-frame-cpp 95,7 // 2 blocker bugs (2021-05-19)

      19. ric-plt/xapp-frame-py 96,3 // 0 blocker bugs (2021-01-26)

    2. 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.
    3. How to get reports more frequently?
    4. ric-plt/e2 is already being handled via RIC-731 (expected to complete in Dawn)
  3. 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
    1. Note I added RC-5 in Release criteria checklist template  "RC-5: Check that "blocker" code smells in Sonartype have been addressed"
  4. Sunil. RIC-723 further input needed from Thoralf
  5. demos again?
    1. From last meeting:
      1. Thoralf to discuss demos for Nokia features.
      2. Possible influxDB demo to show how to use from xApps
      3. Possible demos in devops areas (e.g. RIC-754)
      4. demo for RIC-747/RIC-748 slidesets
  6. RICAPP agenda: RICAPP Meetings.

...