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

Add to build log indication if a known cause was found

      Today, the plugin writes to the build log that it is scanning the log, but doesn't mention if a known cause was found.
      This might be useful to trigger other plugins (for example - a build retry plugin)

          [JENKINS-25751] Add to build log indication if a known cause was found

          Kari Sivonen added a comment -

          I made my own modifications and it looks like this can be done quite easily. Only BuildFailureScanner.java need to modify.

          Diff file attached. How to continue with this? Would you like to review and do this changes based on diff file, or should I try to do PullRequest?

          Kari Sivonen added a comment - I made my own modifications and it looks like this can be done quite easily. Only BuildFailureScanner.java need to modify. Diff file attached. How to continue with this? Would you like to review and do this changes based on diff file, or should I try to do PullRequest?

          Do a pull request, I think that is the best way to handle it.

          Tomas Westling added a comment - Do a pull request, I think that is the best way to handle it.

          Kari Sivonen added a comment - - edited

          Pull request is now ready. Please merge it to trunk.

          Kari Sivonen added a comment - - edited Pull request is now ready. Please merge it to trunk.

          Kari Sivonen added a comment -

          Now I think pull request is ready. All test is passed. If you not see anything wrong it, please merge to trunk. Or how this should continue?

          Kari Sivonen added a comment - Now I think pull request is ready. All test is passed. If you not see anything wrong it, please merge to trunk. Or how this should continue?

          Code changed in jenkins
          User: Sivonen Kari Matti Juhani
          Path:
          src/main/java/com/sonyericsson/jenkins/plugins/bfa/BuildFailureScanner.java
          http://jenkins-ci.org/commit/build-failure-analyzer-plugin/7b27fcb86dc2c4a88d784ed9477a8e23871a7efa
          Log:
          JENKINS-25751 Add to build log indication if a known cause was found

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Sivonen Kari Matti Juhani Path: src/main/java/com/sonyericsson/jenkins/plugins/bfa/BuildFailureScanner.java http://jenkins-ci.org/commit/build-failure-analyzer-plugin/7b27fcb86dc2c4a88d784ed9477a8e23871a7efa Log: JENKINS-25751 Add to build log indication if a known cause was found

          Code changed in jenkins
          User: Tomas Westling
          Path:
          src/main/java/com/sonyericsson/jenkins/plugins/bfa/BuildFailureScanner.java
          http://jenkins-ci.org/commit/build-failure-analyzer-plugin/74192b987f7107e90029a8932e73fc4a8c4243a6
          Log:
          Merge pull request #35 from karisivo/master

          JENKINS-25751 Add to build log indication if a known cause was found

          Compare: https://github.com/jenkinsci/build-failure-analyzer-plugin/compare/01a2056b7bb4...74192b987f71

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Tomas Westling Path: src/main/java/com/sonyericsson/jenkins/plugins/bfa/BuildFailureScanner.java http://jenkins-ci.org/commit/build-failure-analyzer-plugin/74192b987f7107e90029a8932e73fc4a8c4243a6 Log: Merge pull request #35 from karisivo/master JENKINS-25751 Add to build log indication if a known cause was found Compare: https://github.com/jenkinsci/build-failure-analyzer-plugin/compare/01a2056b7bb4...74192b987f71

          I merged the pull request, this issue can be closed.

          Tomas Westling added a comment - I merged the pull request, this issue can be closed.

          Tomas Westling added a comment - Feature done in https://github.com/jenkinsci/build-failure-analyzer-plugin/pull/35

          Feature hasn't been released yet, keeping this open until it is.

          Tomas Westling added a comment - Feature hasn't been released yet, keeping this open until it is.

            t_westling Tomas Westling
            galunto Tomer Galun
            Votes:
            1 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated: