Date  

Meeting Logistics

DayTimeMeeting IDZoom LinkJoin by phone
Monday[sdc] Team ONAP3, Mon UTC 11:00 / China 19:00 / Eastern 06:00 / Pacific 03:00https://lists.onap.org/g/onap-meetings/calendarSee Groups.io calendar invite

See Groups.io Calendar Invite

Note that a Waiting room has been added to the meeting room, the host will let you join once identified

Meeting Attendees

TBD - see recording

Agenda & Minutes

Review open action items from last week

AgendaMinutes (Draft to be updated during call)
Slides
Helm package verification

In Guilin, we decided to not do helm package validation/verification, several ideas were discussed at that time. Some improvements are planned for H release.

Presented by Nokia team to discuss how to add Helm validation to SDC onboarding

agreed to :

  • add a new deployed container running the validation service, that can
    • run multiple version of helm and test that the package is deployabe
    • stretch do linting
    • stretch provide feedback to user in SDC UI
  • target H release to add the basic functionality, add configuration data to existing AUTO.JSON loaded in SDC config maps (onboarding container reads this at startup)

Not enough time to complete, code is dark until next release,


New repo is being created under SDC project


Verify Job split

During the Committer's meeting, we've decided to split SDC verify Job in 2 steps :

1) build + Unit tests (No Docker)

2) build + NO unit tests + Docker + Run integration phase


Change is implemented, note that the jobs run in // but require both to be successful for a merge, the plan is to be able to provide faster feedback

We've reused Integration team new templates for CSIT (job is called Review-Verification Job)

It's enabled only on Master for Now


Note : the Unit test job is failing due to Catalog BE Unit tests, work in progress (any idea welcome)

 - tests are now passing but still a few errors happen from time to time

    npm error when downloading packages

   onap-configuration-core test module is not thread safe (file manipulation)

  a few reviews in progress to try and improve


  • integration tests do fail from time to time in one test case, it seems the UI is hanging when coming back to home screen (then test timeout)


pls recheck if you see one of the failures above, any improvement to fix is welcome

had to rework this a bit last week as CSIT template got changed, now using default java template - will try to improve further in next release

2 tickets open for improvements with the LF :


Call for Committers

Requesting active committers to apply for formal Committer's role

Fill the template under : Committer Promotion Request for [SDC]

Will review and request a vote from all current committers


Note the new repo for Helm has been contributed by Nokia team, and they are committers on that repo (sub repo)


JIRA Updates

  • High priority defects
  • Open Reviews
  • Defects & tasks backlog
  • Others


High priority defects for Honolulu:

 

T Key Summary Assignee Reporter P Status Resolution Created Updated Due
Loading...
Refresh

  • Other High priority ticket to discuss ?

Open Reviews :


New features review / Bug fixes for H release

closing tickets once released : SDC 1.8.2 is released, and distribution client 1.4.2


oom reviews:

Gating of SDC 1.8.3 was ok with Snapshots build, waiting result :

https://gerrit.onap.org/r/c/oom/+/117348


PLAN to move MASTER to I release (must be done for RC0 which is Thursday in the plan)


Gerrit REVIEW can proceed with auto test in verify job, now accepting changes for Honolulu

  • Reminder of merge conditions : Verify must pass AND tests must be provided for new features (SONAR metrics are important but can only be checked AFTER code is merged)

Important : for new features, please provide explanation of changes and API/UI/unit testing, it is also advised to present changes at SDC weekly

we have a plan to add SONAR test to the verify job, will work on this in the coming months


Defect & Tasks Backlog (no update this week):

 

T Key Summary Assignee Reporter P Status Resolution Created Updated Due
Loading...
Refresh

 

  • Use the AID (Architecture integration documentSDC Data model also for new comers

DEFECT AND BUGFIX on H Release :

first commit on Master then cherrypick on H release


Others:


3 BP have been approved for H release :

  • up-to-date new dependencies (not transitive) - can contact seccom for guidance
  • Helm V3 (mostly for OOM)
  • CII badging improvements (use good crypto)

2 GR

  • no more use of python2 (deprecated) -#GR
  • java 11 -# GR


Important note to committers and code contributors : please make sure these are followed or your contribution will not be merged




Automated Rolling Upgrade and Data Migration

No Update this release (parking topic for later)


Support of automated upgrade from version X to version X+1


Needed for every change that breaks existing data validity : Have a strategy to handle data migration

  • Create a wiki page, details for every feature and its impact on data for migration / schema changes -  check the wiki page: Upgrade plan
    • Feature X: All services metadata must have new property Y
  • Implementing a "task" for migrating data accordingly.
    • Example: asdctool mig2002

SDC-3155 - Getting issue details... STATUS Epic created

→ Discuss further and create User stories along with Bell Canada, Amdocs, Tata communications

Will need to prioritize VS Guilin Requirements →

this may shift to next release due to low commitment

Learn from SDNC experience;

"Normatives" types: challenging


Important Jira tasks we need to pick up


Long Term tasks

INCREASE SONAR Coverage and lower SONAR bugs

We've noticed a growth of SONAR issues and coverage is getting lower,

https://sonarcloud.io/dashboard?id=onap_sdc

Stats are computed only on Merged code, it would be good for contributors to :

1) Enable Sonar plugins in IDE try to fix issues before they get in

2) To reduce issues, we could try and run through module per module (divide and conquer approach)

3) other ideas ?

4) Coverage call for help to improve and go above 55%  - currently at 54.2%, exception requested to complete this by RC0:

  • Reduce line of codes (unused code)
  • Increase testing
  • Use LomBok to cover getter setter
  • You can see where coverage per module in sonar views


Not much response, going with 150 line length will propose change to the broader ONAP community

   - done on on common, some commits in progress

Other long term tasks

  • SDC-2708 Add assertions for unit tests  - #NewComers
  • SDC-2812Refactor all usage of Eithers - Guilin R7, #NewComers
  • SDC-2844 JUnit 5 unit test  - on going for new UT
  • SDC-3108 - Getting issue details... STATUS Migrate all TestNG tests to Junit5 , #NewComers



Security  - Guilin

Looking at these and this will significantly impact SDC APIs, since SDC does not have password it has to defer request to portal/aaf

exception for this in Guilin requested , though PORTAL provided Credentials validation API so that SDC can check users - will see if we have time to do something about these  

Only one left open OJSI 94, presented to SECCOM and they agreed on current progress

T Key Summary Assignee Reporter P Status Resolution Created Updated Due
Loading...
Refresh


Reviewing last week's action items

see below


Q&AIf you have time, pay attention to SDC mailing list questions
Istambul Requirements

New requirements for I release can be proposed, check with ArchSubcomittee and Usecase/req comittee



M3 is passend, RC0 planned for Thursday ?


Need some input for Marketting launch https://wiki.onap.org/x/FNULBQ


Release planning : SDC R8 Release Planning

Please plan to close your Jira tickets, need to clean up by RC2

Documentation work pending (Release Note and/or doc updates)


1 POC and 1 REQ without stories


Discuss SDC APIs and Swagger generation

Call happened last Wed,

We reviewed the current SDC APIs pictured in Architecture documentation and even if we believe the split makes sense, it seems that there are some mismatch with the current implementation and/or API locations, also Doc is incomplete

Decided to move forward to :

1) regroup APIs per Backend

2) enable Swagger doc generation (current one is a static copy that happened at one point ? not sure when)

3) annotations are there in the code so let's reuse them

Anybody welcome to help


Deletion of Old catalog entriesEricsson would be interesting in progressing this item, Chris to check if Ofir has some old discussion around this


Action Items ()

  • Chris to report SONAR changes to PTL call : Done, LF is looking to see if we can have a better SONAR Quality profile management (currently relying on SONAR auto profile which are managed by SonarCloud and updated frequently)
    • No update on this topic from LF
  • Chris to send invite for Swagger / Api discussion on Wed → done call happened
  • Chris to investigate discussions about deletion of old catalog entries


Slack Channel : #SDC on LFN ONAP.org Slack instance


SDC Dev lab on Windriver :

ssh ubuntu@10.12.6.10 (with onap key)

10.12.6.10 - LAb is installed with latest release (SDC 1.8.3) on helm 3 and K8s 1.19


Recording


zoom_0.mp4


Chat