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

Allow disabling the default commit status notifier

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      In JENKINS-33151, giving users the option to disable the default commit status notifier was raised (in lieu of more complex configuration, such as the ability to change the context string, which is hard-coded to 'Jenkins': https://github.com/jenkinsci/github-branch-source-plugin/blob/master@%7B2016-06-08%7D/src/main/java/org/jenkinsci/plugins/github_branch_source/GitHubBuildStatusNotification.java#L78)

      I think there's probably a use case for this, and it would allow users with more complicated status logic (such as in that ticket) to implement their own, and just opt out of the default one.

        Attachments

          Issue Links

            Activity

            Hide
            myoung34 marc young added a comment - - edited

            This bug is annoying my coworkers. We use a Jenkinsfile to test branches or deploy based on boolean parameter

            IS_DEPLOY

            Everything works great, except if you deploy jenkins sends a pass/fail to the branch/PR which isn't what I want, i only care that the tests send that pass/fail.

            Show
            myoung34 marc young added a comment - - edited This bug is annoying my coworkers. We use a Jenkinsfile to test branches or deploy based on boolean parameter IS_DEPLOY Everything works great, except if you deploy jenkins sends a pass/fail to the branch/PR which isn't what I want, i only care that the tests send that pass/fail.
            Hide
            sparklepony Sparkle Pony added a comment -

            I'd also like to give a +1 that it's a bummer I can't turn these off. I'm using my own custom notification, so having these in there as well creates confusing clutter that I have to explain to my coworkers.

            Show
            sparklepony Sparkle Pony added a comment - I'd also like to give a +1 that it's a bummer I can't turn these off. I'm using my own custom notification, so having these in there as well creates confusing clutter that I have to explain to my coworkers.
            Hide
            allan_burdajewicz Allan BURDAJEWICZ added a comment -

            I can see the PR has been closed. Are there still plans to implement this improvement ? Stephen Connolly Vincent Latombe

            Show
            allan_burdajewicz Allan BURDAJEWICZ added a comment - I can see the PR has been closed. Are there still plans to implement this improvement ? Stephen Connolly Vincent Latombe
            Hide
            jamesdumay James Dumay added a comment -

            There is a plugin that can provide this as a trait to the Github branch source.

            https://github.com/bluesliverx/disable-github-multibranch-status-plugin

            Show
            jamesdumay James Dumay added a comment - There is a plugin that can provide this as a trait to the Github branch source. https://github.com/bluesliverx/disable-github-multibranch-status-plugin
            Hide
            akom Alexander Komarov added a comment - - edited

            I didn't notice this ticket until now - I just implemented a somewhat more flexible version of this as a PR to github-branch-source in JENKINS-48408 (oops).

            https://github.com/jenkinsci/github-branch-source-plugin/pull/169

            The advantage is that you get an option to hide the Jenkins URL but still have notifications in github.

            Show
            akom Alexander Komarov added a comment - - edited I didn't notice this ticket until now - I just implemented a somewhat more flexible version of this as a PR to github-branch-source in  JENKINS-48408 (oops). https://github.com/jenkinsci/github-branch-source-plugin/pull/169 The advantage is that you get an option to hide the Jenkins URL but still have notifications in github.

              People

              • Assignee:
                vlatombe Vincent Latombe
                Reporter:
                dominics Dominic Scheirlinck
              • Votes:
                13 Vote for this issue
                Watchers:
                26 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: