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

Publishing Selenium report blocks concurrent builds

      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.

          [JENKINS-12345] Publishing Selenium report blocks concurrent builds

          markmandel added a comment - - edited

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

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

          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.

          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.

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

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

          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.

          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.

          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

          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

          Stefan Lorenz added a comment -

          Fixed.

          Stefan Lorenz added a comment - Fixed.

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

              Created:
              Updated:
              Resolved: