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

Make Warnings Plugin compatible with parallel Workflow

    XMLWordPrintable

Details

    • 5.0.0-beta2

    Description

      29959 made it compatible with a single build.
      But it doesn't yet work nicely for multiple parallel builds.

      If one puts the WarningsPublisher inside each parallel task, then the build page contains a link to the warning output once per task. But each link points to the same page and only contains the warnings of one task.

      If instead one runs the WarningsPublisher step only once after all all parallel tasks, then the links to the source code do not work (because the workspace path is different on the different node) and the list of warnings doesn't show for which task/node the warning occurred.

      Based on how other plugins (e.g. junit) work, I assume the first is how it should work. To it work, the warnings plugin should summarize the warnings before reporting them.

      To make the 2nd work it would need to parse the task label in the console output (to display the task) and resolve the workspace relative to the node of that task.

      Is anyone working on this? Any suggestions of how to get started on fixing this?

      Attachments

        Issue Links

          Activity

            Code changed in jenkins
            User: Ulli Hafner
            Path:
            src/main/java/io/jenkins/plugins/analysis/core/views/ResultAction.java
            src/test/java/io/jenkins/plugins/analysis/core/views/ResultActionTest.java
            http://jenkins-ci.org/commit/analysis-core-plugin/17c4d9f1e018b25814da27b75994dcb8c31fe2b8
            Log:
            JENKINS-30551 Make ResultAction serializable.

            *NOTE:* This service been marked for deprecation: https://developer.github.com/changes/2018-04-25-github-services-deprecation/

            Functionality will be removed from GitHub.com on January 31st, 2019.

            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Ulli Hafner Path: src/main/java/io/jenkins/plugins/analysis/core/views/ResultAction.java src/test/java/io/jenkins/plugins/analysis/core/views/ResultActionTest.java http://jenkins-ci.org/commit/analysis-core-plugin/17c4d9f1e018b25814da27b75994dcb8c31fe2b8 Log: JENKINS-30551 Make ResultAction serializable. * NOTE: * This service been marked for deprecation: https://developer.github.com/changes/2018-04-25-github-services-deprecation/ Functionality will be removed from GitHub.com on January 31st, 2019.

            Code changed in jenkins
            User: Ulli Hafner
            Path:
            src/test/java/io/jenkins/plugins/analysis/core/testutil/IntegrationTest.java
            http://jenkins-ci.org/commit/analysis-core-plugin/59d761b521ff2eb229e828215750a9d9e562c475
            Log:
            JENKINS-30551 Added a test case.

            *NOTE:* This service been marked for deprecation: https://developer.github.com/changes/2018-04-25-github-services-deprecation/

            Functionality will be removed from GitHub.com on January 31st, 2019.

            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Ulli Hafner Path: src/test/java/io/jenkins/plugins/analysis/core/testutil/IntegrationTest.java http://jenkins-ci.org/commit/analysis-core-plugin/59d761b521ff2eb229e828215750a9d9e562c475 Log: JENKINS-30551 Added a test case. * NOTE: * This service been marked for deprecation: https://developer.github.com/changes/2018-04-25-github-services-deprecation/ Functionality will be removed from GitHub.com on January 31st, 2019.

            Code changed in jenkins
            User: Ulli Hafner
            Path:
            src/test/java/io/jenkins/plugins/analysis/warnings/FreestyleJobITest.java
            src/test/java/io/jenkins/plugins/analysis/warnings/IssuesRecorderITest.java
            src/test/java/io/jenkins/plugins/analysis/warnings/PipelineITest.java
            src/test/java/io/jenkins/plugins/analysis/warnings/StepsITest.java
            src/test/resources/io/jenkins/plugins/analysis/warnings/parallel.jenkinsfile
            http://jenkins-ci.org/commit/warnings-plugin/0315547bc1a3084c0ef34a9153a7ac2d81ac87a4
            Log:
            JENKINS-30551 Added a test case.

            *NOTE:* This service been marked for deprecation: https://developer.github.com/changes/2018-04-25-github-services-deprecation/

            Functionality will be removed from GitHub.com on January 31st, 2019.

            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Ulli Hafner Path: src/test/java/io/jenkins/plugins/analysis/warnings/FreestyleJobITest.java src/test/java/io/jenkins/plugins/analysis/warnings/IssuesRecorderITest.java src/test/java/io/jenkins/plugins/analysis/warnings/PipelineITest.java src/test/java/io/jenkins/plugins/analysis/warnings/StepsITest.java src/test/resources/io/jenkins/plugins/analysis/warnings/parallel.jenkinsfile http://jenkins-ci.org/commit/warnings-plugin/0315547bc1a3084c0ef34a9153a7ac2d81ac87a4 Log: JENKINS-30551 Added a test case. * NOTE: * This service been marked for deprecation: https://developer.github.com/changes/2018-04-25-github-services-deprecation/ Functionality will be removed from GitHub.com on January 31st, 2019.
            drulli Ulli Hafner added a comment -

            What still needs to be checked:

            • If a build is done in parallel, then the log files might be distributed on different machines. Where is the scanner invoked, how can the scanner access the different log files.
            • See JENKINS-45774
            drulli Ulli Hafner added a comment - What still needs to be checked: If a build is done in parallel, then the log files might be distributed on different machines. Where is the scanner invoked, how can the scanner access the different log files. See JENKINS-45774
            drulli Ulli Hafner added a comment -

            Released in 5.0.0-beta2.

            drulli Ulli Hafner added a comment - Released in 5.0.0-beta2.

            People

              drulli Ulli Hafner
              rschulz Roland Schulz
              Votes:
              4 Vote for this issue
              Watchers:
              13 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: