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

Join Plugin ignores dependencies wrapped with flexible-publish

    Details

    • Similar Issues:

      Description

      Join trigger doesn't work in the expected way when combined with Flexible Publish plugin.

      • Project A triggers project B, project C conditionally (using flexible-publish).
      • Project A triggers project D via Join plugin.

      In this case, join plugin works as followings unexpectedly:

      • (not critical) Project B,C,D will be displayed in the project status page.
        • Expected bahaivor: Only project B, C will be displayed.
      • (critical) Project D is triggered just after project A finished.
        • Project D is triggered after project B, C finished.

      This is the behavior just like when project A has no project dependencies except join trigger.
      This means join trigger fails to find out dependencies wrapped with flexible-publish.


      Hi,

      I want to check some env variable before triggering 2 downstream projects which are joint after their build to 1 other project.

      So I declared my conditions then add a 'build downstream project' action with my two projects plus a 'Join Trigger' action to join the projects to another project once they have built successfully.

      When I have a look of my main project dashboard I see it triggers all of the 3 projects as downstream whereas It should trigger only the first two ones

        Attachments

          Activity

          Hide
          edgar0119 Edgar Yu added a comment -

          This bug is solved in the join-plugin version 1.20

          Reference:
          1. join-plugin GitHub releases page
          2. https://wiki.jenkins-ci.org/display/JENKINS/Join+Plugin

          Show
          edgar0119 Edgar Yu added a comment - This bug is solved in the join-plugin version 1.20 Reference: 1. join-plugin GitHub releases page 2. https://wiki.jenkins-ci.org/display/JENKINS/Join+Plugin
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Michael Donohue
          Path:
          pom.xml
          src/main/java/join/JoinTrigger.java
          http://jenkins-ci.org/commit/join-plugin/7ff674339affb5d72116c405ace1d1a90acc63c3
          Log:
          Merge pull request #10 from Edgar0119/master

          (Bug Fixed!)JENKINS-28709 Join Plugin ignores dependencies wrapped with flexible-…

          Compare: https://github.com/jenkinsci/join-plugin/compare/54908f824cc4...7ff674339aff

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Michael Donohue Path: pom.xml src/main/java/join/JoinTrigger.java http://jenkins-ci.org/commit/join-plugin/7ff674339affb5d72116c405ace1d1a90acc63c3 Log: Merge pull request #10 from Edgar0119/master (Bug Fixed!) JENKINS-28709 Join Plugin ignores dependencies wrapped with flexible-… Compare: https://github.com/jenkinsci/join-plugin/compare/54908f824cc4...7ff674339aff
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: 俞韋廷
          Path:
          pom.xml
          src/main/java/join/JoinTrigger.java
          http://jenkins-ci.org/commit/join-plugin/246a1fb66f5c39b5f36489014269937668d58412
          Log:
          JENKINS-28709 Join Plugin ignores dependencies wrapped with flexible-publish

          (https://issues.jenkins-ci.org/browse/JENKINS-28709)

          Now it can handle "hudson.plugins.parameterizedtrigger.BuildTrigger" in Flexible publisher!

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: 俞韋廷 Path: pom.xml src/main/java/join/JoinTrigger.java http://jenkins-ci.org/commit/join-plugin/246a1fb66f5c39b5f36489014269937668d58412 Log: JENKINS-28709 Join Plugin ignores dependencies wrapped with flexible-publish ( https://issues.jenkins-ci.org/browse/JENKINS-28709 ) Now it can handle "hudson.plugins.parameterizedtrigger.BuildTrigger" in Flexible publisher!
          Hide
          edgar0119 Edgar Yu added a comment -

          Hello, I has pull request again to fix this bug. https://github.com/jenkinsci/join-plugin/pull/10

          Show
          edgar0119 Edgar Yu added a comment - Hello, I has pull request again to fix this bug. https://github.com/jenkinsci/join-plugin/pull/10
          Hide
          mdonohue mdonohue added a comment -

          I commented on the pull request - it looks like whitespace was changed as part of the commit, making the diff difficult to read.
          It is recommended practice on any project to separate whitespace changes from content changes, so I can't accept this as-is.

          Show
          mdonohue mdonohue added a comment - I commented on the pull request - it looks like whitespace was changed as part of the commit, making the diff difficult to read. It is recommended practice on any project to separate whitespace changes from content changes, so I can't accept this as-is.

            People

            • Assignee:
              mdonohue mdonohue
              Reporter:
              raphc Raphael CHAUMIER
            • Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: