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

Concurrent builds are serialized when publishing

        [JENKINS-43658] Concurrent builds are serialized when publishing

        Jakub Bochenski added a comment - npiguet jieryn ssogabe mixalturek bump?

        Hi jbochenski,

        Can you provide a scenario I can use to reproduce the bug?

        Currently, I've set up a job that publishes a report and am running builds simultaneously, but can't reproduce it. The issue is that if multiple builds are all in the publish SLOCC report step at the same time, they will all need to finish before any of the runs can end, right?

        Ivaïlo Marinkov added a comment - Hi jbochenski , Can you provide a scenario I can use to reproduce the bug? Currently, I've set up a job that publishes a report and am running builds simultaneously, but can't reproduce it. The issue is that if multiple builds are all in the publish SLOCC report step at the same time, they will all need to finish before any of the runs can end, right?

        imarinkov to reproduce this you need at least two builds (e.g. #1 and #2). If #1 is started before #2, but #2 finishes before #1 then #2 will wait until #1 is done.

        The issue is discussed in detail in JENKINS-9913 as it affects multiple plugins

        Jakub Bochenski added a comment - imarinkov to reproduce this you need at least two builds (e.g. #1 and #2). If #1 is started before #2, but #2 finishes before #1 then #2 will wait until #1 is done. The issue is discussed in detail in JENKINS-9913 as it affects multiple plugins

        Jakub Bochenski added a comment - - edited

        imarinkov here is a fix for the same issue in email-ext by a cloudbees dev I think you can safely follow that: https://github.com/jenkinsci/email-ext-plugin/commit/6fb41818d63ca0067171f8c01fcdbed196335b44

        Jakub Bochenski added a comment - - edited imarinkov here is a fix for the same issue in email-ext by a cloudbees dev I think you can safely follow that: https://github.com/jenkinsci/email-ext-plugin/commit/6fb41818d63ca0067171f8c01fcdbed196335b44

        SCM/JIRA link daemon added a comment - Code changed in jenkins User: jakub-bochenski Path: src/main/java/hudson/plugins/sloccount/SloccountPublisher.java http://jenkins-ci.org/commit/sloccount-plugin/d205dc8a50f4f488657fac2ee293235fc31d1712 Log: JENKINS-43658 serializing builds is not neccessary c.f https://github.com/jenkinsci/email-ext-plugin/commit/6fb41818d63ca0067171f8c01fcdbed196335b44 https://github.com/jenkinsci/email-ext-plugin/commit/e7bc7c814709e433d98fab1b31f340380043caf5 https://github.com/jenkinsci/slack-plugin/commit/832e7b50028ca086dd49311ce454e7a19c9a3af9

        Code changed in jenkins
        User: Ivaïlo Marinkov
        Path:
        src/main/java/hudson/plugins/sloccount/SloccountPublisher.java
        http://jenkins-ci.org/commit/sloccount-plugin/b0c6e87d6e06eba24b7b17f9c50ea7d05e8fd5b3
        Log:
        Merge pull request #51 from jakub-bochenski/patch-1

        JENKINS-43658 serializing builds is not neccessary

        Compare: https://github.com/jenkinsci/sloccount-plugin/compare/b6a828e93e7e...b0c6e87d6e06

        SCM/JIRA link daemon added a comment - Code changed in jenkins User: Ivaïlo Marinkov Path: src/main/java/hudson/plugins/sloccount/SloccountPublisher.java http://jenkins-ci.org/commit/sloccount-plugin/b0c6e87d6e06eba24b7b17f9c50ea7d05e8fd5b3 Log: Merge pull request #51 from jakub-bochenski/patch-1 JENKINS-43658 serializing builds is not neccessary Compare: https://github.com/jenkinsci/sloccount-plugin/compare/b6a828e93e7e...b0c6e87d6e06

        jbochenski, your change has been merged and released with v. 1.24 of the plugin.

        Ivaïlo Marinkov added a comment - jbochenski , your change has been merged and released with v. 1.24 of the plugin.

          imarinkov Ivaïlo Marinkov
          jbochenski Jakub Bochenski
          Votes:
          0 Vote for this issue
          Watchers:
          3 Start watching this issue

            Created:
            Updated:
            Resolved: