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

Publishing Selenium report blocks concurrent builds

    Details

    • Similar Issues:

      Description

      Set up a job with parameters that does a single "Publish Selenium Report" at the end.
      Then set this job to "Execute concurrent builds if necessary".

      Now run this job concurrently with different parameters.

      If 3 concurrent builds are running, then all 3 builds will block until all 3 builds are complete to publish the selenium report.

      i.e. if build 1 takes 5 minutes, build 2 takes 2 minutes, and build 3 takes 9 minutes - build 1 & 2 will wait until build 3 is done to publish the selenium report on all 3 builds.

      Ideally, this shouldn't block at all, and let concurrent builds kick through.

        Attachments

          Activity

          Hide
          markmandel markmandel added a comment - - edited

          I just noted that this doesn't seem to occur when using a multi-configuration job.

          Show
          markmandel markmandel added a comment - - edited I just noted that this doesn't seem to occur when using a multi-configuration job.
          Hide
          orrc Christopher Orr added a comment -

          Reassigning this to the "seleniumhtmlreport" component, as that's where ths functionality lies.

          As discussed in IRC, the plugin can't work with concurrent builds as it returns BuildStepMonitor.BUILD from its getRequiredMonitorService() method.

          I'm not sure whether this is really required (for example, the HTML Publisher plugin doesn't do this), but I guess the maintainer will have a more definitive idea.

          Show
          orrc Christopher Orr added a comment - Reassigning this to the "seleniumhtmlreport" component, as that's where ths functionality lies. As discussed in IRC, the plugin can't work with concurrent builds as it returns BuildStepMonitor.BUILD from its getRequiredMonitorService() method. I'm not sure whether this is really required (for example, the HTML Publisher plugin doesn't do this), but I guess the maintainer will have a more definitive idea.
          Hide
          orrc Christopher Orr added a comment -

          Ok, make that the "seleniumhq" plugin! Same issue though.

          Show
          orrc Christopher Orr added a comment - Ok, make that the "seleniumhq" plugin! Same issue though.
          Hide
          m211 m211 added a comment -

          hi christopher,
          the method getRequiredMonitorService() is a relict of copying from seleniumhq.
          i will change the return value to BuildStepMonitor.NONE.

          Show
          m211 m211 added a comment - hi christopher, the method getRequiredMonitorService() is a relict of copying from seleniumhq. i will change the return value to BuildStepMonitor.NONE.
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Marco Machmer
          Path:
          src/main/java/org/jvnet/hudson/plugins/seleniumhtmlreport/SeleniumHtmlReportPublisher.java
          http://jenkins-ci.org/commit/seleniumhtmlreport/a758b0073928ae93a91ba89f2f9bef5f4acccc35
          Log:
          jobs of this plugin can run concurrently
          see Jira issue JENKINS-12345

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Marco Machmer Path: src/main/java/org/jvnet/hudson/plugins/seleniumhtmlreport/SeleniumHtmlReportPublisher.java http://jenkins-ci.org/commit/seleniumhtmlreport/a758b0073928ae93a91ba89f2f9bef5f4acccc35 Log: jobs of this plugin can run concurrently see Jira issue JENKINS-12345
          Hide
          slorenz Stefan Lorenz added a comment -

          Fixed.

          Show
          slorenz Stefan Lorenz added a comment - Fixed.

            People

            • Assignee:
              m211 m211
              Reporter:
              markmandel markmandel
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: