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

Build cause condition fails on matrix projects

XMLWordPrintable

      If using the build cause condition with matrix project, this fails when using with the conditional buildstep plugin, as the build steps are run in the child builds of the parent build, and these are started with the cause upstream(parent) so all of the checking for other cases.

      In this case the we should try looking at the parent build. via AbstractBuild.getRootBuild()

      http://javadoc.jenkins-ci.org/hudson/model/AbstractBuild.html#getRootBuild()

          [JENKINS-14438] Build cause condition fails on matrix projects

          cjo9900 added a comment -

          Test case and fix created and added as pull #7

          https://github.com/jenkinsci/run-condition-plugin/pull/7

          Note: this requires that the minimum Jenkins version is now 1.424 rather than 1.409

          cjo9900 added a comment - Test case and fix created and added as pull #7 https://github.com/jenkinsci/run-condition-plugin/pull/7 Note: this requires that the minimum Jenkins version is now 1.424 rather than 1.409

          SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: cjo9900
          Path:
          pom.xml
          src/test/java/org/jenkins_ci/plugins/run_condition/core/CauseConditionTest.java
          http://jenkins-ci.org/commit/run-condition-plugin/69b09d0a91499bdee3b3c7f2c35ce6e3d2ed2f8b
          Log:
          Test cases to show JENKINS-14438 occuring

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: cjo9900 Path: pom.xml src/test/java/org/jenkins_ci/plugins/run_condition/core/CauseConditionTest.java http://jenkins-ci.org/commit/run-condition-plugin/69b09d0a91499bdee3b3c7f2c35ce6e3d2ed2f8b Log: Test cases to show JENKINS-14438 occuring

          SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: cjo9900
          Path:
          pom.xml
          src/main/java/org/jenkins_ci/plugins/run_condition/core/CauseCondition.java
          http://jenkins-ci.org/commit/run-condition-plugin/1918a2b09e89509c5a055692529eac89b810b345
          Log:
          Use the root build rather that the current build for checking cause

          [FIXED JENKINS-14438]
          Updates to minimum Jenkins version 1.424 for this.

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: cjo9900 Path: pom.xml src/main/java/org/jenkins_ci/plugins/run_condition/core/CauseCondition.java http://jenkins-ci.org/commit/run-condition-plugin/1918a2b09e89509c5a055692529eac89b810b345 Log: Use the root build rather that the current build for checking cause [FIXED JENKINS-14438] Updates to minimum Jenkins version 1.424 for this.

          SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: imod
          Path:
          pom.xml
          src/main/java/org/jenkins_ci/plugins/run_condition/core/CauseCondition.java
          src/test/java/org/jenkins_ci/plugins/run_condition/core/CauseConditionTest.java
          http://jenkins-ci.org/commit/run-condition-plugin/a669d82ff05fd1fde089adc4115facab6bda4e70
          Log:
          [FIXED JENKINS-14438] Merge branch 'ISSUE-14438' of github.com:cjo9900/run-condition-plugin

          Compare: https://github.com/jenkinsci/run-condition-plugin/compare/a3af64462b1b...a669d82ff05f

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: imod Path: pom.xml src/main/java/org/jenkins_ci/plugins/run_condition/core/CauseCondition.java src/test/java/org/jenkins_ci/plugins/run_condition/core/CauseConditionTest.java http://jenkins-ci.org/commit/run-condition-plugin/a669d82ff05fd1fde089adc4115facab6bda4e70 Log: [FIXED JENKINS-14438] Merge branch 'ISSUE-14438' of github.com:cjo9900/run-condition-plugin Compare: https://github.com/jenkinsci/run-condition-plugin/compare/a3af64462b1b...a669d82ff05f

            bap bap
            cjo9900 cjo9900
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: