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/com/asn1c/+/6266 OK
    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/submgr/+/6134 OK
    4. https://gerrit.o-ran-sc.org/r/c/ric-plt/sdl/+/6128 OK
    5. https://gerrit.o-ran-sc.org/r/c/ric-plt/xapp-frame/+/6014 OK
    6. https://gerrit.o-ran-sc.org/r/c/ric-plt/xapp-frame/+/6157 OK (existing test case was updated with new attributes)
    7. https://gerrit.o-ran-sc.org/r/c/ric-plt/sdlpy/+/6060 OK
    8. https://gerrit.o-ran-sc.org/r/c/ric-plt/submgr/+/6167 OK
    9. https://gerrit.o-ran-sc.org/r/c/ric-plt/e2mgr/+/6183 OK
    10. https://gerrit.o-ran-sc.org/r/c/ric-plt/libe2ap/+/6131 OK (as per meeting minutes from 2021-05-25)
    11. https://gerrit.o-ran-sc.org/r/c/ric-plt/ric-dep/+/6199 OK (all code copied from other place into 3rdparty subdir)
    12. https://gerrit.o-ran-sc.org/r/c/ric-plt/alarm-go/+/6237 OK (removal)
    13. https://gerrit.o-ran-sc.org/r/c/ric-plt/xapp-frame/+/6241 OK (test case was adapted to change in data type)
    14. https://gerrit.o-ran-sc.org/r/c/ric-plt/submgr/+/6242 OK (only documentation)
    15. https://gerrit.o-ran-sc.org/r/c/ric-plt/sdlpy/+/6261 OK
    16. https://gerrit.o-ran-sc.org/r/c/ric-plt/submgr/+/6211 OK
    17. https://gerrit.o-ran-sc.org/r/c/ric-plt/submgr/+/6265 OK
    18. https://gerrit.o-ran-sc.org/r/c/ric-plt/ric-dep/+/6301 OK as Influx helm chart changes.
    19. https://gerrit.o-ran-sc.org/r/c/ric-plt/alarm-go/+/6309 OK
    20. https://gerrit.o-ran-sc.org/r/c/ric-plt/submgr/+/6313 OK
    21. https://gerrit.o-ran-sc.org/r/c/ric-plt/submgr/+/6322 OK
    22. https://gerrit.o-ran-sc.org/r/c/ric-plt/submgr/+/6081 OK
    23. https://gerrit.o-ran-sc.org/r/c/ric-plt/xapp-frame-py/+/6345 OK
    24. https://gerrit.o-ran-sc.org/r/c/ric-plt/xapp-frame-py/+/4880 OK (not yet merged)
  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)
      1. ric-plt/alarm-go 0,0 // no bugs (2021-01-31)

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

      3. ric-plt/asn1-documents not applicable

      4. 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.

      5. ric-plt/e2 0,0 //

    1. no
      1. 2 blocker bugs (2021-

    2. 04
      1. 06-

    3. 23)
      1. 18) - Thoralf raised RIC-802 and send e-mail to Dhiraj, Subhash and Naman.

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

      3. ric-plt/lib/rmr 86,1 // no bugs (2021-

    4. 05
      1. 06-

    5. 25
      1. 22)

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

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

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

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

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

      7. ric-plt/tracelibcpp 0,0 // 1 major bug (2021-

    6. 05
      1. 06-

    7. 18
      1. 22)

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

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

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

      5. ric-plt/xapp-frame-cpp 95,

    8. 7
      1. 6 // 2 blocker bugs (2021-

    9. 05
      1. 06-

    10. 19
      1. 23) - Thoralf sent e-mail reminder to Anssi, Ron, Matti.

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

      3. com/golog 80,7 // no bugs (2021-01-31)
      4. com/log 73,5 // 1 blocker bug and 8 minor ones (2021-01-27) - Thoralf sent e-mail to Abdulwahid
      5. com/pylog 33,7 // 11 major and minor bugs (2021-01-26)
  3. Subhash: pls. merge: https://gerrit.o-ran-sc.org/r/c/Subhash: pls. merge: https://gerrit.o-ran-sc.org/r/c/ric-plt/a1/+/6365 and re-release
  4. check D-release status: Release criteria checklist template, including e-mail reminder. All components to be handled by Nokia, except A1 → Subhash and RMR → Scott
  5. Subhash shared the information about this public anybody-can-attend conference. It also includes a session about O-RAN SC (Subhash and Thoralf speaking): https://opensource.samsung.com/community/blogList
  6. RICAPP agenda: RICAPP Meetings.

...

  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 // 2 blocker bugs (2021-06-18) - Thoralf raised RIC-802 and send e-mail to Dhiraj, Subhash and Naman.

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

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

      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-06-22)

      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,6 // 2 blocker bugs (2021-06-23) - Thoralf sent e-mail reminder to Anssi, Ron, Matti.

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

      20. com/golog 80,7 // no bugs (2021-01-31
      21. com/log 73,5 // 1

        blocker

        bug and 8 minor ones (2021-01-27) - Thoralf sent e-mail to Abdulwahidcom/pylog 33,7 // 11 major and minor

        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 - we went through the D release status of various items Samsung was planning to work on.
  6. Sunil. RIC-723 further input needed from Thoralf
    1. Chaitanya (taking over from Alok): discuss in Wednesday's call.
  7. Sunil:

    1. 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.

    2. 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.

    3. RIC-731- [Static code and test coverage for E2] - 90% Completed. // Blocked on RIC-760 for test coverage.HCL working actively on RIC-731

    4. 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.

...