• Icon: Task Task
    • Resolution: Won't Do
    • Icon: Major Major
    • core
    • None

      JENKINS-36715 made a typical developer build of Jenkins (mvn -DskipTests clean install) much slower, adding about a minute of heavy CPU activity on my laptop. Static analysis is useless during developer builds: if there are any regressions, CI builds will catch them alongside automated tests.

      While you can disable the slow plugin execution with -Dfindbugs.skip, it would be nicer to have this be implied by the standard -DskipTests, which is an indication that you are just interested in producing artifacts as quickly as possible.

      For plugins we already do this, so it should be easy to copy that logic to core.

          [JENKINS-37440] Disable FindBugs analysis when skipping tests

          Oleg Nenashev added a comment -

          Agreed. But we should run FindBugs in the case of lightweight tests, so the logic will be a bit different

          Oleg Nenashev added a comment - Agreed. But we should run FindBugs in the case of lightweight tests, so the logic will be a bit different

          James Nord added a comment - - edited

          I'm against this. As a Maven user I expect that -DskipTests skips tests and that -Dfindbugs.skip skips findbugs.

          Please do not overload a specific flag for a plugin with another flag. If you want to use a profile that you can activate -Pquickbuild or activate via a property -DskipTestsAndFindbugs.

          James Nord added a comment - - edited I'm against this. As a Maven user I expect that -DskipTests skips tests and that -Dfindbugs.skip skips findbugs. Please do not overload a specific flag for a plugin with another flag. If you want to use a profile that you can activate -Pquickbuild or activate via a property -DskipTestsAndFindbugs.

          Oleg Nenashev added a comment -

          I would also prefer to have FB by default. This change just aligns the behavior with Parent POM.
          But we can change our mind about parent POM of course

          Oleg Nenashev added a comment - I would also prefer to have FB by default. This change just aligns the behavior with Parent POM. But we can change our mind about parent POM of course

          Against. @jglick could implement https://wiki.jenkins-ci.org/display/JENKINS/Designing+pre-tested+commit or create his own global maven profile.

          Kanstantsin Shautsou added a comment - Against. @jglick could implement https://wiki.jenkins-ci.org/display/JENKINS/Designing+pre-tested+commit or create his own global maven profile.

          Oleg Nenashev added a comment -

          I did my work on this grumbling from jglick. Since there is not so many fans of the proposed approach, somebody else can proceed. I am fine with the current state when FindBugs is being launched with "-DskipTests"

          Oleg Nenashev added a comment - I did my work on this grumbling from jglick . Since there is not so many fans of the proposed approach, somebody else can proceed. I am fine with the current state when FindBugs is being launched with "-DskipTests"

            Unassigned Unassigned
            jglick Jesse Glick
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated:
              Resolved: