[GENIUS-118] Fix any slf4j API abuses, and enforce future non-regressions by enabling builds to fail via findbugs-slf4j Created: 07/Mar/18  Updated: 15/Mar/18  Resolved: 15/Mar/18

Status: Resolved
Project: genius
Component/s: General
Affects Version/s: None
Fix Version/s: Fluorine

Type: Bug Priority: Medium
Reporter: Faseela K Assignee: Saurav Malani
Resolution: Done Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified


 Description   

vorburger writing the initial version of this issue's description opened (thanks) by k.faseela to clarify scope:

This netvirt-dev list post is an example of bad (buggy, basically) use, or rather misuse (abuse) of the slf4j API; it's in netvirt, but I bet that in genius we have a few things like that as well...

The goal of this issue is (A) to find and fix ALL wrong use of slf4j logging everywhere in genius (by raising relatively small and specific individual Gerrits very similar to most of those you can now start to see being raised for netvirt; please just re-use the topic "findbugs-logging" for all such changes), AND (B) to make it so that it can never happen again by enforcing future non-regressions by enabling builds to fail via findbugs-slf4j; this can be done in the genius parent by by re-using how we've done it in project infrautils (NB you don't have to create yet another genius:findbugs artifact; just use infrautils').

In terms of how to go about this practically in a local dev environment, it's most probably easiest to first do (B) only locally just to easily find all the problems, then raise all patches for (A), and then when they are all merged push the Gerrit for (B) to finish it up.



 Comments   
Comment by Faseela K [ 09/Mar/18 ]

We have an intern who wants to get started on Genius, and I have assigned this task to him.

Welcome Malani

Comment by Michael Vorburger [ 14/Mar/18 ]

https://git.opendaylight.org/gerrit/#/c/69452/

Comment by Michael Vorburger [ 14/Mar/18 ]

https://git.opendaylight.org/gerrit/#/q/topic:findbugs-logging+project:genius

Comment by Michael Vorburger [ 15/Mar/18 ]

https://git.opendaylight.org/gerrit/#/c/69361/ concludes this, and now enforces findbugs-slf4j for genius (as already for netvirt, and infrautils). Thanks Malani and sathwiksai for your contributions!

Generated at Wed Feb 07 19:59:57 UTC 2024 using Jira 8.20.10#820010-sha1:ace47f9899e9ee25d7157d59aa17ab06aee30d3d.