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

earliest git hash passed by parameterized build trigger when downstream job was triggered multiple times

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      Job A triggers job B and passes its git hash along. Job A is fast but job B is slow. So there are times when job A finishes and triggers job B while job B is still running and processing the earlier git hash. When job B finishes its current run, the next run would be triggered by multiple runs of job A. The parameterized trigger passes the git hash from the earliest trigger by job A. As a result, git hashes from the later triggers are never processed by job B. The parameterized trigger should pass the git hash from the latest trigger by job A.

        Attachments

          Activity

          Hide
          cjo9900 cjo9900 added a comment -

          Apologies, looks like I missed passing that flag to the Action that is created, so it never took effect.

          https://github.com/jenkinsci/git-plugin/pull/129

          Show
          cjo9900 cjo9900 added a comment - Apologies, looks like I missed passing that flag to the Action that is created, so it never took effect. https://github.com/jenkinsci/git-plugin/pull/129
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: cjo9900
          Path:
          src/main/java/hudson/plugins/git/GitRevisionBuildParameters.java
          http://jenkins-ci.org/commit/git-plugin/951517ccd2b99a8a4464b444e03976a1ff958239
          Log:
          [FIXED JENKINS-15160] Pass combineCommits to action

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: cjo9900 Path: src/main/java/hudson/plugins/git/GitRevisionBuildParameters.java http://jenkins-ci.org/commit/git-plugin/951517ccd2b99a8a4464b444e03976a1ff958239 Log: [FIXED JENKINS-15160] Pass combineCommits to action
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Nicolas De loof
          Path:
          src/main/java/hudson/plugins/git/GitRevisionBuildParameters.java
          http://jenkins-ci.org/commit/git-plugin/1349eaba9c5767e9bf52ab27b6c31681f0c13733
          Log:
          Merge pull request #129 from cjo9900/JENKINS-15160-2

          [FIXED JENKINS-15160] Pass combineCommits to action

          Compare: https://github.com/jenkinsci/git-plugin/compare/c72b83216954...1349eaba9c57


          You received this message because you are subscribed to the Google Groups "Jenkins Commits" group.
          To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-commits+unsubscribe@googlegroups.com.
          For more options, visit https://groups.google.com/groups/opt_out.

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Nicolas De loof Path: src/main/java/hudson/plugins/git/GitRevisionBuildParameters.java http://jenkins-ci.org/commit/git-plugin/1349eaba9c5767e9bf52ab27b6c31681f0c13733 Log: Merge pull request #129 from cjo9900/ JENKINS-15160 -2 [FIXED JENKINS-15160] Pass combineCommits to action Compare: https://github.com/jenkinsci/git-plugin/compare/c72b83216954...1349eaba9c57 – You received this message because you are subscribed to the Google Groups "Jenkins Commits" group. To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-commits+unsubscribe@googlegroups.com. For more options, visit https://groups.google.com/groups/opt_out .
          Hide
          inger Gergely Nagy added a comment - - edited

          Thank you very much for this - it would fix an important regression for me (my long running tests are flooding the queue with the current version - this started to happen with the latest upgrade).
          Any way to release it soon?

          Show
          inger Gergely Nagy added a comment - - edited Thank you very much for this - it would fix an important regression for me (my long running tests are flooding the queue with the current version - this started to happen with the latest upgrade). Any way to release it soon?
          Hide
          hlau hlau added a comment -

          Indeed I'm also still having this problem. Have to manually kill some of the jobs in the long queue. Quite a hassle. Please release the fix soon.

          Show
          hlau hlau added a comment - Indeed I'm also still having this problem. Have to manually kill some of the jobs in the long queue. Quite a hassle. Please release the fix soon.

            People

            • Assignee:
              cjo9900 cjo9900
              Reporter:
              hlau hlau
            • Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: