[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 ] |
| 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! |