Date  

Meeting Logistics

DayTimeMeeting IDZoom Link
Monday[sdc] Team ONAP3, Mon UTC 10:00 / China 18:00 / Eastern 5:00 / Pacific 02:00see listsee ONAP-Meeting invite

Meeting Attendees


TBD

Agenda & Minutes

Review open action items from last week

Main purpose is to sync between Committers and discuss open reviews, other topics below

Open Reviews :


AgendaMinutes
Check on pending Reviews

Any urgent pending reviews ?

Discussed several review

 - Topic : SONAR bugs and coverage,

    SDC code is not pretty (to say the least) we should remind the community to improve it

    - do a reminder of the rules (enable checkstyle, sonar checks in IDE)

   - agree on common rules, maybe enforce them in verify job/add a sonar job in review to catch newly introduced issues quickly

    - assign resolution to originator of issue

   since Sept, SDC grow to 145 SONAR bugs

  AP : Chris to setup a weekly item on items - Done

Vote on Code style proposal

Improve coverage by gathering Integration Test coverage

Issue due to Change in Sonar/Jacoco - Seb may have a trick

  


Amir's review, assigned to Julien - Done

Seb raised the issue that Log4J 1.2 still identified by NExus IQ as a dependency in Catalog-BE and FE (Chris to look at it) - Fixed by Vasyl


Split verify job in 2

speed up results by splitting integration phase and unit test/build

create new keyword for retest all or only one

vote +1 if both are ok


mvn clean install -P skip-integration-tests

mvn clean install -DskipTests -DskipUICleanup=true -Djacoco.skip=true -DskipPMD -Dmaven.test.skip=true -Dcheckstyle.skip -P docker && mvn install -P start-sdc && mvn install -P run-integration-tests


Done, see new verify Jobs

Still missing : Keyword to re trigger only one, this seems not possible as reusing JJB templates from LF and Integration that are wrapped together


Action Items

  • Chris to Setup Weekly item on SONAR and remind community about best practice
  • All to follow up on reviews
  • Chris to start the jjb setup for dual job and raise IT ticket with LF


Recording


No recording, forgot to launch it, see minutes


Chat