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

P4 plugin does not poll when builds running

XMLWordPrintable

    • Icon: Improvement Improvement
    • Resolution: Fixed
    • Icon: Critical Critical
    • p4-plugin
    • None
    • p4-plugin 1.3.9

      The p4 plugin does not poll while builds are running, I get the message:

      Build in progress, polling delayed.

      This makes some small sense for normal projects, but when using the pipeline plugin it makes things difficult as pipeline scripts may pause for a long time while not using an executor or a workspace to let other jobs build.

      The offending code is in PerforceScm.java

                      // Delay polling if build is in progress                                                                                                                                                           
                if (job.isBuilding()) {
                        listener.getLogger().println("Build in progress, polling delayed.");
			return PollingResult.NO_CHANGES;
                }

          [JENKINS-34052] P4 plugin does not poll when builds running

          Kyle Mott added a comment -

          This is a problem for us as well, since p4 plugin returns NO_CHANGES, a job that allows concurrent builds will never fire again if that job is already building.

          I'd also like to bump this up in priority to Major, but not sure what the proper procedure for that is.

          Kyle Mott added a comment - This is a problem for us as well, since p4 plugin returns NO_CHANGES, a job that allows concurrent builds will never fire again if that job is already building. I'd also like to bump this up in priority to Major, but not sure what the proper procedure for that is.

          Priya Gangaraju added a comment - - edited

          This is problem for us as well and would like to bump this up in priority.

          Priya Gangaraju added a comment - - edited This is problem for us as well and would like to bump this up in priority.

          Robby Pocase added a comment -

          This is a huge issue for users with long running pipelines (particularly if long layers of test are included but you may want multiple compile portions running).

          Robby Pocase added a comment - This is a huge issue for users with long running pipelines (particularly if long layers of test are included but you may want multiple compile portions running).

          Robby Pocase added a comment -

          Started working on this and saw the related ticket. Reaching out on the other ticket to see if one of these should really be marked as a duplicate.

          Robby Pocase added a comment - Started working on this and saw the related ticket. Reaching out on the other ticket to see if one of these should really be marked as a duplicate.

          SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Robby Pocase
          Path:
          src/main/java/org/jenkinsci/plugins/p4/PerforceScm.java
          http://jenkins-ci.org/commit/p4-plugin/52bc05a5c16f9c30903b4bc92e43531b7a918e0d
          Log:
          [FIX JENKINS-34052] Poll for concurrent jobs

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Robby Pocase Path: src/main/java/org/jenkinsci/plugins/p4/PerforceScm.java http://jenkins-ci.org/commit/p4-plugin/52bc05a5c16f9c30903b4bc92e43531b7a918e0d Log: [FIX JENKINS-34052] Poll for concurrent jobs

          SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Paul Allen
          Path:
          src/main/java/org/jenkinsci/plugins/p4/PerforceScm.java
          src/main/java/org/jenkinsci/plugins/p4/tagging/TagAction.java
          http://jenkins-ci.org/commit/p4-plugin/c9159c2c0ec587d1a84f2fcccba6e901b4fece5f
          Log:
          Merge pull request #63 from ADTRAN/JENKINS-34052

          [FIX JENKINS-34052] Poll for concurrent jobs

          Compare: https://github.com/jenkinsci/p4-plugin/compare/b22fe610f82c...c9159c2c0ec5

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Paul Allen Path: src/main/java/org/jenkinsci/plugins/p4/PerforceScm.java src/main/java/org/jenkinsci/plugins/p4/tagging/TagAction.java http://jenkins-ci.org/commit/p4-plugin/c9159c2c0ec587d1a84f2fcccba6e901b4fece5f Log: Merge pull request #63 from ADTRAN/ JENKINS-34052 [FIX JENKINS-34052] Poll for concurrent jobs Compare: https://github.com/jenkinsci/p4-plugin/compare/b22fe610f82c...c9159c2c0ec5

          Paul Allen added a comment -

          Released in 1.8.2

          Paul Allen added a comment - Released in 1.8.2

            rpocase Robby Pocase
            pyssling Nils Carlson
            Votes:
            7 Vote for this issue
            Watchers:
            13 Start watching this issue

              Created:
              Updated:
              Resolved: