Day

Time

Meeting ID

Zoom Link

Monday

[sdc] Team ONAP3, Mon UTC 10:00 / China 18:00 / Eastern 5:00 / Pacific 02:00

see list

see ONAP-Meeting invite

Agenda

Minutes

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