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

NPE in github status notification, if build not triggered by github push trigger

    Details

    • Similar Issues:

      Description

      I tried to use the github status notification post build action on a job that is triggered by a manually configured remote URL (and not by a standard github push trigger). The GitHubCommitNotifier class does not expect this and crashes when querying the push trigger.

      java.lang.NullPointerException
      at com.cloudbees.jenkins.GitHubCommitNotifier.perform(GitHubCommitNotifier.java:52)
      at hudson.tasks.BuildStepMonitor$1.perform(BuildStepMonitor.java:19)
      at hudson.model.AbstractBuild$AbstractBuildExecution.perform(AbstractBuild.java:804)
      at hudson.model.AbstractBuild$AbstractBuildExecution.performAllBuildSteps(AbstractBuild.java:776)
      at hudson.model.Build$BuildExecution.post2(Build.java:183)
      at hudson.model.AbstractBuild$AbstractBuildExecution.post(AbstractBuild.java:726)
      at hudson.model.Run.execute(Run.java:1601)
      at hudson.model.FreeStyleBuild.run(FreeStyleBuild.java:46)
      at hudson.model.ResourceController.execute(ResourceController.java:88)
      at hudson.model.Executor.run(Executor.java:241)

        Attachments

          Activity

          bananeweizen Michael Keppler created issue -
          Hide
          kohsuke Kohsuke Kawaguchi added a comment -

          Fixed in 1.7

          Show
          kohsuke Kohsuke Kawaguchi added a comment - Fixed in 1.7
          kohsuke Kohsuke Kawaguchi made changes -
          Field Original Value New Value
          Status Open [ 1 ] Resolved [ 5 ]
          Assignee Kohsuke Kawaguchi [ kohsuke ]
          Resolution Fixed [ 1 ]
          Hide
          pacso Jon Pascoe added a comment -

          Whilst 1.7 no longer gives a NPE, the commit status notification does not seem to be pushed to GitHub.

          Show
          pacso Jon Pascoe added a comment - Whilst 1.7 no longer gives a NPE, the commit status notification does not seem to be pushed to GitHub.
          Hide
          pacso Jon Pascoe added a comment -

          OK - Ignore that comment. It's all working fine. Somehow my API key had been wiped out when I updated the plugin, and rather than give me any error in the output it silently ignored the plugin.

          Show
          pacso Jon Pascoe added a comment - OK - Ignore that comment. It's all working fine. Somehow my API key had been wiped out when I updated the plugin, and rather than give me any error in the output it silently ignored the plugin.
          Hide
          nukulb Nukul Bhasin added a comment -

          Also reproducible if you miss the 'GITHUB project' field in your configuration.

          Show
          nukulb Nukul Bhasin added a comment - Also reproducible if you miss the 'GITHUB project' field in your configuration.
          rtyler R. Tyler Croy made changes -
          Workflow JNJira [ 149463 ] JNJira + In-Review [ 193138 ]

            People

            • Assignee:
              kohsuke Kohsuke Kawaguchi
              Reporter:
              bananeweizen Michael Keppler
            • Votes:
              4 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: