Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-15037

Option to consider only stable builds when calculating new warnings

    XMLWordPrintable

Details

    Description

      I'd like an option to only consider stable builds when computing the number of allowed warnings. Justification follows.

      Consider this scenario, where a job is configured to become unstable if the number checkstyle warnings increase by 5 or more compared to the previous build:

      1. The last Jenkins build is stable
      2. A developer commits a failing test and 10 fixed checkstyle issues. The failing test makes the build unstable.
      3. The developer reverts the commit, thus removing the failing test but increasing the number of checkstyle warnings back to the number of warnings in 1, and above the new limit computed in 2. The build is now unstable because of the checkstyle warnings.

      Is there a way to prevent a case like step 3 from failing the build? If not, I'd like the option described above. If you won't have the time to implement this in a near future, I think I could look into it.

      Attachments

        Activity

          davidparsson David Pärsson created issue -
          davidparsson David Pärsson made changes -
          Field Original Value New Value
          Description I'd like an option to only consider stable builds when computing the number of allowed warnings. Justification follows.

          Consider this scenario, where a job is configured to become unstable if the number checkstyle warnings increase by 5 or more compared to the previous build:

          1. The last Jenkins build is stable
          2. A developer commits a failing test and 10 fixed checkstyle issues. The failing test makes the build unstable.
          3. The developer reverts the commit, thus removing the failing test but increasing the number of checkstyle warnings back to the number of warnings in 1., and above the new limit computed in 2. The build is now unstable because of the checkstyle warnings.

          Is there a way to prevent a case like step 3 from failing the build? If not, I'd like the option described above. If you won't have the time to implement this in a near future, I think I could look into it.
          I'd like an option to only consider stable builds when computing the number of allowed warnings. Justification follows.

          Consider this scenario, where a job is configured to become unstable if the number checkstyle warnings increase by 5 or more compared to the previous build:

          1. The last Jenkins build is stable
          2. A developer commits a failing test and 10 fixed checkstyle issues. The failing test makes the build unstable.
          3. The developer reverts the commit, thus removing the failing test but increasing the number of checkstyle warnings back to the number of warnings in 1, and above the new limit computed in 2. The build is now unstable because of the checkstyle warnings.

          Is there a way to prevent a case like step 3 from failing the build? If not, I'd like the option described above. If you won't have the time to implement this in a near future, I think I could look into it.
          drulli Ulli Hafner added a comment -

          I think this is not possible yet.

          There are two ways to implement that feature: either do not consider unstable builds when computing the reference build number (which is used for comparison). Or don't compute a checkstyle result when a build is unstable (currently there already is an option to run the analysis for "UNSTABLE or SUCCESS", or "UNSTABLE or SUCCESS or FAILURE"). I.e. the checkbox needs to be replaced by a radio button.

          If you are interested in contributing please provide a pull request. It would be nice if the change could be made as part of the analysis-core plug-in thus that all other analysis plug-ins will inherit the new behavior.

          drulli Ulli Hafner added a comment - I think this is not possible yet. There are two ways to implement that feature: either do not consider unstable builds when computing the reference build number (which is used for comparison). Or don't compute a checkstyle result when a build is unstable (currently there already is an option to run the analysis for "UNSTABLE or SUCCESS", or "UNSTABLE or SUCCESS or FAILURE"). I.e. the checkbox needs to be replaced by a radio button. If you are interested in contributing please provide a pull request. It would be nice if the change could be made as part of the analysis-core plug-in thus that all other analysis plug-ins will inherit the new behavior.
          drulli Ulli Hafner made changes -
          Component/s analysis-core [ 15709 ]

          Thanks for your suggestions. I'll look into it and get back to you with a pull request. I'll probably go with the second suggestion (with radio buttons).

          davidparsson David Pärsson added a comment - Thanks for your suggestions. I'll look into it and get back to you with a pull request. I'll probably go with the second suggestion (with radio buttons).
          drulli Ulli Hafner added a comment -

          After thinking again about this issue I'm not sure if option 2 still is a good solution. If you set the radio button to SUCCESS then checkstyle will not be started, e.g., after FindBugs has set the build status to UNSTABLE. (Or even worse it depends on the order of the analysis plugins). This makes not much sense.

          What do you think?

          drulli Ulli Hafner added a comment - After thinking again about this issue I'm not sure if option 2 still is a good solution. If you set the radio button to SUCCESS then checkstyle will not be started, e.g., after FindBugs has set the build status to UNSTABLE. (Or even worse it depends on the order of the analysis plugins). This makes not much sense. What do you think?
          davidparsson David Pärsson added a comment - - edited

          You have a good point! The first option might make more sense. I however came up with a case where this would cause a weird behavior. Consider this:

          1. The system is configured to allow one new Checkstyle warning per commit (delta)
          2. A build is green
          3. The build becomes UNSTABLE (reason doesn't matter)
          4. Given that only stable builds are reference builds, the Checkstyle would mark each build as unstable if the sum of new Checkstyle warnings added since the last stable is greater than one.

          Maybe a more complicated rule is desirable, like this one: The only case where the reference build should NOT be changed is when the build is unstable (for other reasons) and the number of warnings has decreased. In other words, don't allow the reference warning count to decrease when the build is unstable. Comments?

          davidparsson David Pärsson added a comment - - edited You have a good point! The first option might make more sense. I however came up with a case where this would cause a weird behavior. Consider this: 1. The system is configured to allow one new Checkstyle warning per commit (delta) 2. A build is green 3. The build becomes UNSTABLE (reason doesn't matter) 4. Given that only stable builds are reference builds, the Checkstyle would mark each build as unstable if the sum of new Checkstyle warnings added since the last stable is greater than one. Maybe a more complicated rule is desirable, like this one: The only case where the reference build should NOT be changed is when the build is unstable (for other reasons) and the number of warnings has decreased. In other words, don't allow the reference warning count to decrease when the build is unstable. Comments?

          This could include adding a checkbox in the section under "Compute new warnings (based on reference build)", which says something like "Only lower reference warning count if build is stable".

          davidparsson David Pärsson added a comment - This could include adding a checkbox in the section under "Compute new warnings (based on reference build)", which says something like "Only lower reference warning count if build is stable".

          Could you possibly point me in a direction where to start? Which class persists these thresholds? I've tried but I can't seem to understand where it's done.

          davidparsson David Pärsson added a comment - Could you possibly point me in a direction where to start? Which class persists these thresholds? I've tried but I can't seem to understand where it's done.
          drulli Ulli Hafner added a comment - - edited

          That makes sense for me.

          You need to change:

          In analysis-core:

          • thresholds.jelly: add a new checkbox after the delta computation
          • in HealthAwarePublisher and HealthAwareReporter:
            • Add a new constructor with new boolean property
            • Add a new boolean field (current behavior must be false to be backward compatible) (automatic persistence)
            • In method perform(): the new flag in the build history needs to be set
          • in BuildHistory:
            • add a boolean setter to toggle the reference build computation
            • add reference build computation based on your algorithm

          In checkstyle:

          • Change pom dependency to analysis-core to SNAPSHOT so that your changes from above will be seen
          • Add a new boolean property in CheckStylePublisher and CheckStyleReporter constructors
          • Call new super constructor
          drulli Ulli Hafner added a comment - - edited That makes sense for me. You need to change: In analysis-core: thresholds.jelly: add a new checkbox after the delta computation in HealthAwarePublisher and HealthAwareReporter: Add a new constructor with new boolean property Add a new boolean field (current behavior must be false to be backward compatible) (automatic persistence) In method perform(): the new flag in the build history needs to be set in BuildHistory: add a boolean setter to toggle the reference build computation add reference build computation based on your algorithm In checkstyle: Change pom dependency to analysis-core to SNAPSHOT so that your changes from above will be seen Add a new boolean property in CheckStylePublisher and CheckStyleReporter constructors Call new super constructor

          Thanks, great instructions!

          davidparsson David Pärsson added a comment - Thanks, great instructions!
          davidparsson David Pärsson added a comment - - edited

          Hm.. What should happen when a project has different warning priorities and one of them increases and another decreases? Ideally when this new option would be enabled, I wouldn't want to decrease any reference warning count for any priority. Using a single build as reference build would however not allow this, and this might complicate things even further.

          I think it's more clear to just use the stable build as a reference, and ignore the fact that deltas might get messed up as stated in my comment above.

          davidparsson David Pärsson added a comment - - edited Hm.. What should happen when a project has different warning priorities and one of them increases and another decreases? Ideally when this new option would be enabled, I wouldn't want to decrease any reference warning count for any priority. Using a single build as reference build would however not allow this, and this might complicate things even further. I think it's more clear to just use the stable build as a reference, and ignore the fact that deltas might get messed up as stated in my comment above .
          drulli Ulli Hafner added a comment -

          I think if the checkbox is set then an unstable build should be completely ignored.

          drulli Ulli Hafner added a comment - I think if the checkbox is set then an unstable build should be completely ignored.

          It seems the CheckStyleResult computes the result of the build, compared to a reference build, before I'm able to set the boolean on the history in HealthAwarePublisher.perform(). I'm currently supplying the boolean flag to the BuildHistory constructor (via CheckStylePublisher and CheckStyleResult).

          I've created pull requests with my changes.

          davidparsson David Pärsson added a comment - It seems the CheckStyleResult computes the result of the build, compared to a reference build, before I'm able to set the boolean on the history in HealthAwarePublisher.perform(). I'm currently supplying the boolean flag to the BuildHistory constructor (via CheckStylePublisher and CheckStyleResult). I've created pull requests with my changes.
          drulli Ulli Hafner added a comment -

          I see. I forgot that the reference build is used not only during evaluation of the build result but also when computing the new and fixed warnings. I.e., your approach with the new constructor is the only possible solution.

          drulli Ulli Hafner added a comment - I see. I forgot that the reference build is used not only during evaluation of the build result but also when computing the new and fixed warnings. I.e., your approach with the new constructor is the only possible solution.

          Code changed in jenkins
          User: David Pärsson
          Path:
          src/main/java/hudson/plugins/analysis/core/BuildHistory.java
          src/main/java/hudson/plugins/analysis/core/HealthAwarePublisher.java
          src/main/java/hudson/plugins/analysis/core/HealthAwareReporter.java
          src/main/resources/util/thresholds.jelly
          src/main/resources/util/thresholds.properties
          src/test/java/hudson/plugins/analysis/core/BuildHistoryTest.java
          http://jenkins-ci.org/commit/analysis-core-plugin/f73958097bad7344fb12c75d8aa2928572a51c60
          Log:
          JENKINS-15037 Optionally consider only stable builds as reference builds

          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: David Pärsson Path: src/main/java/hudson/plugins/analysis/core/BuildHistory.java src/main/java/hudson/plugins/analysis/core/HealthAwarePublisher.java src/main/java/hudson/plugins/analysis/core/HealthAwareReporter.java src/main/resources/util/thresholds.jelly src/main/resources/util/thresholds.properties src/test/java/hudson/plugins/analysis/core/BuildHistoryTest.java http://jenkins-ci.org/commit/analysis-core-plugin/f73958097bad7344fb12c75d8aa2928572a51c60 Log: JENKINS-15037 Optionally consider only stable builds as reference builds

          Code changed in jenkins
          User: Ulli Hafner
          Path:
          src/main/java/hudson/plugins/analysis/core/BuildHistory.java
          src/main/java/hudson/plugins/analysis/core/HealthAwarePublisher.java
          src/main/java/hudson/plugins/analysis/core/HealthAwareReporter.java
          src/main/resources/util/thresholds.jelly
          src/main/resources/util/thresholds.properties
          src/test/java/hudson/plugins/analysis/core/BuildHistoryTest.java
          http://jenkins-ci.org/commit/analysis-core-plugin/b4bf0441cd3ec069687ea06ee2d9acff22b687af
          Log:
          Merge pull request #14 from davidparsson/master

          [FIXED JENKINS-15037] Added flag to consider only stable reference builds.

          Compare: https://github.com/jenkinsci/analysis-core-plugin/compare/37339d98c253...b4bf0441cd3e

          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Ulli Hafner Path: src/main/java/hudson/plugins/analysis/core/BuildHistory.java src/main/java/hudson/plugins/analysis/core/HealthAwarePublisher.java src/main/java/hudson/plugins/analysis/core/HealthAwareReporter.java src/main/resources/util/thresholds.jelly src/main/resources/util/thresholds.properties src/test/java/hudson/plugins/analysis/core/BuildHistoryTest.java http://jenkins-ci.org/commit/analysis-core-plugin/b4bf0441cd3ec069687ea06ee2d9acff22b687af Log: Merge pull request #14 from davidparsson/master [FIXED JENKINS-15037] Added flag to consider only stable reference builds. Compare: https://github.com/jenkinsci/analysis-core-plugin/compare/37339d98c253...b4bf0441cd3e
          scm_issue_link SCM/JIRA link daemon made changes -
          Resolution Fixed [ 1 ]
          Status Open [ 1 ] Resolved [ 5 ]

          Code changed in jenkins
          User: David Pärsson
          Path:
          pom.xml
          src/main/java/hudson/plugins/checkstyle/CheckStylePublisher.java
          src/main/java/hudson/plugins/checkstyle/CheckStyleReporter.java
          src/main/java/hudson/plugins/checkstyle/CheckStyleResult.java
          http://jenkins-ci.org/commit/checkstyle-plugin/6dd279810046a986945ced3d3922cfbab4d3a388
          Log:
          JENKINS-15037 Optionally consider only stable builds as reference builds

          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: David Pärsson Path: pom.xml src/main/java/hudson/plugins/checkstyle/CheckStylePublisher.java src/main/java/hudson/plugins/checkstyle/CheckStyleReporter.java src/main/java/hudson/plugins/checkstyle/CheckStyleResult.java http://jenkins-ci.org/commit/checkstyle-plugin/6dd279810046a986945ced3d3922cfbab4d3a388 Log: JENKINS-15037 Optionally consider only stable builds as reference builds

          Code changed in jenkins
          User: Ulli Hafner
          Path:
          pom.xml
          src/main/java/hudson/plugins/checkstyle/CheckStylePublisher.java
          src/main/java/hudson/plugins/checkstyle/CheckStyleReporter.java
          src/main/java/hudson/plugins/checkstyle/CheckStyleResult.java
          http://jenkins-ci.org/commit/checkstyle-plugin/cd2daf490dc04cc28d97285d85269848d1505421
          Log:
          Merge pull request #3 from davidparsson/master

          JENKINS-15037 Added flag to consider stable reference builds only.

          Compare: https://github.com/jenkinsci/checkstyle-plugin/compare/73c898ef1890...cd2daf490dc0

          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Ulli Hafner Path: pom.xml src/main/java/hudson/plugins/checkstyle/CheckStylePublisher.java src/main/java/hudson/plugins/checkstyle/CheckStyleReporter.java src/main/java/hudson/plugins/checkstyle/CheckStyleResult.java http://jenkins-ci.org/commit/checkstyle-plugin/cd2daf490dc04cc28d97285d85269848d1505421 Log: Merge pull request #3 from davidparsson/master JENKINS-15037 Added flag to consider stable reference builds only. Compare: https://github.com/jenkinsci/checkstyle-plugin/compare/73c898ef1890...cd2daf490dc0
          drulli Ulli Hafner added a comment -

          I tried to remove the deprecated constructors in the checkstyle plug-in now, but that is not that easy. Seems that I still need to think on how to distribute the new boolean flag to the maven and matrix classes... So please be patient until this will be released, I want to minimize the work in the other plug-ins (and maintain backward compatibility for other plugins that depend on analysis core).

          drulli Ulli Hafner added a comment - I tried to remove the deprecated constructors in the checkstyle plug-in now, but that is not that easy. Seems that I still need to think on how to distribute the new boolean flag to the maven and matrix classes... So please be patient until this will be released, I want to minimize the work in the other plug-ins (and maintain backward compatibility for other plugins that depend on analysis core).

          Ok, sure! While we're somewhat eager to get this, we of course understand that you must take your time to do it properly.

          davidparsson David Pärsson added a comment - Ok, sure! While we're somewhat eager to get this, we of course understand that you must take your time to do it properly.

          Code changed in jenkins
          User: Ulli Hafner
          Path:
          src/main/java/hudson/plugins/analysis/core/AbstractProjectAction.java
          src/main/java/hudson/plugins/analysis/core/AnnotationsAggregator.java
          src/main/java/hudson/plugins/analysis/core/BuildHistory.java
          src/main/java/hudson/plugins/analysis/core/BuildResult.java
          src/main/java/hudson/plugins/analysis/core/HealthAwarePublisher.java
          src/main/java/hudson/plugins/analysis/core/HealthAwareRecorder.java
          src/main/java/hudson/plugins/analysis/core/HealthAwareReporter.java
          src/main/java/hudson/plugins/analysis/core/NullBuildHistory.java
          src/test/java/hudson/plugins/analysis/core/BuildHistoryTest.java
          http://jenkins-ci.org/commit/analysis-core-plugin/e9fbcb2fc9769f9cde9f073be92c39525b64e635
          Log:
          JENKINS-15037 Added flag to consider stable reference builds only.

          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Ulli Hafner Path: src/main/java/hudson/plugins/analysis/core/AbstractProjectAction.java src/main/java/hudson/plugins/analysis/core/AnnotationsAggregator.java src/main/java/hudson/plugins/analysis/core/BuildHistory.java src/main/java/hudson/plugins/analysis/core/BuildResult.java src/main/java/hudson/plugins/analysis/core/HealthAwarePublisher.java src/main/java/hudson/plugins/analysis/core/HealthAwareRecorder.java src/main/java/hudson/plugins/analysis/core/HealthAwareReporter.java src/main/java/hudson/plugins/analysis/core/NullBuildHistory.java src/test/java/hudson/plugins/analysis/core/BuildHistoryTest.java http://jenkins-ci.org/commit/analysis-core-plugin/e9fbcb2fc9769f9cde9f073be92c39525b64e635 Log: JENKINS-15037 Added flag to consider stable reference builds only.

          Code changed in jenkins
          User: Ulli Hafner
          Path:
          src/main/resources/util/thresholds_de.properties
          http://jenkins-ci.org/commit/analysis-core-plugin/fa56e0b8a18c476e956da4f071daab813cf2d780
          Log:
          JENKINS-15037 Added flag to consider stable reference builds only.

          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Ulli Hafner Path: src/main/resources/util/thresholds_de.properties http://jenkins-ci.org/commit/analysis-core-plugin/fa56e0b8a18c476e956da4f071daab813cf2d780 Log: JENKINS-15037 Added flag to consider stable reference builds only.

          Code changed in jenkins
          User: Ulli Hafner
          Path:
          src/main/java/hudson/plugins/checkstyle/CheckStyleAnnotationsAggregator.java
          src/main/java/hudson/plugins/checkstyle/CheckStyleMavenResult.java
          src/main/java/hudson/plugins/checkstyle/CheckStyleMavenResultAction.java
          src/main/java/hudson/plugins/checkstyle/CheckStylePublisher.java
          src/main/java/hudson/plugins/checkstyle/CheckStyleReporter.java
          src/main/java/hudson/plugins/checkstyle/CheckStyleReporterResult.java
          src/main/java/hudson/plugins/checkstyle/CheckStyleResult.java
          http://jenkins-ci.org/commit/checkstyle-plugin/3839a1d61430a3d0d878d54d75b7e40e15d3de02
          Log:
          JENKINS-15037 Added flag to consider stable reference builds only.

          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Ulli Hafner Path: src/main/java/hudson/plugins/checkstyle/CheckStyleAnnotationsAggregator.java src/main/java/hudson/plugins/checkstyle/CheckStyleMavenResult.java src/main/java/hudson/plugins/checkstyle/CheckStyleMavenResultAction.java src/main/java/hudson/plugins/checkstyle/CheckStylePublisher.java src/main/java/hudson/plugins/checkstyle/CheckStyleReporter.java src/main/java/hudson/plugins/checkstyle/CheckStyleReporterResult.java src/main/java/hudson/plugins/checkstyle/CheckStyleResult.java http://jenkins-ci.org/commit/checkstyle-plugin/3839a1d61430a3d0d878d54d75b7e40e15d3de02 Log: JENKINS-15037 Added flag to consider stable reference builds only.

          Code changed in jenkins
          User: Ulli Hafner
          Path:
          src/main/java/hudson/plugins/analysis/collector/AnalysisAnnotationsAggregator.java
          src/main/java/hudson/plugins/analysis/collector/AnalysisPublisher.java
          src/main/java/hudson/plugins/analysis/collector/AnalysisResult.java
          src/main/java/hudson/plugins/analysis/collector/AnalysisResultAction.java
          src/main/resources/hudson/plugins/analysis/collector/AnalysisPublisher/config.jelly
          http://jenkins-ci.org/commit/analysis-collector-plugin/019fe5126b9ee42fff7ffeae9d322eec2582e17e
          Log:
          JENKINS-15037: Upgrade to new API in order to compute warnings only
          for stable builds (if option is checked).

          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Ulli Hafner Path: src/main/java/hudson/plugins/analysis/collector/AnalysisAnnotationsAggregator.java src/main/java/hudson/plugins/analysis/collector/AnalysisPublisher.java src/main/java/hudson/plugins/analysis/collector/AnalysisResult.java src/main/java/hudson/plugins/analysis/collector/AnalysisResultAction.java src/main/resources/hudson/plugins/analysis/collector/AnalysisPublisher/config.jelly http://jenkins-ci.org/commit/analysis-collector-plugin/019fe5126b9ee42fff7ffeae9d322eec2582e17e Log: JENKINS-15037 : Upgrade to new API in order to compute warnings only for stable builds (if option is checked).

          Code changed in jenkins
          User: Ulli Hafner
          Path:
          src/main/java/hudson/plugins/dry/DryAnnotationsAggregator.java
          src/main/java/hudson/plugins/dry/DryMavenResult.java
          src/main/java/hudson/plugins/dry/DryMavenResultAction.java
          src/main/java/hudson/plugins/dry/DryPublisher.java
          src/main/java/hudson/plugins/dry/DryReporter.java
          src/main/java/hudson/plugins/dry/DryReporterResult.java
          src/main/java/hudson/plugins/dry/DryResult.java
          src/main/java/hudson/plugins/dry/DryResultAction.java
          src/main/java/hudson/plugins/dry/MavenDryResultAction.java
          src/main/resources/hudson/plugins/dry/DryPublisher/config.jelly
          src/main/resources/hudson/plugins/dry/DryReporter/config.jelly
          http://jenkins-ci.org/commit/dry-plugin/a715b62caf14f978da2e50dd89fc48beabe2356f
          Log:
          JENKINS-15037: Upgrade to new API in order to compute warnings only
          for stable builds (if option is checked).

          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Ulli Hafner Path: src/main/java/hudson/plugins/dry/DryAnnotationsAggregator.java src/main/java/hudson/plugins/dry/DryMavenResult.java src/main/java/hudson/plugins/dry/DryMavenResultAction.java src/main/java/hudson/plugins/dry/DryPublisher.java src/main/java/hudson/plugins/dry/DryReporter.java src/main/java/hudson/plugins/dry/DryReporterResult.java src/main/java/hudson/plugins/dry/DryResult.java src/main/java/hudson/plugins/dry/DryResultAction.java src/main/java/hudson/plugins/dry/MavenDryResultAction.java src/main/resources/hudson/plugins/dry/DryPublisher/config.jelly src/main/resources/hudson/plugins/dry/DryReporter/config.jelly http://jenkins-ci.org/commit/dry-plugin/a715b62caf14f978da2e50dd89fc48beabe2356f Log: JENKINS-15037 : Upgrade to new API in order to compute warnings only for stable builds (if option is checked).

          Code changed in jenkins
          User: Ulli Hafner
          Path:
          plugin/src/main/java/hudson/plugins/findbugs/FindBugsAnnotationsAggregator.java
          plugin/src/main/java/hudson/plugins/findbugs/FindBugsMavenResult.java
          plugin/src/main/java/hudson/plugins/findbugs/FindBugsMavenResultAction.java
          plugin/src/main/java/hudson/plugins/findbugs/FindBugsPublisher.java
          plugin/src/main/java/hudson/plugins/findbugs/FindBugsReporter.java
          plugin/src/main/java/hudson/plugins/findbugs/FindBugsReporterResult.java
          plugin/src/main/java/hudson/plugins/findbugs/FindBugsResult.java
          plugin/src/main/java/hudson/plugins/findbugs/FindBugsResultAction.java
          plugin/src/main/java/hudson/plugins/findbugs/MavenFindBugsResultAction.java
          plugin/src/main/resources/hudson/plugins/findbugs/FindBugsPublisher/config.jelly
          plugin/src/main/resources/hudson/plugins/findbugs/FindBugsResult/index.jelly
          http://jenkins-ci.org/commit/findbugs-plugin/070456af4914838b5bb3dd16bd9f1aef242056a0
          Log:
          JENKINS-15037: Upgrade to new API in order to compute warnings only
          for stable builds (if option is checked).

          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Ulli Hafner Path: plugin/src/main/java/hudson/plugins/findbugs/FindBugsAnnotationsAggregator.java plugin/src/main/java/hudson/plugins/findbugs/FindBugsMavenResult.java plugin/src/main/java/hudson/plugins/findbugs/FindBugsMavenResultAction.java plugin/src/main/java/hudson/plugins/findbugs/FindBugsPublisher.java plugin/src/main/java/hudson/plugins/findbugs/FindBugsReporter.java plugin/src/main/java/hudson/plugins/findbugs/FindBugsReporterResult.java plugin/src/main/java/hudson/plugins/findbugs/FindBugsResult.java plugin/src/main/java/hudson/plugins/findbugs/FindBugsResultAction.java plugin/src/main/java/hudson/plugins/findbugs/MavenFindBugsResultAction.java plugin/src/main/resources/hudson/plugins/findbugs/FindBugsPublisher/config.jelly plugin/src/main/resources/hudson/plugins/findbugs/FindBugsResult/index.jelly http://jenkins-ci.org/commit/findbugs-plugin/070456af4914838b5bb3dd16bd9f1aef242056a0 Log: JENKINS-15037 : Upgrade to new API in order to compute warnings only for stable builds (if option is checked).

          Code changed in jenkins
          User: Ulli Hafner
          Path:
          src/main/java/hudson/plugins/pmd/MavenPmdResultAction.java
          src/main/java/hudson/plugins/pmd/PmdAnnotationsAggregator.java
          src/main/java/hudson/plugins/pmd/PmdMavenResult.java
          src/main/java/hudson/plugins/pmd/PmdMavenResultAction.java
          src/main/java/hudson/plugins/pmd/PmdPublisher.java
          src/main/java/hudson/plugins/pmd/PmdReporter.java
          src/main/java/hudson/plugins/pmd/PmdReporterResult.java
          src/main/java/hudson/plugins/pmd/PmdResult.java
          src/main/java/hudson/plugins/pmd/PmdResultAction.java
          src/main/resources/hudson/plugins/pmd/PmdReporter/config.jelly
          http://jenkins-ci.org/commit/pmd-plugin/fba349e969bfdfaebfdfaa9785dd6b9d4841c88e
          Log:
          JENKINS-15037: Upgrade to new API in order to compute warnings only
          for stable builds (if option is checked).

          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Ulli Hafner Path: src/main/java/hudson/plugins/pmd/MavenPmdResultAction.java src/main/java/hudson/plugins/pmd/PmdAnnotationsAggregator.java src/main/java/hudson/plugins/pmd/PmdMavenResult.java src/main/java/hudson/plugins/pmd/PmdMavenResultAction.java src/main/java/hudson/plugins/pmd/PmdPublisher.java src/main/java/hudson/plugins/pmd/PmdReporter.java src/main/java/hudson/plugins/pmd/PmdReporterResult.java src/main/java/hudson/plugins/pmd/PmdResult.java src/main/java/hudson/plugins/pmd/PmdResultAction.java src/main/resources/hudson/plugins/pmd/PmdReporter/config.jelly http://jenkins-ci.org/commit/pmd-plugin/fba349e969bfdfaebfdfaa9785dd6b9d4841c88e Log: JENKINS-15037 : Upgrade to new API in order to compute warnings only for stable builds (if option is checked).

          Code changed in jenkins
          User: Ulli Hafner
          Path:
          src/main/java/hudson/plugins/warnings/WarningsAnnotationsAggregator.java
          src/main/java/hudson/plugins/warnings/WarningsBuildHistory.java
          src/main/java/hudson/plugins/warnings/WarningsProjectAction.java
          src/main/java/hudson/plugins/warnings/WarningsPublisher.java
          src/main/java/hudson/plugins/warnings/WarningsResult.java
          src/main/resources/hudson/plugins/warnings/WarningsPublisher/config.jelly
          src/test/java/hudson/plugins/warnings/WarningsPublisherTest.java
          http://jenkins-ci.org/commit/warnings-plugin/559c94f82212c265803bd14efa5fe5c3d00f172d
          Log:
          JENKINS-15037: Upgrade to new API in order to compute warnings only
          for stable builds (if option is checked).

          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Ulli Hafner Path: src/main/java/hudson/plugins/warnings/WarningsAnnotationsAggregator.java src/main/java/hudson/plugins/warnings/WarningsBuildHistory.java src/main/java/hudson/plugins/warnings/WarningsProjectAction.java src/main/java/hudson/plugins/warnings/WarningsPublisher.java src/main/java/hudson/plugins/warnings/WarningsResult.java src/main/resources/hudson/plugins/warnings/WarningsPublisher/config.jelly src/test/java/hudson/plugins/warnings/WarningsPublisherTest.java http://jenkins-ci.org/commit/warnings-plugin/559c94f82212c265803bd14efa5fe5c3d00f172d Log: JENKINS-15037 : Upgrade to new API in order to compute warnings only for stable builds (if option is checked).

          Code changed in jenkins
          User: Ulli Hafner
          Path:
          src/main/java/hudson/plugins/checkstyle/CheckStyleMavenResultAction.java
          src/main/resources/hudson/plugins/checkstyle/CheckStylePublisher/config.jelly
          src/main/resources/hudson/plugins/checkstyle/CheckStyleReporter/config.jelly
          http://jenkins-ci.org/commit/checkstyle-plugin/d8c652dc9f5826e7ed72a255b3776b60bca4c0f2
          Log:
          JENKINS-15037: Upgrade to new API in order to compute warnings only
          for stable builds (if option is checked).

          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Ulli Hafner Path: src/main/java/hudson/plugins/checkstyle/CheckStyleMavenResultAction.java src/main/resources/hudson/plugins/checkstyle/CheckStylePublisher/config.jelly src/main/resources/hudson/plugins/checkstyle/CheckStyleReporter/config.jelly http://jenkins-ci.org/commit/checkstyle-plugin/d8c652dc9f5826e7ed72a255b3776b60bca4c0f2 Log: JENKINS-15037 : Upgrade to new API in order to compute warnings only for stable builds (if option is checked).

          Code changed in jenkins
          User: Ulli Hafner
          Path:
          src/main/java/hudson/plugins/tasks/MavenTasksResultAction.java
          src/main/java/hudson/plugins/tasks/TasksAnnotationsAggregator.java
          src/main/java/hudson/plugins/tasks/TasksMavenResult.java
          src/main/java/hudson/plugins/tasks/TasksMavenResultAction.java
          src/main/java/hudson/plugins/tasks/TasksPublisher.java
          src/main/java/hudson/plugins/tasks/TasksReporter.java
          src/main/java/hudson/plugins/tasks/TasksReporterResult.java
          src/main/java/hudson/plugins/tasks/TasksResult.java
          src/main/java/hudson/plugins/tasks/TasksResultAction.java
          src/main/resources/hudson/plugins/tasks/FixedTasksDetail/index.jelly
          http://jenkins-ci.org/commit/tasks-plugin/7fc0ef593e20049dd7f627834945f22ce8d5f6ff
          Log:
          JENKINS-15037: Upgrade to new API in order to compute warnings only
          for stable builds (if option is checked).

          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Ulli Hafner Path: src/main/java/hudson/plugins/tasks/MavenTasksResultAction.java src/main/java/hudson/plugins/tasks/TasksAnnotationsAggregator.java src/main/java/hudson/plugins/tasks/TasksMavenResult.java src/main/java/hudson/plugins/tasks/TasksMavenResultAction.java src/main/java/hudson/plugins/tasks/TasksPublisher.java src/main/java/hudson/plugins/tasks/TasksReporter.java src/main/java/hudson/plugins/tasks/TasksReporterResult.java src/main/java/hudson/plugins/tasks/TasksResult.java src/main/java/hudson/plugins/tasks/TasksResultAction.java src/main/resources/hudson/plugins/tasks/FixedTasksDetail/index.jelly http://jenkins-ci.org/commit/tasks-plugin/7fc0ef593e20049dd7f627834945f22ce8d5f6ff Log: JENKINS-15037 : Upgrade to new API in order to compute warnings only for stable builds (if option is checked).
          drulli Ulli Hafner added a comment -

          Integrated in Jenkins Analysis Plug-ins (Compile) #609
          JENKINS-15037 Optionally consider only stable builds as reference builds (Revision f73958097bad7344fb12c75d8aa2928572a51c60)
          JENKINS-15037 Added flag to consider stable reference builds only. (Revision e9fbcb2fc9769f9cde9f073be92c39525b64e635)
          JENKINS-15037 Added flag to consider stable reference builds only. (Revision fa56e0b8a18c476e956da4f071daab813cf2d780)

          Result = SUCCESS

          drulli Ulli Hafner added a comment - Integrated in Jenkins Analysis Plug-ins (Compile) #609 JENKINS-15037 Optionally consider only stable builds as reference builds (Revision f73958097bad7344fb12c75d8aa2928572a51c60) JENKINS-15037 Added flag to consider stable reference builds only. (Revision e9fbcb2fc9769f9cde9f073be92c39525b64e635) JENKINS-15037 Added flag to consider stable reference builds only. (Revision fa56e0b8a18c476e956da4f071daab813cf2d780) Result = SUCCESS
          rtyler R. Tyler Croy made changes -
          Workflow JNJira [ 145756 ] JNJira + In-Review [ 191615 ]

          People

            drulli Ulli Hafner
            davidparsson David Pärsson
            Votes:
            1 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: