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

Multibranch webhook notifications broken in 2.4.0

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      The implementation of JENKINS-27902 was incompatible for GitStatus.Listener implementations, and did not even update one of the two implementations inside the Git plugin. The result is that in 2.4.0 and later, webhook notifications are not delivered to multibranch projects.

        Attachments

          Issue Links

          is blocking

          New Feature - A new feature of the product, which has yet to be developed. JENKINS-27902 Allow to pass additional request parameter for notifyCommit

          • Major - Major loss of function.
          • Closed
          links to

          Web Link PR 371

          Web Link workflow PR 262 (integration test)

            Activity

            Descending order - Click to sort in ascending order
            Hide
            Permalink
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Jesse Glick
            Path:
            multibranch/src/test/java/org/jenkinsci/plugins/workflow/multibranch/SCMBinderTest.java
            http://jenkins-ci.org/commit/workflow-multibranch-plugin/f00ff182ba2dea3298455db7da99bf7790c7d85f
            Log:
            JENKINS-31884 Integration test for Git webhooks in multibranch projects.
            Originally-Committed-As: 1fd0023a0cfb1f0d0aa0dd069371e58ae24d208d

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Jesse Glick Path: multibranch/src/test/java/org/jenkinsci/plugins/workflow/multibranch/SCMBinderTest.java http://jenkins-ci.org/commit/workflow-multibranch-plugin/f00ff182ba2dea3298455db7da99bf7790c7d85f Log: JENKINS-31884 Integration test for Git webhooks in multibranch projects. Originally-Committed-As: 1fd0023a0cfb1f0d0aa0dd069371e58ae24d208d
            Hide
            Permalink
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Mark Waite
            Path:
            src/main/java/hudson/plugins/git/GitStatus.java
            src/main/java/jenkins/plugins/git/GitSCMSource.java
            http://jenkins-ci.org/commit/git-plugin/a09fcd613e5056e85ca8fea44ea614b62aa9052b
            Log:
            Merge pull request #371 from jglick/GitStatus-multibranch-JENKINS-31884

            JENKINS-31884 Deliver webhooks to all listeners

            Compare: https://github.com/jenkinsci/git-plugin/compare/f00f37972bb3...a09fcd613e50

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Mark Waite Path: src/main/java/hudson/plugins/git/GitStatus.java src/main/java/jenkins/plugins/git/GitSCMSource.java http://jenkins-ci.org/commit/git-plugin/a09fcd613e5056e85ca8fea44ea614b62aa9052b Log: Merge pull request #371 from jglick/GitStatus-multibranch- JENKINS-31884 JENKINS-31884 Deliver webhooks to all listeners Compare: https://github.com/jenkinsci/git-plugin/compare/f00f37972bb3...a09fcd613e50
            Hide
            Permalink
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Jesse Glick
            Path:
            src/main/java/jenkins/plugins/git/GitSCMSource.java
            http://jenkins-ci.org/commit/git-plugin/9d1d316e0e95c99b44862daaba349dbe9b870d00
            Log:
            JENKINS-31884 Override the newest version of onNotifyCommit.

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Jesse Glick Path: src/main/java/jenkins/plugins/git/GitSCMSource.java http://jenkins-ci.org/commit/git-plugin/9d1d316e0e95c99b44862daaba349dbe9b870d00 Log: JENKINS-31884 Override the newest version of onNotifyCommit.
            Hide
            Permalink
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Jesse Glick
            Path:
            src/main/java/hudson/plugins/git/GitStatus.java
            http://jenkins-ci.org/commit/git-plugin/1502180512ec9a2b2008d2c13b214b9a25cd5eb5
            Log:
            [FIXED JENKINS-31884] Since the newest version of onNotifyCommit is called, newer versions must delegate to older versions that may still have been overridden.

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Jesse Glick Path: src/main/java/hudson/plugins/git/GitStatus.java http://jenkins-ci.org/commit/git-plugin/1502180512ec9a2b2008d2c13b214b9a25cd5eb5 Log: [FIXED JENKINS-31884] Since the newest version of onNotifyCommit is called, newer versions must delegate to older versions that may still have been overridden.
            Hide
            Permalink
            abayer Andrew Bayer added a comment -

            Gotcha - just wanted to check.

            Show
            abayer Andrew Bayer added a comment - Gotcha - just wanted to check.
            4 older comments

              People

              Assignee:
              jglick Jesse Glick
              Reporter:
              jglick Jesse Glick
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: