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

PeepholePermalink RunListenerImpl oncompleted should be triggered before downstream builds are triggered

    Details

    • Type: Bug
    • Status: Resolved (View Workflow)
    • Priority: Major
    • Resolution: Fixed
    • Component/s: core
    • Labels:
      None
    • Similar Issues:

      Description

      Scenario
      ProjectA triggers ProjectB
      ProjectB has a RunParameter for ProjectA with a filter set for Successful Builds Only. The Run Parameter uses the PeepholePermalink to determine the LastSuccessful build.

      When ProjectA completes successfully, it will trigger a build of ProjectB before the PeepholePermalink cache is updated. This causes the RunParameter to default to the previous successful build of ProjectA, not the one which has just completed.

        Attachments

          Issue Links

            Activity

            Hide
            olivergondza Oliver Gondža added a comment -
            Show
            olivergondza Oliver Gondža added a comment - Giving this a try in https://github.com/jenkinsci/jenkins/pull/1628
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Oliver Gondža
            Path:
            test/src/test/java/hudson/tasks/BuildTriggerTest.java
            http://jenkins-ci.org/commit/jenkins/021c1b381deec7cab185db4ccaa59ad2b68c4b7a
            Log:
            JENKINS-20989 Reproduce

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Oliver Gondža Path: test/src/test/java/hudson/tasks/BuildTriggerTest.java http://jenkins-ci.org/commit/jenkins/021c1b381deec7cab185db4ccaa59ad2b68c4b7a Log: JENKINS-20989 Reproduce
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Oliver Gondža
            Path:
            core/src/main/java/hudson/model/Run.java
            http://jenkins-ci.org/commit/jenkins/ca5b9b667231fb4805e6001910dee1a26c5d4c68
            Log:
            [FIXED JENKINS-20989] PeepholePermalink RunListenerImpl oncompleted should be triggered before downstream builds are triggered

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Oliver Gondža Path: core/src/main/java/hudson/model/Run.java http://jenkins-ci.org/commit/jenkins/ca5b9b667231fb4805e6001910dee1a26c5d4c68 Log: [FIXED JENKINS-20989] PeepholePermalink RunListenerImpl oncompleted should be triggered before downstream builds are triggered
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Oliver Gondža
            Path:
            core/src/main/java/hudson/model/Run.java
            test/src/test/java/hudson/tasks/BuildTriggerTest.java
            http://jenkins-ci.org/commit/jenkins/ad56599e5c54d9a5f850f365ca0ff969d7c7f87d
            Log:
            Merge pull request #1628 from olivergondza/JENKINS-27635

            JENKINS-20989 PeepholePermalink RunListenerImpl oncompleted should be triggered before downstream builds are triggered

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Oliver Gondža Path: core/src/main/java/hudson/model/Run.java test/src/test/java/hudson/tasks/BuildTriggerTest.java http://jenkins-ci.org/commit/jenkins/ad56599e5c54d9a5f850f365ca0ff969d7c7f87d Log: Merge pull request #1628 from olivergondza/ JENKINS-27635 JENKINS-20989 PeepholePermalink RunListenerImpl oncompleted should be triggered before downstream builds are triggered
            Hide
            dogfood dogfood added a comment -

            Integrated in jenkins_main_trunk #4065

            Result = SUCCESS

            Show
            dogfood dogfood added a comment - Integrated in jenkins_main_trunk #4065 Result = SUCCESS

              People

              • Assignee:
                olivergondza Oliver Gondža
                Reporter:
                gcummings Geoff Cummings
              • Votes:
                0 Vote for this issue
                Watchers:
                7 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: