Details
-
Type:
Improvement
-
Status: Resolved (View Workflow)
-
Priority:
Minor
-
Resolution: Fixed
-
Component/s: analysis-core-plugin
-
Labels:None
-
Similar Issues:
Description
I would like to have a job which sends a warning mail whenever a new checkstyle warning appears in a project. However, this job should not be unstable or failing after this one new build because of the warning. With the current implementation this is not possible as the build will be failing or unstable until that warning is fixed again. A simple way to achieve this behavior is to always use the latest build as the reference build, so that old warning are bascially ignored in the new running build.
Attachments
Issue Links
- is duplicated by
-
JENKINS-17690 Checkbox "Only use stable builds as reference" probably doesnt work
-
- Resolved
-
-
JENKINS-24289 Improve description of "Only use stable builds as reference" option
-
- Resolved
-
-
JENKINS-21416 Findbugs new warnings based on last successful build when findbugs plugin is cause of unstable builds
-
- Resolved
-
-
JENKINS-16481 Make it configurable if new findbugs for unstable builds should be computed by comparison to the reference build (last stable build)
-
- Resolved
-
Activity
Well, this feature seems not very useful in my projects (you just need to press build again to get rid of the unstable status). (However I'm willing to integrate a pull request that adds that feature, so if you are interested please go ahead. It should not be that complicated.)
I was looking into providing this but was having issues with the inheritance from the -core, and presentation in the job (Can get a setting saved per job, but cannot display it back in the configuration).
I'm not sure what you mean by pressing build again getting rid of the unstable status. Simply pressing "Build Now" on the project does not work for me--it still uses the old reference build, and therefore fails again. Am I missing something?
@Ivan: if this feature will be implemented then you can press "Build now" again to reset the build status to stable. This is not really desired, feels like a bit of cheating to me...
@Thomas: I can can give you some pointers on where to start if you are still interested...
@Ulli, now I understand what you are saying. But that's exactly what I want because the warnings I get are often not really valid and/or impossible to fix (e.g. bugs in the tools). Some are good though, and so I want to know about new warnings so I can potentially fix them. But if I don't want to fix them I don't want to be constantly pestered.
Thanks.
Similiarly to Ivan, I have a project on which I want to know when issues are being introduced, and let the people who have introduced know at the time, but I don't want everyone who has ever introduced an issue being emailed every time anyone triggers an update until all issues are resolved.
Code changed in jenkins
User: Ulli Hafner
Path:
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/HealthAwarePublisher.java
src/main/java/hudson/plugins/analysis/core/HealthAwareRecorder.java
src/main/java/hudson/plugins/analysis/core/HealthAwareReporter.java
src/main/resources/util/thresholds.jelly
src/main/resources/util/thresholds.properties
http://jenkins-ci.org/commit/analysis-core-plugin/b8712ca11282696b1e8d51eac8a45880a59dff72
Log:
Merge pull request #26 from tom-saunders/master
JENKINS-13458 Added option to use the previous build as reference.
Compare: https://github.com/jenkinsci/analysis-core-plugin/compare/e001040655c1...b8712ca11282
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/PmdMavenResultAction.java
src/main/java/hudson/plugins/pmd/PmdPublisher.java
src/main/java/hudson/plugins/pmd/PmdReporter.java
src/main/java/hudson/plugins/pmd/PmdResult.java
http://jenkins-ci.org/commit/pmd-plugin/9494c7483bef499a10f1585ddfde004d0718ed07
Log:
Merge pull request #2 from tom-saunders/master
JENKINS-13458 Added option to use the previous build always as reference.
Compare: https://github.com/jenkinsci/pmd-plugin/compare/0724a86ae7c7...9494c7483bef
Code changed in jenkins
User: Ulli Hafner
Path:
src/main/java/org/jenkinsci/test/acceptance/plugins/analysis_core/AnalysisSettings.java
src/main/java/org/jenkinsci/test/acceptance/po/Build.java
src/test/java/plugins/CheckStylePluginTest.java
src/test/resources/checkstyle_plugin/checkstyle-result-build1.xml
src/test/resources/checkstyle_plugin/checkstyle-result-build2.xml
src/test/resources/checkstyle_plugin/checkstyle-result-build3.xml
src/test/resources/checkstyle_plugin/checkstyle-result-build4.xml
http://jenkins-ci.org/commit/acceptance-test-harness/67eaa6d48a03db24cf537c482f819d9d171703ef
Log:
JENKINS-13458 Added a testcase to verify the original delta computation.
This test ensures that the build status is computed from the warnings delta of the current build and the reference build.
Code changed in jenkins
User: Ulli Hafner
Path:
src/main/java/org/jenkinsci/test/acceptance/plugins/analysis_core/AnalysisSettings.java
src/test/java/plugins/AbstractAnalysisTest.java
src/test/java/plugins/CheckStylePluginTest.java
http://jenkins-ci.org/commit/acceptance-test-harness/4a3fd08b1f13d3990cbcc40284055f131fe82eae
Log:
JENKINS-13458 Added a testcase to verify the new delta computation.
This test ensures that the build status is computed from the warnings delta of the current build and the previous build.
Code changed in jenkins
User: Ulli Hafner
Path:
src/test/java/plugins/PmdPluginTest.java
src/test/resources/pmd_plugin/pmd-warnings-build1.xml
src/test/resources/pmd_plugin/pmd-warnings-build2.xml
src/test/resources/pmd_plugin/pmd-warnings-build3.xml
src/test/resources/pmd_plugin/pmd-warnings-build4.xml
http://jenkins-ci.org/commit/acceptance-test-harness/2b967fc8a1b3cef6c843b026186574214b0c0cb8
Log:
JENKINS-13458 Added test cases to verify the delta computation.
Two tests: one to verify the existing and one to verify the new behavior.
Code changed in jenkins
User: Ulli Hafner
Path:
src/main/java/hudson/plugins/analysis/core/AbstractAnnotationParser.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/resources/util/thresholds.jelly
src/main/resources/util/thresholds.properties
src/main/resources/util/thresholds_de.properties
src/test/java/hudson/plugins/analysis/core/BuildHistoryTest.java
http://jenkins-ci.org/commit/analysis-core-plugin/1a2ebe2a80da6cdf429a442cc3b2be8494337441
Log:
[FIXED JENKINS-13458] Added an option to use previous build as a reference.
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/MavenFindBugsResultAction.java
http://jenkins-ci.org/commit/findbugs-plugin/2b0b1b8d1b3da6fdb454b4e3caabce62e22c8a7d
Log:
[FIXED JENKINS-13458] Added an option to use previous build as a reference.
Code changed in jenkins
User: Ulli Hafner
Path:
.idea/codeStyleSettings.xml
.idea/libraries/Maven__org_jvnet_hudson_plugins_analysis_core_1_65.xml
.idea/libraries/Maven__org_jvnet_hudson_plugins_analysis_test_1_11.xml
analysis-collector
analysis-core
dry
pmd
tasks
warnings
http://jenkins-ci.org/commit/analysis-suite-plugin/43474638ad5a84ca7d9a697a1ad9d19119f603b1
Log:
[FIXED JENKINS-13458] Added an option to use previous build as a reference.
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/MavenDryResultAction.java
http://jenkins-ci.org/commit/dry-plugin/ee4a8b6203a4caf10855dd8ddf5341a6f55e08ee
Log:
[FIXED JENKINS-13458] Added an option to use previous build as a reference.
Code changed in jenkins
User: Ulli Hafner
Path:
checkstyle.iml
pom.xml
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
src/main/java/hudson/plugins/checkstyle/MavenCheckStyleResultAction.java
src/main/java/hudson/plugins/checkstyle/parser/CheckStyleParser.java
src/test/java/hudson/plugins/checkstyle/parser/CheckStyleParserTest.java
src/test/resources/hudson/plugins/checkstyle/parser/checkstyle-result-build1.xml
src/test/resources/hudson/plugins/checkstyle/parser/checkstyle-result-build2.xml
src/test/resources/hudson/plugins/checkstyle/parser/checkstyle-result-build3.xml
src/test/resources/hudson/plugins/checkstyle/parser/checkstyle-result-build4.xml
http://jenkins-ci.org/commit/checkstyle-plugin/369e5fb065dc8fa3137f0c9016451d3e11a8f5fc
Log:
[FIXED JENKINS-13458] Added an option to use previous build as a reference.
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
http://jenkins-ci.org/commit/analysis-collector-plugin/8c4ac1d376dfa056f60c7968d426a7a0aed0982d
Log:
[FIXED JENKINS-13458] Added an option to use previous build as a reference.
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
http://jenkins-ci.org/commit/tasks-plugin/0ae7469b4f68430d7c2d29567fb80ef41819b989
Log:
[FIXED JENKINS-13458] Added an option to use previous build as a reference.
Code changed in jenkins
User: Ulli Hafner
Path:
pmd.iml
pom.xml
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/parser/PmdParser.java
http://jenkins-ci.org/commit/pmd-plugin/bf11ab5a02fdbbae20a7503fad468723f6bee29b
Log:
[FIXED JENKINS-13458] Added an option to use previous build as a reference.
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/test/java/hudson/plugins/warnings/WarningsPublisherTest.java
http://jenkins-ci.org/commit/warnings-plugin/96c2df676a21dc3c3368f8e91e7c188d2946403e
Log:
[FIXED JENKINS-13458] Added an option to use previous build as a reference.
Any thoughts on this? I would really like this.