Details

    • Similar Issues:

      Description

      Jenkins stable release
      Github plugins 1.14.1

      Running a build with the set build status in build step (and the post-build had another status setter), resulted in the following:

      14:38:58 ERROR: Build step failed with exception
      14:38:58 java.lang.NullPointerException
      14:38:58 	at com.cloudbees.jenkins.GitHubSetCommitStatusBuilder.perform(GitHubSetCommitStatusBuilder.java:52)
      14:38:58 	at hudson.tasks.BuildStepMonitor$1.perform(BuildStepMonitor.java:20)
      14:38:58 	at hudson.model.AbstractBuild$AbstractBuildExecution.perform(AbstractBuild.java:785)
      14:38:58 	at hudson.model.Build$BuildExecution.build(Build.java:205)
      14:38:58 	at hudson.model.Build$BuildExecution.doRun(Build.java:162)
      14:38:58 	at hudson.model.AbstractBuild$AbstractBuildExecution.run(AbstractBuild.java:537)
      14:38:58 	at hudson.model.Run.execute(Run.java:1741)
      14:38:58 	at hudson.model.FreeStyleBuild.run(FreeStyleBuild.java:43)
      14:38:58 	at hudson.model.ResourceController.execute(ResourceController.java:98)
      14:38:58 	at hudson.model.Executor.run(Executor.java:408)
      14:38:58 Build step 'Set build status to "pending" on GitHub commit' marked build as failure
      14:38:59 Skipped archiving because build is not successful
      14:38:59 ERROR: Build step failed with exception
      14:38:59 java.lang.NullPointerException
      14:38:59 	at com.cloudbees.jenkins.GitHubCommitNotifier.updateCommitStatus(GitHubCommitNotifier.java:127)
      14:38:59 	at com.cloudbees.jenkins.GitHubCommitNotifier.perform(GitHubCommitNotifier.java:105)
      14:38:59 	at hudson.tasks.BuildStepMonitor$1.perform(BuildStepMonitor.java:20)
      14:38:59 	at hudson.model.AbstractBuild$AbstractBuildExecution.perform(AbstractBuild.java:785)
      14:38:59 	at hudson.model.AbstractBuild$AbstractBuildExecution.performAllBuildSteps(AbstractBuild.java:726)
      14:38:59 	at hudson.model.Build$BuildExecution.post2(Build.java:185)
      14:38:59 	at hudson.model.AbstractBuild$AbstractBuildExecution.post(AbstractBuild.java:671)
      14:38:59 	at hudson.model.Run.execute(Run.java:1766)
      14:38:59 	at hudson.model.FreeStyleBuild.run(FreeStyleBuild.java:43)
      14:38:59 	at hudson.model.ResourceController.execute(ResourceController.java:98)
      14:38:59 	at hudson.model.Executor.run(Executor.java:408)
      14:38:59 Build step 'Set build status on GitHub commit' marked build as failure
      

        Attachments

          Activity

          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Kirill Merkushev
          Path:
          src/main/java/com/cloudbees/jenkins/GitHubCommitNotifier.java
          src/main/java/com/cloudbees/jenkins/GitHubSetCommitStatusBuilder.java
          http://jenkins-ci.org/commit/github-plugin/57d7870f7a170f9b7ae772b3ccc107554cbf6269
          Log:
          [FIXES JENKINS-32132] Check status message is null (in case of wrong deserialization)

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Kirill Merkushev Path: src/main/java/com/cloudbees/jenkins/GitHubCommitNotifier.java src/main/java/com/cloudbees/jenkins/GitHubSetCommitStatusBuilder.java http://jenkins-ci.org/commit/github-plugin/57d7870f7a170f9b7ae772b3ccc107554cbf6269 Log: [FIXES JENKINS-32132] Check status message is null (in case of wrong deserialization)
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Merkushev Kirill
          Path:
          src/main/java/com/cloudbees/jenkins/GitHubCommitNotifier.java
          src/main/java/com/cloudbees/jenkins/GitHubSetCommitStatusBuilder.java
          http://jenkins-ci.org/commit/github-plugin/2c193b499b29cda7723e46329d9c2c57fe73adc9
          Log:
          Merge pull request #103 from lanwen/fix_JENKINS-32132

          [FIXES JENKINS-32132] Check status message is null (in case of wrong …

          Compare: https://github.com/jenkinsci/github-plugin/compare/c8af6ec5af5f...2c193b499b29

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Merkushev Kirill Path: src/main/java/com/cloudbees/jenkins/GitHubCommitNotifier.java src/main/java/com/cloudbees/jenkins/GitHubSetCommitStatusBuilder.java http://jenkins-ci.org/commit/github-plugin/2c193b499b29cda7723e46329d9c2c57fe73adc9 Log: Merge pull request #103 from lanwen/fix_ JENKINS-32132 [FIXES JENKINS-32132] Check status message is null (in case of wrong … Compare: https://github.com/jenkinsci/github-plugin/compare/c8af6ec5af5f...2c193b499b29
          Hide
          lanwen Kirill Merkushev added a comment -

          released as 1.14.2
          (should be available for download after some delay)

          Show
          lanwen Kirill Merkushev added a comment - released as 1.14.2 (should be available for download after some delay)
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Kirill Merkushev
          Path:
          src/test/java/com/cloudbees/jenkins/GitHubCommitNotifierTest.java
          src/test/java/com/cloudbees/jenkins/GitHubSetCommitStatusBuilderTest.java
          src/test/java/com/cloudbees/jenkins/GitHubWebHookFullTest.java
          src/test/java/org/jenkinsci/plugins/github/internal/GitHubClientCacheCleanupTest.java
          src/test/java/org/jenkinsci/plugins/github/test/GHMockRule.java
          src/test/java/org/jenkinsci/plugins/github/test/InjectJenkinsMembersRule.java
          src/test/resources/com/cloudbees/jenkins/GitHubSetCommitStatusBuilderTest/shouldLoadNullStatusMessage/config.xml
          src/test/resources/com/cloudbees/jenkins/GitHubSetCommitStatusBuilderTest/shouldLoadNullStatusMessage/jobs/step/config.xml
          src/test/resources/log4j.properties
          src/test/resources/org/jenkinsci/plugins/github/internal/GitHubClientCacheCleanupTest/user.json
          src/test/resources/org/jenkinsci/plugins/github/test/GHMockRule/repos-repo.json
          src/test/resources/org/jenkinsci/plugins/github/test/GHMockRule/user.json
          http://jenkins-ci.org/commit/github-plugin/4024d6249812fb76cf98fca4bb04098ff47e834e
          Log:
          Test for JENKINS-32132 with rule to mock GH

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Kirill Merkushev Path: src/test/java/com/cloudbees/jenkins/GitHubCommitNotifierTest.java src/test/java/com/cloudbees/jenkins/GitHubSetCommitStatusBuilderTest.java src/test/java/com/cloudbees/jenkins/GitHubWebHookFullTest.java src/test/java/org/jenkinsci/plugins/github/internal/GitHubClientCacheCleanupTest.java src/test/java/org/jenkinsci/plugins/github/test/GHMockRule.java src/test/java/org/jenkinsci/plugins/github/test/InjectJenkinsMembersRule.java src/test/resources/com/cloudbees/jenkins/GitHubSetCommitStatusBuilderTest/shouldLoadNullStatusMessage/config.xml src/test/resources/com/cloudbees/jenkins/GitHubSetCommitStatusBuilderTest/shouldLoadNullStatusMessage/jobs/step/config.xml src/test/resources/log4j.properties src/test/resources/org/jenkinsci/plugins/github/internal/GitHubClientCacheCleanupTest/user.json src/test/resources/org/jenkinsci/plugins/github/test/GHMockRule/repos-repo.json src/test/resources/org/jenkinsci/plugins/github/test/GHMockRule/user.json http://jenkins-ci.org/commit/github-plugin/4024d6249812fb76cf98fca4bb04098ff47e834e Log: Test for JENKINS-32132 with rule to mock GH
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Merkushev Kirill
          Path:
          src/test/java/com/cloudbees/jenkins/GitHubCommitNotifierTest.java
          src/test/java/com/cloudbees/jenkins/GitHubSetCommitStatusBuilderTest.java
          src/test/java/com/cloudbees/jenkins/GitHubWebHookFullTest.java
          src/test/java/org/jenkinsci/plugins/github/internal/GitHubClientCacheCleanupTest.java
          src/test/java/org/jenkinsci/plugins/github/test/GHMockRule.java
          src/test/java/org/jenkinsci/plugins/github/test/InjectJenkinsMembersRule.java
          src/test/resources/com/cloudbees/jenkins/GitHubSetCommitStatusBuilderTest/shouldLoadNullStatusMessage/config.xml
          src/test/resources/com/cloudbees/jenkins/GitHubSetCommitStatusBuilderTest/shouldLoadNullStatusMessage/jobs/step/config.xml
          src/test/resources/log4j.properties
          src/test/resources/org/jenkinsci/plugins/github/internal/GitHubClientCacheCleanupTest/user.json
          src/test/resources/org/jenkinsci/plugins/github/test/GHMockRule/repos-repo.json
          src/test/resources/org/jenkinsci/plugins/github/test/GHMockRule/user.json
          http://jenkins-ci.org/commit/github-plugin/5e14ccae9de8c0ba5226ac4008f05c6f7bbd0a79
          Log:
          Merge pull request #104 from lanwen/JENKINS-32132_tests

          Test for JENKINS-32132 with rule to mock GH

          Compare: https://github.com/jenkinsci/github-plugin/compare/4197fdf14807...5e14ccae9de8

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Merkushev Kirill Path: src/test/java/com/cloudbees/jenkins/GitHubCommitNotifierTest.java src/test/java/com/cloudbees/jenkins/GitHubSetCommitStatusBuilderTest.java src/test/java/com/cloudbees/jenkins/GitHubWebHookFullTest.java src/test/java/org/jenkinsci/plugins/github/internal/GitHubClientCacheCleanupTest.java src/test/java/org/jenkinsci/plugins/github/test/GHMockRule.java src/test/java/org/jenkinsci/plugins/github/test/InjectJenkinsMembersRule.java src/test/resources/com/cloudbees/jenkins/GitHubSetCommitStatusBuilderTest/shouldLoadNullStatusMessage/config.xml src/test/resources/com/cloudbees/jenkins/GitHubSetCommitStatusBuilderTest/shouldLoadNullStatusMessage/jobs/step/config.xml src/test/resources/log4j.properties src/test/resources/org/jenkinsci/plugins/github/internal/GitHubClientCacheCleanupTest/user.json src/test/resources/org/jenkinsci/plugins/github/test/GHMockRule/repos-repo.json src/test/resources/org/jenkinsci/plugins/github/test/GHMockRule/user.json http://jenkins-ci.org/commit/github-plugin/5e14ccae9de8c0ba5226ac4008f05c6f7bbd0a79 Log: Merge pull request #104 from lanwen/ JENKINS-32132 _tests Test for JENKINS-32132 with rule to mock GH Compare: https://github.com/jenkinsci/github-plugin/compare/4197fdf14807...5e14ccae9de8

            People

            • Assignee:
              lanwen Kirill Merkushev
              Reporter:
              intel352 Jonathan Langevin
            • Votes:
              1 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: