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

Publishing Selenium report blocks concurrent builds

    Details

      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.

        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

          People

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

            Dates

            • Created:
              Updated: