...

2021-05-25

Recording: link to zoom recording

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 ok (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 // we can compromise and extract this into separate item for E)
   13. https://gerrit.o-ran-sc.org/r/c/ric-plt/submgr/+/6081ok  (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) - seems not to be anymore in latest version of source code.

      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) - Thoralf sent e-mail to Anssi, Ron, Matti.

      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.
      1. AP-Sunil: check this with HCL team
   3. How to get reports more frequently and up to date?
      1. AP-Sunil: check this with HCL team
   4. check why some have zero code coverage
      1. AP-Sunil: check this with HCL team
   5. 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. Status check of D release items.
5. Subhash's mail
6. Sunil. RIC-723 further input needed from Thoralf
   1. Chaitanya (taking over from Alok): discuss in Wednesday's call.

7. Sunil: RIC-184 - [Adapt to latest K8S and helm (testing in OSC labs)] - completed
   Subtask RIC-746 (Testing nanobot on top of helm 3 and k8s v1.18 version): Closed
   Subtask RIC-747( Study how nanobot tests can be extended to Xapp) - Closed

   Subtask RIC-748(Study OTF & Xtesting as a prospective test framework candidate to draw from for ORAN Near RT RIC) : Attached the ppt to the ticket. Given presentation to Thoralf and Matti.
   As suggested we need some hands-on experience with OTF to confirm on this. OTF installation on local setup - Moved to Next release.
   Subtask RIC-779(Xapp Testing Enhancement like Subscription deletion need to be handled using nanobot):
   Newly created ticket for handling subscription deletion using nanobot automation. Next Release.
   Subtask RIC-775(Run nanobot based test automation from - it/test repository on RIC with k8 1.18/helm3 in OSC lab)
   Working on the vpn client connectivity (Softether). - In-progress
   Ticket RIC-791:
   Raised for updating the container tag and pushing in to staging nexus repo.

   RIC-723 - [Study performance overhead of latest opentracing solutions] - InProgress
   Able to generate sampling data after getting some help from Thoralf. Continue to work with other scenarios to compare with previous results.
   
   RIC-731- [Static code and test coverage for E2] - 90% Completed.
   Blocked on RIC-760 for test coverage.HCL working actively on RIC-731

   RIC-743 Create multiple e2sim for RIC benchmarking
   Thoralf created RIC-790 and moved the two remaining subtasks from RIC-150 to it. I closed RIC-150 as DONE

8. demos again? Yes. On Jun-8 same time slot as this meeting, but 2 hours. Organizational issues after the demo.
   
   1. From last meeting:
      1. Thoralf to discuss demos for Nokia features, e.g. REST subscriptions, O2 CLI (DMS CLI - Subhash wished for this as REST API is the next step)
      2. Possible influxDB demo to show how to use from xApps
      3. Possible demos in devops areas (e.g. RIC-754)
      4. RIC-129 as slideset demo as real data cannot be used in public demo
      5. RIC-773 if ready by demo
      6. demo for RIC-747/RIC-748 slidesets
9. Remove Scott as committer - AP Thoralf to start mail thread
10. Replace Rahul with Naman - AP Thoralf to start approval mail on this.
11. RICAPP agenda: RICAPP Meetings.

...